[PATCH 0/2] virtio_blk header fixes

[PATCH 0/2] virtio_blk header fixes

[Michael S. Tsirkin]

Now that QEmu reuses linux virtio headers, we noticed
a typo in the exported virtio block header. Fix it up.

I'd like these merged for 4.0 so that Qemu 2.3 can
alredy get it right.

Michael S. Tsirkin (2):
  virtio_blk: typo fix
  virtio_blk: fix comment for virtio 1.0

 include/uapi/linux/virtio_blk.h | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

-- 
MST

[Qemu-devel] [PATCH 0/2] virtio_blk header fixes

[Michael S. Tsirkin]

Now that QEmu reuses linux virtio headers, we noticed
a typo in the exported virtio block header. Fix it up.

I'd like these merged for 4.0 so that Qemu 2.3 can
alredy get it right.

Michael S. Tsirkin (2):
  virtio_blk: typo fix
  virtio_blk: fix comment for virtio 1.0

 include/uapi/linux/virtio_blk.h | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

-- 
MST

[PATCH 1/2] virtio_blk: typo fix

[Michael S. Tsirkin]

Now that QEmu reuses linux virtio headers, we noticed
a typo in the exported virtio block header. Fix it up.

Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
---
 include/uapi/linux/virtio_blk.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/uapi/linux/virtio_blk.h b/include/uapi/linux/virtio_blk.h
