linux-cifs.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Re: [Linux-cifsd-devel] [PATCH] ksmbd-tools: add support for SMB3 multichannel
       [not found] ` <20210614084135.19753-1-namjae.jeon@samsung.com>
@ 2021-07-16  7:50   ` Stefan Metzmacher
  2021-07-16  7:56     ` Namjae Jeon
  0 siblings, 1 reply; 2+ messages in thread
From: Stefan Metzmacher @ 2021-07-16  7:50 UTC (permalink / raw)
  To: Namjae Jeon, linux-cifsd-devel, linux-cifs


[-- Attachment #1.1: Type: text/plain, Size: 1307 bytes --]


Hi Namjae,

> Add support for SMB3 multichannel.
> 
> Signed-off-by: Namjae Jeon <namjae.jeon@samsung.com>
> ---
>  Documentation/configuration.txt | 3 +++
>  include/linux/ksmbd_server.h    | 1 +
>  lib/config_parser.c             | 9 +++++++++
>  3 files changed, 13 insertions(+)
> 
> diff --git a/Documentation/configuration.txt b/Documentation/configuration.txt
> index f38aceb..1289cbf 100644
> --- a/Documentation/configuration.txt
> +++ b/Documentation/configuration.txt
> @@ -109,6 +109,9 @@ Define ksmbd configuration parameters list.
>  		host where ksmbd runs. the format is "cifs/<FQDN>". if this
>  		option is not given, ksmbd sets "cifs" to the service name and
>  		try to get the host FQDN using getaddrinfo(3).
> +	- server multi channel support
> +		This boolean parameter controls whether ksmbd will support
> +		SMB3 multi-channel.

Did you actually implemented all replay related features?
- Create replay
- Lock replay
- Write/IOCtl/SetInfo replay
- Lease/OplockBreak replay

Otherwise you should mark this as experimental,
as it's likely to end with potential data coruption.
That's why Samba had this flagged as experimental up to now,
Samba 4.15 will be the first release with all required features
in order to enable multi channel.

metze


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

* RE: [Linux-cifsd-devel] [PATCH] ksmbd-tools: add support for SMB3 multichannel
  2021-07-16  7:50   ` [Linux-cifsd-devel] [PATCH] ksmbd-tools: add support for SMB3 multichannel Stefan Metzmacher
@ 2021-07-16  7:56     ` Namjae Jeon
  0 siblings, 0 replies; 2+ messages in thread
From: Namjae Jeon @ 2021-07-16  7:56 UTC (permalink / raw)
  To: 'Stefan Metzmacher'; +Cc: linux-cifsd-devel, linux-cifs

> Hi Namjae,
Hi Metze,

> 
> > Add support for SMB3 multichannel.
> >
> > Signed-off-by: Namjae Jeon <namjae.jeon@samsung.com>
> > ---
> >  Documentation/configuration.txt | 3 +++
> >  include/linux/ksmbd_server.h    | 1 +
> >  lib/config_parser.c             | 9 +++++++++
> >  3 files changed, 13 insertions(+)
> >
> > diff --git a/Documentation/configuration.txt
> > b/Documentation/configuration.txt index f38aceb..1289cbf 100644
> > --- a/Documentation/configuration.txt
> > +++ b/Documentation/configuration.txt
> > @@ -109,6 +109,9 @@ Define ksmbd configuration parameters list.
> >  		host where ksmbd runs. the format is "cifs/<FQDN>". if this
> >  		option is not given, ksmbd sets "cifs" to the service name and
> >  		try to get the host FQDN using getaddrinfo(3).
> > +	- server multi channel support
> > +		This boolean parameter controls whether ksmbd will support
> > +		SMB3 multi-channel.
> 
> Did you actually implemented all replay related features?
> - Create replay
> - Lock replay
> - Write/IOCtl/SetInfo replay
> - Lease/OplockBreak replay
Not yet.
> 
> Otherwise you should mark this as experimental, as it's likely to end with potential data coruption.
> That's why Samba had this flagged as experimental up to now, Samba 4.15 will be the first release with
> all required features in order to enable multi channel.
Okay. I will mark it as experimental till relay features are implemented.

Thanks for your review!
> 
> metze



^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2021-07-16  7:56 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <CGME20210614085106epcas1p4143f68f380c480f5f8a504370b10a969@epcas1p4.samsung.com>
     [not found] ` <20210614084135.19753-1-namjae.jeon@samsung.com>
2021-07-16  7:50   ` [Linux-cifsd-devel] [PATCH] ksmbd-tools: add support for SMB3 multichannel Stefan Metzmacher
2021-07-16  7:56     ` Namjae Jeon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).