vtd:fix "negative shift"

Shawnshh · lijinxia · commit c5c338aecce9 · 2018-06-22T12:18:45.000+08:00
MISRA C doesn't allowed negative shift, changed any potential signed value
to unsigned value.

Signed-off-by: Huihuang Shi &lt;huihuang.shi@intel.com&gt;
Acked-by: Eddie Dong &lt;eddie.dong@intel.com&gt;
diff --git a/hypervisor/arch/x86/vtd.c b/hypervisor/arch/x86/vtd.c
@@ -472,9 +472,9 @@ static struct dmar_drhd_rt *device_to_dmaru(uint16_t segment, uint8_t bus,
 	return NULL;
 }
 
-static int alloc_domain_id(void)
+static uint8_t alloc_domain_id(void)
 {
-	int i;
+	uint8_t i;
 	uint64_t mask;
 
 	spinlock_obtain(&domain_lock);
@@ -492,7 +492,7 @@ static int alloc_domain_id(void)
 	return i;
 }
 
-static void free_domain_id(int dom_id)
+static void free_domain_id(uint16_t dom_id)
 {
 	uint64_t mask = (1 << dom_id);
 

Original file line number	Diff line number	Diff line change
`@@ -472,9 +472,9 @@ static struct dmar_drhd_rt *device_to_dmaru(uint16_t segment, uint8_t bus,`
`472`	`472`	`return NULL;`
`473`	`473`	`}`
`474`	`474`
`475`		`-static int alloc_domain_id(void)`
	`475`	`+static uint8_t alloc_domain_id(void)`
`476`	`476`	`{`
`477`		`- int i;`
	`477`	`+ uint8_t i;`
`478`	`478`	`uint64_t mask;`
`479`	`479`
`480`	`480`	`spinlock_obtain(&domain_lock);`
`@@ -492,7 +492,7 @@ static int alloc_domain_id(void)`
`492`	`492`	`return i;`
`493`	`493`	`}`
`494`	`494`
`495`		`-static void free_domain_id(int dom_id)`
	`495`	`+static void free_domain_id(uint16_t dom_id)`
`496`	`496`	`{`
`497`	`497`	`uint64_t mask = (1 << dom_id);`
`498`	`498`