[EMAIL PROTECTED] wrote:

> Here's a new version of the patch.
> This should be more correct.
> But, I do make one assumption on line # 359 which I am not sure
> is correct.

Yes, this should be fine: if you already get page numbers, there's
no need to check for alignment, of course.

Just one question:  at some places you use 
  #if LINUX_VERSION_CODE >= VERSION_CODE(2,3,25)
and at others
  #if LINUX_VERSION_CODE >= VERSION_CODE(2,3,26)

(e.g. compare adding the /proc entry and removing it).

I'd assume that you'd want the same criterion for both,
but I'm not sure myself exactly when the new /proc handling
(or the new vm_pgoff, for that matter) was introduced ...

Bye,
Ulrich

Reply via email to