Skip to contentSkip to conversation

virtio additions

PATCH RESEND In review Dor Laor 1 participant 1 message Dec 21, 2007

Subsystems (2)
lore
Share & tools
lore

Dor Laor

author
#0 lore raw

PATCH RESEND 9/13 virtio additions

From e761e0d22af833c9d0d48f191f38fc25f6a5f753 Mon Sep 17 00:00:00 2001
From: Anthony Liguori <aliguori-r/[email protected]>
Date: Fri, 21 Dec 2007 02:17:47 +0200
Subject: [PATCH] Fix vring_init/vring_size to take unsigned long

Using unsigned int resulted in silent truncation of the upper 32-bit on x86_64
resulting in an OOPS since the ring was being initialized wrong.

Please reconsider my previous patch to just use PAGE_ALIGN(). Open coding this
sort of stuff, no matter how simple it seems, is just asking for this sort of
trouble.

---
 include/linux/virtio_ring.h |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)
include/linux/virtio_ring.h +1 -1
index 1a4ed49..5de09f9 100644
@@ -89,7 +89,7 @@ struct vring {
  * };
  */
 static inline void vring_init(struct vring *vr, unsigned int num, void *p,
-                  unsigned int pagesize)
+                  unsigned long pagesize)
 {
     vr->num = num;
     vr->desc = p;
@@ -98,7 +98,7 @@ static inline void vring_init(struct vring *vr, 

unsigned int num, void *p,

                 & ~(pagesize - 1));

}

-static inline unsigned vring_size(unsigned int num, unsigned int pagesize) +static inline unsigned vring_size(unsigned int num, unsigned long pagesize)
{
return ((sizeof(struct vring_desc) * num + sizeof(__u16) * (2 + num)
+ pagesize - 1) & ~(pagesize - 1))

Signature · 7 lines
1.5.3.3


-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/