2023-08-26 09:50:02

by Jiang Dongxu

[permalink] [raw]
Subject: [PATCH 0/2] vdpa: Add logging operatins

From: jiangdongxu <[email protected]>

Currently, the vdpa device supports suspend and resume operations.
To support vdpa device live migration, we need to support logging
operations and device state save/load opertions.

These series introduces some new operations for vdpa devices.
They allow vdpa to enable logging while vm start live migration.

And I will submit another patches about saving and loading
vdpa device state later.

jiangdongxu (2):
vdpa: add log operations
vhost-vdpa: add uAPI for logging

drivers/vhost/vdpa.c | 49 ++++++++++++++++++++++++++++++++++++++
include/linux/vdpa.h | 14 +++++++++++
include/uapi/linux/vhost.h | 4 ++++
3 files changed, 67 insertions(+)

--
2.27.0



2023-10-18 15:09:14

by Michael S. Tsirkin

[permalink] [raw]
Subject: Re: [PATCH 0/2] vdpa: Add logging operatins

On Mon, Sep 11, 2023 at 02:56:58PM +0800, Jason Wang wrote:
> Adding Eugenio and Si Wei.
>
> On Sat, Aug 26, 2023 at 9:24 AM Jiang Dongxu <[email protected]> wrote:
> >
> > From: jiangdongxu <[email protected]>
> >
> > Currently, the vdpa device supports suspend and resume operations.
> > To support vdpa device live migration, we need to support logging
> > operations and device state save/load opertions.
> >
> > These series introduces some new operations for vdpa devices.
> > They allow vdpa to enable logging while vm start live migration.
> >
> > And I will submit another patches about saving and loading
> > vdpa device state later.
>
> Thanks for working on this, I have several questions:
>
> 1) Is there an example implementation of the logging in the drivers?
> We need a real user in order to merge this.
> 2) Is the logging based on IOVA or VA? How the DMA isolation is being
> done with this? Do we need a SET_LOGGING_ASID uAPI for this? (We had
> some discussion on this in the past).
>
> Thanks


Seems to be stuck. Dropped this for now.

> >
> > jiangdongxu (2):
> > vdpa: add log operations
> > vhost-vdpa: add uAPI for logging
> >
> > drivers/vhost/vdpa.c | 49 ++++++++++++++++++++++++++++++++++++++
> > include/linux/vdpa.h | 14 +++++++++++
> > include/uapi/linux/vhost.h | 4 ++++
> > 3 files changed, 67 insertions(+)
> >
> > --
> > 2.27.0
> >

2023-11-01 13:03:26

by Michael S. Tsirkin

[permalink] [raw]
Subject: Re: [PATCH 0/2] vdpa: Add logging operatins

On Mon, Sep 11, 2023 at 02:56:58PM +0800, Jason Wang wrote:
> Adding Eugenio and Si Wei.
>
> On Sat, Aug 26, 2023 at 9:24 AM Jiang Dongxu <[email protected]> wrote:
> >
> > From: jiangdongxu <[email protected]>
> >
> > Currently, the vdpa device supports suspend and resume operations.
> > To support vdpa device live migration, we need to support logging
> > operations and device state save/load opertions.
> >
> > These series introduces some new operations for vdpa devices.
> > They allow vdpa to enable logging while vm start live migration.
> >
> > And I will submit another patches about saving and loading
> > vdpa device state later.
>
> Thanks for working on this, I have several questions:
>
> 1) Is there an example implementation of the logging in the drivers?
> We need a real user in order to merge this.
> 2) Is the logging based on IOVA or VA? How the DMA isolation is being
> done with this? Do we need a SET_LOGGING_ASID uAPI for this? (We had
> some discussion on this in the past).
>
> Thanks


No answer so far so I'll drop this for now.

> >
> > jiangdongxu (2):
> > vdpa: add log operations
> > vhost-vdpa: add uAPI for logging
> >
> > drivers/vhost/vdpa.c | 49 ++++++++++++++++++++++++++++++++++++++
> > include/linux/vdpa.h | 14 +++++++++++
> > include/uapi/linux/vhost.h | 4 ++++
> > 3 files changed, 67 insertions(+)
> >
> > --
> > 2.27.0
> >