All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] vdpa: Mark vdpa_config_ops.get_vq_notification as optional
@ 2021-11-04 19:52 Eugenio Pérez
  2021-11-05  2:35   ` Jason Wang
  2021-11-05  8:32   ` Stefano Garzarella
  0 siblings, 2 replies; 5+ messages in thread
From: Eugenio Pérez @ 2021-11-04 19:52 UTC (permalink / raw)
  To: linux-kernel; +Cc: Michael S. Tsirkin, Jason Wang, kvm, virtualization, netdev

Since vhost_vdpa_mmap checks for its existence before calling it.

Signed-off-by: Eugenio Pérez <eperezma@redhat.com>
---
 include/linux/vdpa.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/linux/vdpa.h b/include/linux/vdpa.h
index c3011ccda430..0bdc7f785394 100644
--- a/include/linux/vdpa.h
+++ b/include/linux/vdpa.h
@@ -155,7 +155,7 @@ struct vdpa_map_file {
  *				@vdev: vdpa device
  *				@idx: virtqueue index
  *				@state: pointer to returned state (last_avail_idx)
- * @get_vq_notification:	Get the notification area for a virtqueue
+ * @get_vq_notification:	Get the notification area for a virtqueue (optional)
  *				@vdev: vdpa device
  *				@idx: virtqueue index
  *				Returns the notifcation area
-- 
2.27.0


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH] vdpa: Mark vdpa_config_ops.get_vq_notification as optional
  2021-11-04 19:52 [PATCH] vdpa: Mark vdpa_config_ops.get_vq_notification as optional Eugenio Pérez
@ 2021-11-05  2:35   ` Jason Wang
  2021-11-05  8:32   ` Stefano Garzarella
  1 sibling, 0 replies; 5+ messages in thread
From: Jason Wang @ 2021-11-05  2:35 UTC (permalink / raw)
  To: Eugenio Pérez
  Cc: linux-kernel, Michael S. Tsirkin, kvm, virtualization, netdev

On Fri, Nov 5, 2021 at 3:53 AM Eugenio Pérez <eperezma@redhat.com> wrote:
>
> Since vhost_vdpa_mmap checks for its existence before calling it.
>
> Signed-off-by: Eugenio Pérez <eperezma@redhat.com>

Acked-by: Jason Wang <jasowang@redhat.com>