index 3c53eec..b695ba9 100644
--- a/include/uapi/linux/virtio_blk.h
+++ b/include/uapi/linux/virtio_blk.h
@@ -60,7 +60,7 @@ struct virtio_blk_config {
 	__u32 size_max;
 	/* The maximum number of segments (if VIRTIO_BLK_F_SEG_MAX) */
 	__u32 seg_max;
-	/* geometry the device (if VIRTIO_BLK_F_GEOMETRY) */
+	/* geometry of the device (if VIRTIO_BLK_F_GEOMETRY) */
 	struct virtio_blk_geometry {
 		__u16 cylinders;
 		__u8 heads;
-- 
MST

[Qemu-devel] [PATCH 1/2] virtio_blk: typo fix

[Michael S. Tsirkin]

Now that QEmu reuses linux virtio headers, we noticed
a typo in the exported virtio block header. Fix it up.

Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
---
 include/uapi/linux/virtio_blk.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/uapi/linux/virtio_blk.h b/include/uapi/linux/virtio_blk.h
index 3c53eec..b695ba9 100644
--- a/include/uapi/linux/virtio_blk.h
+++ b/include/uapi/linux/virtio_blk.h
@@ -60,7 +60,7 @@ struct virtio_blk_config {
 	__u32 size_max;
 	/* The maximum number of segments (if VIRTIO_BLK_F_SEG_MAX) */
 	__u32 seg_max;
-	/* geometry the device (if VIRTIO_BLK_F_GEOMETRY) */
+	/* geometry of the device (if VIRTIO_BLK_F_GEOMETRY) */
 	struct virtio_blk_geometry {
 		__u16 cylinders;
 		__u8 heads;
-- 
MST

[PATCH 2/2] virtio_blk: fix comment for virtio 1.0

[Michael S. Tsirkin]

Fix up comment to match virtio 1.0 logic:
virtio_blk_outhdr isn't the first elements anymore,
the only requirement is that it comes first in
the s/g list.

Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
---
 include/uapi/linux/virtio_blk.h | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/include/uapi/linux/virtio_blk.h b/include/uapi/linux/virtio_blk.h
index b695ba9..19c66fc 100644
--- a/include/uapi/linux/virtio_blk.h
+++ b/include/uapi/linux/virtio_blk.h
@@ -119,7 +119,11 @@ struct virtio_blk_config {
 #define VIRTIO_BLK_T_BARRIER	0x80000000
 #endif /* !VIRTIO_BLK_NO_LEGACY */
 
-/* This is the first element of the read scatter-gather list. */
+/*
+ * This comes first in the read scatter-gather list.
+ * For legacy virtio, if VIRTIO_F_ANY_LAYOUT is not negotiated,
+ * this is the first element of the read scatter-gather list.
+ */
 struct virtio_blk_outhdr {
 	/* VIRTIO_BLK_T* */
 	__virtio32 type;
-- 
MST

[Qemu-devel] [PATCH 2/2] virtio_blk: fix comment for virtio 1.0

[Michael S. Tsirkin]

Fix up comment to match virtio 1.0 logic:
virtio_blk_outhdr isn't the first elements anymore,
the only requirement is that it comes first in
the s/g list.

Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
---
 include/uapi/linux/virtio_blk.h | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/include/uapi/linux/virtio_blk.h b/include/uapi/linux/virtio_blk.h
index b695ba9..19c66fc 100644
--- a/include/uapi/linux/virtio_blk.h
+++ b/include/uapi/linux/virtio_blk.h
@@ -119,7 +119,11 @@ struct virtio_blk_config {
 #define VIRTIO_BLK_T_BARRIER	0x80000000
 #endif /* !VIRTIO_BLK_NO_LEGACY */
 
-/* This is the first element of the read scatter-gather list. */
+/*
+ * This comes first in the read scatter-gather list.
+ * For legacy virtio, if VIRTIO_F_ANY_LAYOUT is not negotiated,
+ * this is the first element of the read scatter-gather list.
+ */
 struct virtio_blk_outhdr {
 	/* VIRTIO_BLK_T* */
 	__virtio32 type;
-- 
MST

Re: [PATCH 2/2] virtio_blk: fix comment for virtio 1.0

[Rusty Russell]

"Michael S. Tsirkin" <mst@redhat.com> writes:
> Fix up comment to match virtio 1.0 logic:
> virtio_blk_outhdr isn't the first elements anymore,
> the only requirement is that it comes first in
> the s/g list.
>
> Signed-off-by: Michael S. Tsirkin <mst@redhat.com>

Thanks, both applied.

Cheers,
Rusty.

Re: [Qemu-devel] [PATCH 2/2] virtio_blk: fix comment for virtio 1.0

[Rusty Russell]

"Michael S. Tsirkin" <mst@redhat.com> writes:
> Fix up comment to match virtio 1.0 logic:
> virtio_blk_outhdr isn't the first elements anymore,
> the only requirement is that it comes first in
> the s/g list.
>
> Signed-off-by: Michael S. Tsirkin <mst@redhat.com>

Thanks, both applied.

Cheers,
Rusty.

Re: [PATCH 2/2] virtio_blk: fix comment for virtio 1.0

[Michael S. Tsirkin]

On Mon, Mar 02, 2015 at 10:45:37AM +1030, Rusty Russell wrote:
> "Michael S. Tsirkin" <mst@redhat.com> writes:
> > Fix up comment to match virtio 1.0 logic:
> > virtio_blk_outhdr isn't the first elements anymore,
> > the only requirement is that it comes first in
> > the s/g list.
> >
> > Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
> 
> Thanks, both applied.
> 
> Cheers,
> Rusty.

Hmm, I don't see them in your virtio-next tree.
Did I miss something?
I know these are just headers but it's in UAPI
so I'd like to make them correct ASAP.

Re: [Qemu-devel] [PATCH 2/2] virtio_blk: fix comment for virtio 1.0

[Michael S. Tsirkin]

On Mon, Mar 02, 2015 at 10:45:37AM +1030, Rusty Russell wrote:
> "Michael S. Tsirkin" <mst@redhat.com> writes:
> > Fix up comment to match virtio 1.0 logic:
> > virtio_blk_outhdr isn't the first elements anymore,
> > the only requirement is that it comes first in
> > the s/g list.
> >
> > Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
> 
> Thanks, both applied.
> 
> Cheers,
> Rusty.

Hmm, I don't see them in your virtio-next tree.
Did I miss something?
I know these are just headers but it's in UAPI
so I'd like to make them correct ASAP.

Re: [PATCH 2/2] virtio_blk: fix comment for virtio 1.0

[Michael S. Tsirkin]

On Mon, Mar 02, 2015 at 10:45:37AM +1030, Rusty Russell wrote:
> "Michael S. Tsirkin" <mst@redhat.com> writes:
> > Fix up comment to match virtio 1.0 logic:
> > virtio_blk_outhdr isn't the first elements anymore,
> > the only requirement is that it comes first in
> > the s/g list.
> >
> > Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
> 
> Thanks, both applied.
> 
> Cheers,
> Rusty.

Hmm, I don't see them in your virtio-next tree.
Did I miss something?
I know these are just headers but it's in UAPI
so I'd like to make them correct ASAP.