| 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
| ||
|
Message-ID: <CACGkMEvfFV8w34=SiS4XFyEH5+EEkb9JqYg0t_rKpU1rRBMLiQ@mail.gmail.com>
Date: Thu, 30 Jun 2022 14:40:02 +0800
From: Jason Wang <jasowang@...hat.com>
To: Xuan Zhuo <xuanzhuo@...ux.alibaba.com>
Cc: virtualization <virtualization@...ts.linux-foundation.org>,
Richard Weinberger <richard@....at>,
Anton Ivanov <anton.ivanov@...bridgegreys.com>,
Johannes Berg <johannes@...solutions.net>,
"Michael S. Tsirkin" <mst@...hat.com>,
"David S. Miller" <davem@...emloft.net>,
Eric Dumazet <edumazet@...gle.com>,
Jakub Kicinski <kuba@...nel.org>,
Paolo Abeni <pabeni@...hat.com>,
Hans de Goede <hdegoede@...hat.com>,
Mark Gross <markgross@...nel.org>,
Vadim Pasternak <vadimp@...dia.com>,
Bjorn Andersson <bjorn.andersson@...aro.org>,
Mathieu Poirier <mathieu.poirier@...aro.org>,
Cornelia Huck <cohuck@...hat.com>,
Halil Pasic <pasic@...ux.ibm.com>,
Eric Farman <farman@...ux.ibm.com>,
Heiko Carstens <hca@...ux.ibm.com>,
Vasily Gorbik <gor@...ux.ibm.com>,
Alexander Gordeev <agordeev@...ux.ibm.com>,
Christian Borntraeger <borntraeger@...ux.ibm.com>,
Sven Schnelle <svens@...ux.ibm.com>,
Alexei Starovoitov <ast@...nel.org>,
Daniel Borkmann <daniel@...earbox.net>,
Jesper Dangaard Brouer <hawk@...nel.org>,
John Fastabend <john.fastabend@...il.com>,
Vincent Whitchurch <vincent.whitchurch@...s.com>,
linux-um@...ts.infradead.org, netdev <netdev@...r.kernel.org>,
platform-driver-x86@...r.kernel.org,
linux-remoteproc@...r.kernel.org, linux-s390@...r.kernel.org,
kvm <kvm@...r.kernel.org>,
"open list:XDP (eXpress Data Path)" <bpf@...r.kernel.org>,
kangjie.xu@...ux.alibaba.com
Subject: Re: [PATCH v11 05/40] virtio_ring: split vring_virtqueue
On Wed, Jun 29, 2022 at 2:57 PM Xuan Zhuo <xuanzhuo@...ux.alibaba.com> wrote:
>
> Separate the two inline structures(split and packed) from the structure
> vring_virtqueue.
>
> In this way, we can use these two structures later to pass parameters
> and retain temporary variables.
>
> Signed-off-by: Xuan Zhuo <xuanzhuo@...ux.alibaba.com>
Acked-by: Jason Wang <jasowang@...hat.com>
> ---
> drivers/virtio/virtio_ring.c | 116 ++++++++++++++++++-----------------
> 1 file changed, 60 insertions(+), 56 deletions(-)
>
> diff --git a/drivers/virtio/virtio_ring.c b/drivers/virtio/virtio_ring.c
> index bb4e8ae09c9b..2806e033a651 100644
> --- a/drivers/virtio/virtio_ring.c
> +++ b/drivers/virtio/virtio_ring.c
> @@ -85,6 +85,64 @@ struct vring_desc_extra {
> u16 next; /* The next desc state in a list. */
> };
>
> +struct vring_virtqueue_split {
> + /* Actual memory layout for this queue. */
> + struct vring vring;
> +
> + /* Last written value to avail->flags */
> + u16 avail_flags_shadow;
> +
> + /*
> + * Last written value to avail->idx in
> + * guest byte order.
> + */
> + u16 avail_idx_shadow;
> +
> + /* Per-descriptor state. */
> + struct vring_desc_state_split *desc_state;
> + struct vring_desc_extra *desc_extra;
> +
> + /* DMA address and size information */
> + dma_addr_t queue_dma_addr;
> + size_t queue_size_in_bytes;
> +};
> +
> +struct vring_virtqueue_packed {
> + /* Actual memory layout for this queue. */
> + struct {
> + unsigned int num;
> + struct vring_packed_desc *desc;
> + struct vring_packed_desc_event *driver;
> + struct vring_packed_desc_event *device;
> + } vring;
> +
> + /* Driver ring wrap counter. */
> + bool avail_wrap_counter;
> +
> + /* Avail used flags. */
> + u16 avail_used_flags;
> +
> + /* Index of the next avail descriptor. */
> + u16 next_avail_idx;
> +
> + /*
> + * Last written value to driver->flags in
> + * guest byte order.
> + */
> + u16 event_flags_shadow;
> +
> + /* Per-descriptor state. */
> + struct vring_desc_state_packed *desc_state;
> + struct vring_desc_extra *desc_extra;
> +
> + /* DMA address and size information */
> + dma_addr_t ring_dma_addr;
> + dma_addr_t driver_event_dma_addr;
> + dma_addr_t device_event_dma_addr;
> + size_t ring_size_in_bytes;
> + size_t event_size_in_bytes;
> +};
> +
> struct vring_virtqueue {
> struct virtqueue vq;
>
> @@ -124,64 +182,10 @@ struct vring_virtqueue {
>
> union {
> /* Available for split ring */
> - struct {
> - /* Actual memory layout for this queue. */
> - struct vring vring;
> -
> - /* Last written value to avail->flags */
> - u16 avail_flags_shadow;
> -
> - /*
> - * Last written value to avail->idx in
> - * guest byte order.
> - */
> - u16 avail_idx_shadow;
> -
> - /* Per-descriptor state. */
> - struct vring_desc_state_split *desc_state;
> - struct vring_desc_extra *desc_extra;
> -
> - /* DMA address and size information */
> - dma_addr_t queue_dma_addr;
> - size_t queue_size_in_bytes;
> - } split;
> + struct vring_virtqueue_split split;
>
> /* Available for packed ring */
> - struct {
> - /* Actual memory layout for this queue. */
> - struct {
> - unsigned int num;
> - struct vring_packed_desc *desc;
> - struct vring_packed_desc_event *driver;
> - struct vring_packed_desc_event *device;
> - } vring;
> -
> - /* Driver ring wrap counter. */
> - bool avail_wrap_counter;
> -
> - /* Avail used flags. */
> - u16 avail_used_flags;
> -
> - /* Index of the next avail descriptor. */
> - u16 next_avail_idx;
> -
> - /*
> - * Last written value to driver->flags in
> - * guest byte order.
> - */
> - u16 event_flags_shadow;
> -
> - /* Per-descriptor state. */
> - struct vring_desc_state_packed *desc_state;
> - struct vring_desc_extra *desc_extra;
> -
> - /* DMA address and size information */
> - dma_addr_t ring_dma_addr;
> - dma_addr_t driver_event_dma_addr;
> - dma_addr_t device_event_dma_addr;
> - size_t ring_size_in_bytes;
> - size_t event_size_in_bytes;
> - } packed;
> + struct vring_virtqueue_packed packed;
> };
>
> /* How to notify other side. FIXME: commonalize hcalls! */
> --
> 2.31.0
>
Powered by blists - more mailing lists