[Pkg-lustre-maintainers] Re: [Lustre-discuss] PAGE_SIZE usage in lustre and a debian fix.

[David Brown]

> I'm not sure that liblustre has ever been tested on ia64, though of course
> it is entirely possible that it works OK.  The value of PAGE_SIZE used by
> liblustre is pretty arbitrary I think, a large part of it is just so that
> the same code can be used by liblustre and linux.
>
> As for your patch, I haven't looked at it closely, but I'm confused why
> you would change PAGE_SIZE -> getpagesize(), and then later make a
> #define PAGE_SIZE?  Also, there is no guarantee that the ia64 PAGE_SIZE
> is 2^14, though whether liblustre really cares is another matter.

I've been having an on going conversation on lkml about it and they
really say, "use getpagesize()" but lustre uses PAGE_MASK PAGE_SHIFT
and all the other defines in your userspace code as well. So the first
part of the patch is what should be happening and the second part is
fast hack to get it to build with a 'safe' value for the page size.
Really the correct fix would be to obtain the value of PAGE_SIZE via
getpagesize() then create some functions to get PAGE_MASK and
PAGE_SHIFT from the value returned by getpagesize(). At least that's
what I'd do...

Thanks,
- David Brown