[PATCH] Fix off by one error in page_region_mask()

Felix Blyakher felixb at sgi.com
Mon Feb 16 11:14:59 CST 2009


On Feb 15, 2009, at 1:16 PM, Christoph Hellwig wrote:

> On Thu, Jan 22, 2009 at 05:25:39PM -0500, Christoph Hellwig wrote:
>> On Mon, Jan 19, 2009 at 05:53:37PM +1100, Lachlan McIlroy wrote:
>>> Lachlan McIlroy wrote:
>>>> Christoph Hellwig wrote:
>>>>> Did you guys come to a conclusion in your discussions?
>>>>>
>>>> No conclusion, well nothing that resulted in an ack.
>>>
>>> Was anyone okay with this fix?
>>
>> I think it's okay to go.  It fixes the one off so it's go.
>>
>> The code is still a very hard to understand mess, but that's not what
>> the patch was supposed to fix..
>
> Any reason this isn't in yet?

Sorry, I was holding it. I had some doubts with the mod, but
couldn't materialize them into a real argument.
I guess, I just check it in (I saw your ACK some time ago, Christoph),
and deal with anything popping up in this area later.

Felix




More information about the xfs mailing list