[lkml]   [2000]   [Sep]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [patch-2.4.0-test8-pre6] misc fixes
On Thu, 7 Sep 2000, Linus Torvalds wrote:

> On Thu, 7 Sep 2000, Tigran Aivazian wrote:
> >
> > k) all swapout functions in mm/vmscan.c can be optimized by removing 'mm'
> > argument. This part was reviewed by Rick van Riel and approved.
> But they then get "mm" themselves anyway.
> What's the point? With argument passing, on certain architectures it stays
> in registers. On other architectures it is in memory on the stack, but
> that's not slower than accessing it from memory off another pointer.
> Linus

Linus, I am sorry to say this (because I know you are busy) but it would
appear you didn't look at the patch (that part of it). The patch does the
right thing, I believe, but my description was too brief.

If you look at those functions you will see that they sometimes access
'mm' via argument and sometimes via vm_mm->mm - this is a complete mess so
my patch tidies it up a bit.

In other words, if that 'mm' is available via vm_mm->mm there is no point
in passing it on the stack. Or if we pass it on the stack, there is no
point accessing it via vm_mm->mm. See my point now?


To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
Please read the FAQ at

 \ /
  Last update: 2005-03-22 12:38    [W:0.052 / U:1.144 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site