linux-kernel - Re: [PATCH 0/2] vdpa: Add logging operatins

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20231018110719-mutt-send-email-mst@kernel.org>
Date:   Wed, 18 Oct 2023 11:07:31 -0400
From:   "Michael S. Tsirkin" <mst@...hat.com>
To:     Jason Wang <jasowang@...hat.com>
Cc:     Jiang Dongxu <jiangdongxu1@...wei.com>,
        virtualization@...ts.linux-foundation.org,
        linux-kernel@...r.kernel.org, longpeng2@...wei.com,
        eric.fangyi@...wei.com, eperezma <eperezma@...hat.com>,
        Siwei Liu <loseweigh@...il.com>
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 <jiangdongxu1@...wei.com> wrote:
> >
> > From: jiangdongxu <jiangdongxu1@...wei.com>
> >
> > 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
> >