[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <87a2822c-c5ac-13d1-336a-b96045c3de5b@suse.de>
Date: Mon, 7 Jun 2021 17:46:09 +0200
From: Hannes Reinecke <hare@...e.de>
To: Shai Malin <smalin@...vell.com>, linux-nvme@...ts.infradead.org,
sagi@...mberg.me, hch@....de, axboe@...com, kbusch@...nel.org
Cc: netdev@...r.kernel.org, davem@...emloft.net, kuba@...nel.org,
aelior@...vell.com, mkalderon@...vell.com, okulkarni@...vell.com,
pkushwaha@...vell.com, prabhakar.pkin@...il.com,
malin1024@...il.com, Arie Gershberg <agershberg@...vell.com>
Subject: Re: [PATCH 5/8] nvme-tcp-offload: Add controller level implementation
On 6/2/21 8:42 PM, Shai Malin wrote:
> From: Arie Gershberg <agershberg@...vell.com>
>
> In this patch we implement controller level functionality including:
> - create_ctrl.
> - delete_ctrl.
> - free_ctrl.
>
> The implementation is similar to other nvme fabrics modules, the main
> difference being that the nvme-tcp-offload ULP calls the vendor specific
> claim_dev() op with the given TCP/IP parameters to determine which device
> will be used for this controller.
> Once found, the vendor specific device and controller will be paired and
> kept in a controller list managed by the ULP.
>
> Acked-by: Igor Russkikh <irusskikh@...vell.com>
> Signed-off-by: Arie Gershberg <agershberg@...vell.com>
> Signed-off-by: Prabhakar Kushwaha <pkushwaha@...vell.com>
> Signed-off-by: Omkar Kulkarni <okulkarni@...vell.com>
> Signed-off-by: Michal Kalderon <mkalderon@...vell.com>
> Signed-off-by: Ariel Elior <aelior@...vell.com>
> Signed-off-by: Shai Malin <smalin@...vell.com>
> Reviewed-by: Himanshu Madhani <himanshu.madhani@...cle.com>
> ---
> drivers/nvme/host/tcp-offload.c | 481 +++++++++++++++++++++++++++++++-
> 1 file changed, 476 insertions(+), 5 deletions(-)
>
Reviewed-by: Hannes Reinecke <hare@...e.de>
Cheers,
Hannes
--
Dr. Hannes Reinecke Kernel Storage Architect
hare@...e.de +49 911 74053 688
SUSE Software Solutions Germany GmbH, 90409 Nürnberg
GF: F. Imendörffer, HRB 36809 (AG Nürnberg)
Powered by blists - more mailing lists