From mboxrd@z Thu Jan  1 00:00:00 1970
From: "Chen, Tiejun" <tiejun.chen@intel.com>
Subject: Re: [v2][PATCH 1/1] x86: make struct e820map's nr_map
 unsigned
Date: Mon, 11 Aug 2014 10:52:57 +0800
Message-ID: <53E83009.3000604@intel.com>
References: <1407490349-24768-1-git-send-email-tiejun.chen@intel.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"; Format="flowed"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <1407490349-24768-1-git-send-email-tiejun.chen@intel.com>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: JBeulich@suse.com, keir@xen.org
Cc: xen-devel@lists.xen.org
List-Id: xen-devel@lists.xenproject.org

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];
>   };
>
>