All of lore.kernel.org
 help / color / mirror / Atom feed
* [v2][PATCH 1/1] x86: make struct e820map's nr_map unsigned
@ 2014-08-08  9:32 Tiejun Chen
  2014-08-11  2:52 ` Chen, Tiejun
  0 siblings, 1 reply; 2+ messages in thread
From: Tiejun Chen @ 2014-08-08  9:32 UTC (permalink / raw)
  To: JBeulich, keir; +Cc: xen-devel

nr_map is used to represent the amount of e820entry in e820 so
this is never negative.

Signed-off-by: Tiejun Chen <tiejun.chen@intel.com>
---
 xen/include/asm-x86/e820.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

v2:

* Just refine the patch description.

diff --git a/xen/include/asm-x86/e820.h b/xen/include/asm-x86/e820.h
index 8418fc4..0bd5c2f 100644
--- a/xen/include/asm-x86/e820.h
+++ b/xen/include/asm-x86/e820.h
@@ -19,7 +19,7 @@ struct __packed e820entry {
 #define E820MAX	128
 
 struct e820map {
-    int nr_map;
+    unsigned int nr_map;
     struct e820entry map[E820MAX];
 };
 
-- 
1.9.1

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

* Re: [v2][PATCH 1/1] x86: make struct e820map's nr_map unsigned
  2014-08-08  9:32 [v2][PATCH 1/1] x86: make struct e820map's nr_map unsigned Tiejun Chen
@ 2014-08-11  2:52 ` Chen, Tiejun
  0 siblings, 0 replies; 2+ messages in thread
From: Chen, Tiejun @ 2014-08-11  2:52 UTC (permalink / raw)
  To: JBeulich, keir; +Cc: xen-devel

I assume this should be fine to you guy.

Thanks
Tiejun

On 2014/8/8 17:32, Tiejun Chen wrote:
> nr_map is used to represent the amount of e820entry in e820 so
> this is never negative.
>
> Signed-off-by: Tiejun Chen <tiejun.chen@intel.com>
> ---
>   xen/include/asm-x86/e820.h | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> v2:
>
> * Just refine the patch description.
>
> diff --git a/xen/include/asm-x86/e820.h b/xen/include/asm-x86/e820.h
> index 8418fc4..0bd5c2f 100644
> --- a/xen/include/asm-x86/e820.h
> +++ b/xen/include/asm-x86/e820.h
> @@ -19,7 +19,7 @@ struct __packed e820entry {
>   #define E820MAX	128
>
>   struct e820map {
> -    int nr_map;
> +    unsigned int nr_map;
>       struct e820entry map[E820MAX];
>   };
>
>

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

end of thread, other threads:[~2014-08-11  2:52 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-08-08  9:32 [v2][PATCH 1/1] x86: make struct e820map's nr_map unsigned Tiejun Chen
2014-08-11  2:52 ` Chen, Tiejun

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.