Hi, ----- Original Message -----
From: "Andrew Morton"
To: "Andrei Warkentin" Cc: linux-kernel@vger.kernel.org, "Rolf Eike Beer" , opensuse-kernel@opensuse.org, "Sergiu Iordache" , "Marco Stornelli" , "Eddie Wai" , "Jayamohan Kallickal" , "Guennadi Liakhovetski" Sent: Thursday, November 17, 2011 6:05:49 PM Subject: Re: [PATCH] include/log2.h: Fix rounddown_pow_of_two(1) I assume that nobody has gone off and checked whether all current callers will survive this change. If they had, they'd have looked in drivers/char/ramoops.c and seen:
rounddown_pow_of_two(pdata->mem_size); rounddown_pow_of_two(pdata->record_size);
These operations are no-ops. It should be
pdata->mem_size = rounddown_pow_of_two(pdata->mem_size); pdata->record_size = rounddown_pow_of_two(pdata->record_size);
Marco or Sergio: please fix, test and send it over sometime?
I did quickly look through for code that expected rounddown_pow_of_two(1) to give 0, but I didn't apparently look close enough for other issues. A -- To unsubscribe, e-mail: opensuse-kernel+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse-kernel+owner@opensuse.org