Re: [patch] remove page->virtual

[William Lee Irwin III]

On Thu, Sep 19, 2002 at 09:55:52PM -0700, Andrew Morton wrote:
> set_page_address() and page_address() implementations consume 0.4% and
> 1.3% of CPU time respectively.   I think that's OK. (Plus the tested code
> was doing an unneeded lookup in set_page_address(), for debug purposes)

Looks yummy. I'll take it for a spin tonight on my benchmark-o-matic.
Clearing some more air in ZONE_NORMAL is always welcome here.

On Thu, Sep 19, 2002 at 09:55:52PM -0700, Andrew Morton wrote:
> c01884f2 6914     10.5108     .text.lock.dir
> c01546b3 5847     8.88872     .text.lock.namei
> c01eb99e 3811     5.79355     .text.lock.dec_and_lock
> c01515dc 3775     5.73883     link_path_walk
> c015207c 3567     5.42262     path_lookup
> c015aba4 3194     4.85558     __d_lookup
> c01eb690 2814     4.2779      __generic_copy_to_user
> c01eb950 2562     3.8948      atomic_dec_and_lock
> c0187580 2473     3.7595      ext2_readdir
> c0155d3c 2172     3.30192     filldir64
> c0151114 1786     2.71511     path_release
> c0145b6a 1753     2.66494     .text.lock.open

What's going on here? fs stuff is really hurting. At any rate, the
overhead of the address calculation and hashtable lookup is microscopic
according to this, and I want space.

Cheers,
Bill