Re: [PATCH] smb3: simplify code by removing CONFIG_CIFS_SMB311

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Jun 29, 2018 at 8:52 AM Pavel Shilovsky <piastryyy@xxxxxxxxx> wrote:
>
> чт, 28 июн. 2018 г. в 23:54, Aurélien Aptel via samba-technical
> <samba-technical@xxxxxxxxxxxxxxx>:
> >
> > Steve French <smfrench@xxxxxxxxx> writes:
> > > Ronnie,
> > > What about the attached wording, slightly updated patch - I made minor
> > > changes to remove the second mention
> > > of SMB3.1.1 and to correct a spelling mistake (and to fix the missing
> > > word "Windows 2016" as
> > > "Windows Server 2016").
> >
> > +1
> >
> > I fully support removing that config option :)
> >
> > Cheers,
> > --
> > Aurélien Aptel / SUSE Labs Samba Team
> > GPG: 1839 CB5F 9F5B FB9B AA97  8C99 03C8 A49B 521B D5D3
> > SUSE Linux GmbH, Maxfeldstraße 5, 90409 Nürnberg, Germany
> > GF: Felix Imendörffer, Jane Smithard, Graham Norton, HRB 21284 (AG Nürnberg)
> >
>
> I like the idea to remove SMB311 config option too but I think we
> should put all the things related to POSIX ext under
> CONFIG_CIFS_POSIX, e.g.:
>
> -#ifdef CONFIG_CIFS_SMB311
> +#ifdef CONFIG_CIFS_POSIX
>          if (server->posix_ext_supported)
>              seq_printf(m, " posix");
> -#endif /* 3.1.1 */
> +#ifdef /* POSIX */
>
> and the same for other places mentioning POSIX ext. Thoughts?

My gut reaction on that is that since the 3.1.1 POSIX changes are
pretty small except a couple functions which are very distinct, and
don't intersect much with complex code (as the CIFS POSIX extensions
did), simply marking the mount option experimental is less likely to
lead to weird build breaks, and problems testing.  We need to continue
testing and finishup of the SMB3.1.1 POSIX Extensions as soon as
possible due to real problems with CIFS (and thus CIFS/POSIX)
deprecation.

Don't have a strong feeling on this though one way or the other (about
whether adding and additional #ifdef would help - I am a little
worried that nesting a 3.1.1 feature ifdef in an ifdef for an SMB1
(CIFS POSIX) feature would get confusing.




-- 
Thanks,

Steve
--
To unsubscribe from this list: send the line "unsubscribe linux-cifs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux