[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 08.08.14 at 09:26, <tiejun.chen@xxxxxxxxx> wrote:
> 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

x86: make struct e820map's nr_map unsigned

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

You still say "shouldn't" instead of "isn't", making me still imply you
didn't verify that fact. I.e. I'd be fine with a suitable variant of the
earlier more terse description as long as it says what it needs to say.


Xen-devel mailing list



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