Re: [patch 05/11] dm-snap: remove SECTOR_SHIFT define

2007-08-13 Thread Jan Blunck
On Fri, Aug 10, Alasdair G Kergon wrote: > On Fri, Aug 10, 2007 at 10:02:09PM +0200, Jan Blunck wrote: > > Sector size on Linux is always 512 bytes. Don't even try to give the > > impression this is changeable. > > If that's what worries you, add a comment next to the definition, > perhaps? That

Re: [patch 05/11] dm-snap: remove SECTOR_SHIFT define

2007-08-10 Thread Alasdair G Kergon
On Fri, Aug 10, 2007 at 10:02:09PM +0200, Jan Blunck wrote: > Sector size on Linux is always 512 bytes. Don't even try to give the > impression this is changeable. If that's what worries you, add a comment next to the definition, perhaps? It's there so you can easily locate all the places within