Thread (4 messages) flat view 4 messages, 4 authors, 2020-09-01

Re: [PATCH net-next] vhost: fix typo in error message

From: Jason Wang <hidden>
Date: 2020-09-01 03:07:10
Also in: kvm, lkml, virtualization

On 2020/9/1 上午10:39, Yunsheng Lin wrote:
quoted hunk ↗ jump to hunk
"enable" should be "disable" when the function name is
vhost_disable_notify(), which does the disabling work.

Signed-off-by: Yunsheng Lin <redacted>
---
  drivers/vhost/vhost.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/vhost/vhost.c b/drivers/vhost/vhost.c
index 5857d4e..b45519c 100644
--- a/drivers/vhost/vhost.c
+++ b/drivers/vhost/vhost.c
@@ -2537,7 +2537,7 @@ void vhost_disable_notify(struct vhost_dev *dev, struct vhost_virtqueue *vq)
  	if (!vhost_has_feature(vq, VIRTIO_RING_F_EVENT_IDX)) {
  		r = vhost_update_used_flags(vq);
  		if (r)
-			vq_err(vq, "Failed to enable notification at %p: %d\n",
+			vq_err(vq, "Failed to disable notification at %p: %d\n",
  			       &vq->used->flags, r);
  	}
  }

Acked-by: Jason Wang <redacted>


Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help