[lkml]   [2010]   [Jan]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH -mmotm-2010-01-06-14-34] Fix fault count of task in GUP
On Tue, Jan 12, 2010 at 8:29 AM, Hugh Dickins
<> wrote:
> On Mon, 11 Jan 2010, Minchan Kim wrote:
>> get_user_pages calls handle_mm_fault to pin the arguemented
>> task's page. handle_mm_fault cause major or minor fault and
>> get_user_pages counts it into task which is passed by argument.
>> But the fault happens in current task's context.
>> So we have to count it not argumented task's context but current
>> task's one.
> Have to?
> current simulates a fault into tsk's address space.
> It is not a fault into current's address space.
> I can see that this could be argued either way, or even
> that such a "fault" should not be counted at all; but I do not
> see a reason to change the way we have been counting it for years.
> Sorry, but NAK (to this and to the v2) -
> unless you have a stronger argument.

Okay. The I/O to get a page happen current's context.
So I thought we have to count it with current.
But now that I think about it, yes. It's not current's _fault_.
I agree with your opinion.

Thanks for correcting me. Hugh.

Kind regards,
Minchan Kim

 \ /
  Last update: 2010-01-12 01:07    [W:0.042 / U:2.852 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site