[PATCH 12/14] nvmet-tcp: allocate socket file

[Hannes Reinecke]

On 8/7/23 10:53, Sagi Grimberg wrote:
> 
>>>> -static int nvmet_tcp_alloc_queue(struct nvmet_tcp_port *port,
>>>> +static void nvmet_tcp_alloc_queue(struct nvmet_tcp_port *port,
>>>>           struct socket *newsock)
>>>
>>> Why does the function change from retcode to void in this patch?
>>
>> Because the return code was never evaluated.
> 
> ??
> 
> I see it is evaluated.

Yes, but this patch moved the 'sock_release()' call into 
nvmet_tcp_alloc_queue(), making the return code obsolete.

But I can make it a separate patch if required.

Cheers,

Hannes
-- 
Dr. Hannes Reinecke                Kernel Storage Architect
hare at suse.de                              +49 911 74053 688
SUSE Software Solutions GmbH, Maxfeldstr. 5, 90409 Nürnberg
HRB 36809 (AG Nürnberg), Geschäftsführer: Ivo Totev, Andrew
Myers, Andrew McDonald, Martje Boudien Moerman