[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240222141559-mutt-send-email-mst@kernel.org>
Date: Thu, 22 Feb 2024 14:18:07 -0500
From: "Michael S. Tsirkin" <mst@...hat.com>
To: Cindy Lu <lulu@...hat.com>
Cc: jasowang@...hat.com, xieyongji@...edance.com,
linux-kernel@...r.kernel.org, maxime.coquelin@...hat.com
Subject: Re: [PATCH v4 0/5] vduse: Add support for reconnection
On Wed, Feb 07, 2024 at 01:43:27PM +0800, Cindy Lu wrote:
> Here is the reconnect support in vduse,
>
> Kernel will allocate pages for reconnection.
> Userspace needs to use mmap to map the memory to userspace and use these pages to
> save the reconnect information.
What is "reconnect"? Not really clear from documentation - it seems to
be assumed that reader has an idea but most don't.
Also what's with all the typos? reconnect with 3 nnn s, sutiable and so
on. Can you pls run a speller?
> test passd in vduse+dpdk-testpmd
>
> change in V2
> 1. Address the comments from v1
> 2. Add the document for reconnect process
>
> change in V3
> 1. Move the vdpa_vq_state to the uAPI. vduse will use this to synchronize the vq info between the kernel and userspace app.
> 2. Add a new ioctl VDUSE_DEV_GET_CONFIG. userspace app use this to get config space
> 3. Rewrite the commit message.
> 4. Only save the address for the page address and remove the index.
> 5. remove the ioctl VDUSE_GET_RECONNECT_INFO, userspace app will use uAPI VDUSE_RECONNCT_MMAP_SIZE to mmap
> 6. Rewrite the document for the reconnect process to make it clearer.
>
> change in v4
> 1. Change the number of map pages to VQ numbers. UserSpace APP can define and maintain the structure for saving reconnection information in userspace. The kernel will not maintain this information.
> 2. Rewrite the document for the reconnect process to make it clearer.
> 3. add the new ioctl for VDUSE_DEV_GET_CONFIG/VDUSE_DEV_GET_STATUS
>
> Cindy Lu (5):
> vduse: Add new ioctl VDUSE_DEV_GET_CONFIG
> vduse: Add new ioctl VDUSE_DEV_GET_STATUS
> vduse: Add function to get/free the pages for reconnection
> vduse: Add file operation for mmap
> Documentation: Add reconnect process for VDUSE
>
> Documentation/userspace-api/vduse.rst | 32 +++++++
> drivers/vdpa/vdpa_user/vduse_dev.c | 125 ++++++++++++++++++++++++++
> include/uapi/linux/vduse.h | 5 ++
> 3 files changed, 162 insertions(+)
>
> --
> 2.43.0
Powered by blists - more mailing lists