Messages in this thread | | | Date | Tue, 24 Jul 2018 17:42:51 +0800 | From | Sean Fu <> | Subject | Re: [PATCH] fs/buffer.c: Optimize grow_buffer function. |
| |
On Sat, Jul 21, 2018 at 07:21:16PM +0100, Al Viro wrote: > On Sun, Jul 22, 2018 at 01:30:17AM +0800, Sean Fu wrote: > > Use PAGE_SHIFT and i_blkbits of bd_inode directly to avoid ugly sizebits > > calculation. > > Remove ugly sizebits calculation. > > Remove unnecessary sizebits parameter of grow_dev_page. > > > > Reduces code size: > > > > Before: > > > > sean@linux-zmni:~/sda5/source/linus_repo/linux> size fs/buffer.o > > text data bss dec hex filename > > 34037 1510 16 35563 8aeb fs/buffer.o > > > > After: > > > > sean@linux-zmni:~/sda5/source/linus_repo/linux> size fs/buffer.o > > text data bss dec hex filename > > 34021 1510 16 35547 8adb fs/buffer.o > > First of all, 16 bytes is pretty much noise. What's more, the "remove ugly > sizebits calculation" part really needs an explanation of the reasons why > it's safe. You assume that size == 1<<bdev->bd_inode->i_blkbits; where's > the proof that it always holds? Thank your for taking a look and apologize for the noise.
I will take a deep look at it and especially history commits regarding with this.
BestRegards Sean
| |