Re: [PATCH] smb: smbdirect: move fs/smb/common/smbdirect/ to fs/smb/smbdirect/
Paulo Alcantara <[email protected]> Wed, 22 Apr 2026 12:16:25 -0300
| Newsgroups | gmane.linux.drivers.rdma,gmane.linux.kernel.cifs,gmane.linux.network,gmane.network.samba.internals |
|---|---|
| Message-ID | <[email protected]> |
Steve French <[email protected]> writes: > On Wed, Apr 22, 2026 at 3:16 AM Stefan Metzmacher <[email protected]> wrote: >> >> Hi Christoph, >> >> >> diff --git a/fs/smb/Makefile b/fs/smb/Makefile >> >> index 9a1bf59a1a65..353b1c2eefc4 100644 >> >> --- a/fs/smb/Makefile >> >> +++ b/fs/smb/Makefile >> >> @@ -1,5 +1,6 @@ >> >> # SPDX-License-Identifier: GPL-2.0 >> >> >> >> obj-$(CONFIG_SMBFS) += common/ >> >> +obj-$(CONFIG_SMBDIRECT) += smbdirect/ >> > >> > Why is this not in net/smbdirect/ or driver/infiniband/ulp/smdirect? >> >> Yes, I also thought about net/smbdirect. > > I would prefer to leave it in fs/smb for the time being, since it makes it > easier to track since fs/smb/server and fs/smb/client have dependencies > on it. In the long run, I don't mind moving it, if it starts being > used outside > of smb client and server. Please let's not break backporting any further. Decide where it will end up at once. We don't want the "fs/cifs -> fs/smb/client" history all over again. Won't samba be using it? If so, you could consider an user outside fs/smb/{client,server} and then leave it in net/ instead, as hch suggested.