[PATCH] SMB2 AAPL create context
Stefan (metze) Metzmacher
metze at samba.org
Fri Nov 21 18:39:16 MST 2014
Hi Ralph.
>>>> On Fri, Oct 31, 2014 at 01:58:38PM -0700, Jeremy Allison wrote:
>>>>> Ralph, can you re-post the AAPL create context
>>>>> patch with your latest changes ?
>>>>>
>>>>> I'd love to get this reviewed and into the
>>>>> code base now you're a Team member :-).
>>>>
>>>> that would be great!
>>>>
>>>> Patchset attached, couldn't rebase on master as sn is still down.
>>>
>>>> diff --git a/source3/smbd/globals.h b/source3/smbd/globals.h
>>>> index d0015d5..5137e5e 100644
>>>> --- a/source3/smbd/globals.h
>>>> +++ b/source3/smbd/globals.h
>>>> @@ -504,6 +504,9 @@ struct smbXsrv_connection {
>>>> uint32_t max_trans;
>>>> uint32_t max_read;
>>>> uint32_t max_write;
>>>> + bool use_aapl_crtctx;
>>>> + bool smb2_crtctx_aapl_readdir_attr;
>>>> + bool smb2_crtctx_aapl_unix_info;
>>>> } server;
>>>>
>>>> struct smbd_smb2_request *requests;
>>>
>>> OK, adding these into the struct smbXsrv_connection is
>>> *so* the wrong place to put these :-).
>>
>> :)
>>
>>>
>>> I'm not sure what the right place is yet, but I don't
>>> think it's there... :-).
>>
>> I'd highly appreciate a recommendation where to hang them off, in the
>> meantime I'll poke around some more looking for the right place.
>
> smbXsrv_connection also has smb1.unix_info.client_cap_low which is
> used similiarily in trans2.c with a bunch of feature flags.
>
> I've looked arond some more, but couldn't come up with any other place
> to hang the AAPL flags off.
If we need to access this in vfs modules it should be in
smbd_server_connection,
as mentioned before can you please write some torture tests which
demonstrate
this needs to be set for the whole transport connection.
Does the latest Apple version support SMB3 multi-channel?
It would also be helpful to ask the Apple developers for some documentation
or hints regarding these questions.
metze
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: OpenPGP digital signature
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20141122/091fc4c2/attachment.pgp>
More information about the samba-technical
mailing list