Messages in this thread | | | From | Linus Torvalds <> | Date | Mon, 22 May 2023 10:33:30 -0700 | Subject | Re: patches to move ksmbd and cifs under new subdirectory |
| |
On Mon, May 22, 2023 at 9:33 AM Steve French <smfrench@gmail.com> wrote: > > Following up on the email thread suggestion to move fs/ksmbd and > fs/cifs and fs/smbfs_common all under a common directory fs/smb, here > is an updated patchset for that (added one small patch).
Looks fine to me.
I wouldn't have noticed the typo that Tom Talpey mentioned (misspelled "common" in the commit message of the first patch), and that SMB_CLIENT config variable is odd.
I'm actually surprised that Kconfig didn't complain about the
select SMB_CLIENT
when there is no actual declaration of that Kconfig variable, just a random use.
That thing seems confusing and confused, and isn't related to the renaming, so please drop the new random SMB_CLIENT config variable. If you want to introduce a new Kconfig variable later for some reason, that's fine, but please don't mix those kinds of changes up with pure renames..
Linus
| |