[PATCH v2 net-next 3/7] socket: Restore sock_create_kern().
David Laight
david.laight.linux at gmail.com
Tue Jun 3 14:30:20 PDT 2025
On Mon, 2 Jun 2025 07:08:17 +0200
Christoph Hellwig <hch at lst.de> wrote:
> On Thu, May 29, 2025 at 07:53:41PM -0700, Kuniyuki Iwashima wrote:
> > In the old days, sock_create_kern() did take a ref to netns,
> > but an implicit change that avoids taking the ref has caused
> > a lot of problems for people who used to the old semantics.
That must have been a long time ago.
Was it even long after the namespace code was added?
(I don't have a system with the git tree up at the moment)
> >
> > This series rather rolls back the change, so I think using
> > the same name here is better than leaving the catchy
> > sock_create_kern() error-prone.
>
> Ok.
Except that you are changing the semantics again.
So you end up with the same problem the other way around.
I can imagine code ending up with an extra reference to the ns.
The obvious name a a function for general driver use would be
kernel_socket() - matching the other functions that were added
when set_fs(KERNEL_DS) was removed.
I definitely aim to end up where the existing code fails to
compile - just to ensure all the code is found.
David
More information about the Linux-nvme
mailing list