Re: 2.6.5-rc2-aa1

[Andrea Arcangeli]

On Sun, Mar 21, 2004 at 10:43:41PM -0500, Rik van Riel wrote:
> Presuming you'll never debug your code ;)

typedefs are more readable to my eyes, so if they're there it's more
debuggable, infact I liked when Ingo tried to inject the page_t thing
(was him Ingo or somebody else?)

> They're _LINUX_RMAP_H and not _LINUX_OBJRMAP_H.  If you want
> to be consistent you may want to either rename the inclusion
> guards, or the file ;)

I see what you mean now, I agree.

> Only if (1) you're using bitkeeper and (2) you used 'bk mv'
> to move rmap.c to objrmap.c and (3) Linus pulls from your
> bitkeeper tree.

I know this would be expected (this is what happens with arch too in the
explicit mode, the one that I prefer to be strict in the commits), but I
was just trying to say that Miles theory is that BK gets it right
automatically, either that or Linus's scripts gets it right before
injecting it into bk, that is his whole point. Miles sent to Linus
_patches_ (obviously w/o bk, AFIK Miles is not legally allowed to use bk
and Larry even refused him to sell him a commercial licence of bk so
that Miles could use bk while still working on arch) renaming files, and
he then found the renames being catched correctly in the web.

> Unless all 3 of these are true, you're giving bitkeeper more
> credit than it deserves ;)

It may not be bk catching the renames but just an external script that
Linus uses. I don't know.

> > I renamed it primarly because rmap is the common name for the tecnique
> > of traking the pagetables with pte_chains
> 
> Funny, first thing I hear about that ;)

Not sure after all those discussions how you may not have ever noticed
that people uses objrmap to mean something different than rmap, it's
really hard to believe that you never noticed.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/