Re: NFS directio

[Chuck Lever <cel@citi.umich.edu>]

Olaf Kirch wrote:
> On Fri, Mar 31, 2006 at 09:35:34AM -0500, Chuck Lever wrote:
>> the check isn't in 2.6.16.  it was removed sometime after 2.6.5.
> 
> It is still in the 2.6.16 tree I'm looking at; else I wouldn't ask :)

it's been in my trees since 2.6.13 or even earlier, my mistake.

that change is part of the aio+dio patches that were just included in 
2.6.17-rc1.  instead of creating a single patch for this change, you 
should consider taking those patches, since they were tested as a unit.

if you can guarantee that atomic_t is 32-bits on every platform you 
support, then it should be save to change that #define to 2^31. 
otherwise, the work to eliminate the limit entirely has already been 
done by the above-mentioned patches.

-- 
corporate:	<cel at netapp dot com>
personal:	<chucklever at bigfoot dot com>


-------------------------------------------------------
This SF.Net email is sponsored by xPML, a groundbreaking scripting language
that extends applications into web and mobile media. Attend the live webcast
and join the prime developer group breaking into this new coding territory!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=110944&bid=241720&dat=121642
_______________________________________________
NFS maillist  -  NFS@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs