Re: [PATCH 0/4] eliminate SECTOR related magic numbers and duplicated conversions
From: antlists
Date: Tue May 05 2020 - 13:32:40 EST
On 05/05/2020 12:55, Zhen Lei wrote:
When I studied the code of mm/swap, I found "1 << (PAGE_SHIFT - 9)" appears
many times. So I try to clean up it.
1. Replace "1 << (PAGE_SHIFT - 9)" or similar with SECTORS_PER_PAGE
2. Replace "PAGE_SHIFT - 9" with SECTORS_PER_PAGE_SHIFT
3. Replace "9" with SECTOR_SHIFT
4. Replace "512" with SECTOR_SIZE
Naive question - what is happening about 4096-byte sectors? Do we need
to forward-plan?
Cheers,
Wol