* "Yinghai Lu" <yhlu.kernel at gmail.com> [2008-06-25 09:58]: > On Wed, Jun 25, 2008 at 9:03 AM, Bernhard Walle <bwalle at suse.de> wrote: > >> only this one is needed now. but please change ULLONG_MAX to > >> ULLONG_MAX - mem_size > > > > Why can't we add that check at the beginning of e820_update_range() as > > you suggested? > that patch about fixing e820_update_rang is in tip/setup-memory. > > so the one is supposed to be ok, but it is good to keep good input > parameter too. Okay, thanks for review and comments. Patch sent. Bernhard -- Bernhard Walle, SUSE LINUX Products GmbH, Architecture Development