[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 1/1] xen:include:asm-x86:e820: just covert nr_map with unsigned int



On 2014/8/8 15:15, Jan Beulich wrote:
On 08.08.14 at 09:00, <tiejun.chen@xxxxxxxxx> wrote:
This shouldn't ever be negative.

This means nothing for the actual change, and makes me imply you did
the change without checking it actually _is_ never negative in any of
the uses.

Also please cut down your patch title prefixes (not just here) to
something reasonable.


What about this?

xen:e820map: just convert nr_map with unsigned int

nr_map is used to represent the amount of e820entry in e820 so it
shouldn't be negative.

Thanks
Tiejun

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.