[Date Prev][Date Next] [Thread Prev][Thread Next]
[Thread Index]
[Date Index]
[Author Index]
Re: [dm-devel] [PATCH v2] dm: Fix alignment stacking on partitioned devices
- From: "Martin K. Petersen" <martin petersen oracle com>
- To: Mike Snitzer <snitzer redhat com>
- Cc: device-mapper development <dm-devel redhat com>, "Alasdair G. Kergon" <agk redhat com>, "Martin K. Petersen" <martin petersen oracle com>
- Subject: Re: [dm-devel] [PATCH v2] dm: Fix alignment stacking on partitioned devices
- Date: Tue, 05 Jan 2010 23:16:13 -0500
>>>>> "Mike" == Mike Snitzer <snitzer redhat com> writes:
Mike> But even with your blk_off_t patch (and prior to it with sector_t)
Mike> you're mixing int with blk_off_t in blk_stack_limits() by doing:
Mike> alignment = queue_limit_alignment_offset(b, offset);
Mike> This helped motivate my "blind" conversion.
Yeah. However, that particular wart is fixed by an unrelated patch in
my queue wrt. discard alignment. But I wanted something you guys could
look at that applied on top of upstream.
I'm tinkering with a few things on this end to see if I can make
the distinction between an offset and an alignment obvious.
--
Martin K. Petersen Oracle Linux Engineering
[Date Prev][Date Next] [Thread Prev][Thread Next]
[Thread Index]
[Date Index]
[Author Index]