From: John Anthony Kazos Jr. <[EMAIL PROTECTED]>
This patch alters the (do...while) construct to a simple (while) and saves
one increment operation. It's entirely possible that gcc optimizes away
the first iteration anyway, but in case it doesn't (and also because it's
easier to read this way), I think this is better.
Signed-off-by: John Anthony Kazos Jr. <[EMAIL PROTECTED]>
---
--- linux-2.6.20.4/fs/buffer.c.orig 2007-03-26 09:42:15.000000000 -0400
+++ linux-2.6.20.4/fs/buffer.c 2007-03-26 10:10:25.000000000 -0400
@@ -1045,10 +1045,10 @@ grow_buffers(struct block_device *bdev,
pgoff_t index;
int sizebits;
- sizebits = -1;
- do {
+ sizebits = 0;
+ while ((size << sizebits) < PAGE_SIZE) {
sizebits++;
- } while ((size << sizebits) < PAGE_SIZE);
+ }
index = block >> sizebits;
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
