linux-kernel - Re: [PATCH v2] virtio: Describe @map and @vmap members in virtio

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] [day] [month] [year] [list]

Message-ID: <aVBkdccdls3fbY6s@archie.me>
Date: Sun, 28 Dec 2025 05:57:57 +0700
From: Bagas Sanjaya <bagasdotme@...il.com>
To: "Michael S. Tsirkin" <mst@...hat.com>
Cc: Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
	Linux Virtualization <virtualization@...ts.linux.dev>,
	Jason Wang <jasowang@...hat.com>,
	Xuan Zhuo <xuanzhuo@...ux.alibaba.com>,
	Eugenio Pérez <eperezma@...hat.com>
Subject: Re: [PATCH v2] virtio: Describe @map and @vmap members in
 virtio_device struct

On Fri, Dec 26, 2025 at 02:13:00PM -0500, Michael S. Tsirkin wrote:
> > diff --git a/include/linux/virtio.h b/include/linux/virtio.h
> > index 132a474e59140a..68ead8fda9c921 100644
> > --- a/include/linux/virtio.h
> > +++ b/include/linux/virtio.h
> > @@ -150,11 +150,13 @@ struct virtio_admin_cmd {
> >   * @id: the device type identification (used to match it with a driver).
> >   * @config: the configuration ops for this device.
> >   * @vringh_config: configuration ops for host vrings.
> > + * @map: configuration ops for device's mapping buffer
> 
> there's no "mapping buffer" that needs configuration.
> the type tells it all:
>  * struct virtio_map_ops - operations for mapping buffer for a virtio device
> 
> 
> >   * @vqs: the list of virtqueues for this device.
> >   * @features: the 64 lower features supported by both driver and device.
> >   * @features_array: the full features space supported by both driver and
> >   *		    device.
> >   * @priv: private pointer for the driver's use.
> > + * @vmap: device virtual map
> 
> this one is not a "virtual map" either.
> more a "device or domain used for mapping memory"

Thanks for the suggestions! I'll apply them in v3 (should be out in Monday).

-- 
An old man doll... just what I always wanted! - Clara

Download attachment "signature.asc" of type "application/pgp-signature" (229 bytes)