> ---
>  include/linux/vdpa.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/linux/vdpa.h b/include/linux/vdpa.h
> index c3011ccda430..0bdc7f785394 100644
> --- a/include/linux/vdpa.h
> +++ b/include/linux/vdpa.h
> @@ -155,7 +155,7 @@ struct vdpa_map_file {
>   *                             @vdev: vdpa device
>   *                             @idx: virtqueue index
>   *                             @state: pointer to returned state (last_avail_idx)
> - * @get_vq_notification:       Get the notification area for a virtqueue
> + * @get_vq_notification:       Get the notification area for a virtqueue (optional)
>   *                             @vdev: vdpa device
>   *                             @idx: virtqueue index
>   *                             Returns the notifcation area
> --
> 2.27.0
>


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] vdpa: Mark vdpa_config_ops.get_vq_notification as optional
@ 2021-11-05  2:35   ` Jason Wang
  0 siblings, 0 replies; 5+ messages in thread
From: Jason Wang @ 2021-11-05  2:35 UTC (permalink / raw)
  To: Eugenio Pérez
  Cc: netdev, virtualization, linux-kernel, kvm, Michael S. Tsirkin

On Fri, Nov 5, 2021 at 3:53 AM Eugenio Pérez <eperezma@redhat.com> wrote:
>
> Since vhost_vdpa_mmap checks for its existence before calling it.
>
> Signed-off-by: Eugenio Pérez <eperezma@redhat.com>

Acked-by: Jason Wang <jasowang@redhat.com>

> ---
>  include/linux/vdpa.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/linux/vdpa.h b/include/linux/vdpa.h
> index c3011ccda430..0bdc7f785394 100644
> --- a/include/linux/vdpa.h
> +++ b/include/linux/vdpa.h
> @@ -155,7 +155,7 @@ struct vdpa_map_file {
>   *                             @vdev: vdpa device
>   *                             @idx: virtqueue index
>   *                             @state: pointer to returned state (last_avail_idx)
> - * @get_vq_notification:       Get the notification area for a virtqueue
> + * @get_vq_notification:       Get the notification area for a virtqueue (optional)
>   *                             @vdev: vdpa device
>   *                             @idx: virtqueue index
>   *                             Returns the notifcation area
> --
> 2.27.0
>

_______________________________________________
Virtualization mailing list
Virtualization@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/virtualization

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] vdpa: Mark vdpa_config_ops.get_vq_notification as optional
  2021-11-04 19:52 [PATCH] vdpa: Mark vdpa_config_ops.get_vq_notification as optional Eugenio Pérez
@ 2021-11-05  8:32   ` Stefano Garzarella
  2021-11-05  8:32   ` Stefano Garzarella
  1 sibling, 0 replies; 5+ messages in thread
From: Stefano Garzarella @ 2021-11-05  8:32 UTC (permalink / raw)
  To: Eugenio Pérez
  Cc: linux-kernel, Michael S. Tsirkin, Jason Wang, kvm,
	virtualization, netdev

On Thu, Nov 04, 2021 at 08:52:48PM +0100, Eugenio Pérez wrote:
>Since vhost_vdpa_mmap checks for its existence before calling it.
>
>Signed-off-by: Eugenio Pérez <eperezma@redhat.com>
>---
> include/linux/vdpa.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/include/linux/vdpa.h b/include/linux/vdpa.h
>index c3011ccda430..0bdc7f785394 100644
>--- a/include/linux/vdpa.h
>+++ b/include/linux/vdpa.h
>@@ -155,7 +155,7 @@ struct vdpa_map_file {
>  *				@vdev: vdpa device
>  *				@idx: virtqueue index
>  *				@state: pointer to returned state (last_avail_idx)
>- * @get_vq_notification:	Get the notification area for a virtqueue
>+ * @get_vq_notification:	Get the notification area for a virtqueue (optional)
>  *				@vdev: vdpa device
>  *				@idx: virtqueue index
>  *				Returns the notifcation area
>-- 
>2.27.0
>

Reviewed-by: Stefano Garzarella <sgarzare@redhat.com>


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] vdpa: Mark vdpa_config_ops.get_vq_notification as optional
@ 2021-11-05  8:32   ` Stefano Garzarella
  0 siblings, 0 replies; 5+ messages in thread
From: Stefano Garzarella @ 2021-11-05  8:32 UTC (permalink / raw)
  To: Eugenio Pérez
  Cc: kvm, Michael S. Tsirkin, netdev, linux-kernel, virtualization

On Thu, Nov 04, 2021 at 08:52:48PM +0100, Eugenio Pérez wrote:
>Since vhost_vdpa_mmap checks for its existence before calling it.
>
>Signed-off-by: Eugenio Pérez <eperezma@redhat.com>
>---
> include/linux/vdpa.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/include/linux/vdpa.h b/include/linux/vdpa.h
>index c3011ccda430..0bdc7f785394 100644
>--- a/include/linux/vdpa.h
>+++ b/include/linux/vdpa.h
>@@ -155,7 +155,7 @@ struct vdpa_map_file {
>  *				@vdev: vdpa device
>  *				@idx: virtqueue index
>  *				@state: pointer to returned state (last_avail_idx)
>- * @get_vq_notification:	Get the notification area for a virtqueue
>+ * @get_vq_notification:	Get the notification area for a virtqueue (optional)
>  *				@vdev: vdpa device
>  *				@idx: virtqueue index
>  *				Returns the notifcation area
>-- 
>2.27.0
>

Reviewed-by: Stefano Garzarella <sgarzare@redhat.com>

_______________________________________________
Virtualization mailing list
Virtualization@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/virtualization

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2021-11-05  8:32 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-04 19:52 [PATCH] vdpa: Mark vdpa_config_ops.get_vq_notification as optional Eugenio Pérez
2021-11-05  2:35 ` Jason Wang
2021-11-05  2:35   ` Jason Wang
2021-11-05  8:32 ` Stefano Garzarella
2021-11-05  8:32   ` Stefano Garzarella

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.