Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/sys/uvm



hi,

> Hi, thanks for review.
> 
> On Thu, Nov 25, 2010 at 01:58:04AM +0000, YAMAMOTO Takashi wrote:
>> hi,
>> 
>> - what's VM_PHYSSEG_OP_PG?
> 
> It's to lookup vm_physseg by "struct vm_page *", relying on that
> "struct vm_page *[]" is allocated linearly.  It'll be used to remove
> vm_page::phys_addr as we talked some time ago.

i'm not sure if commiting this unused uncommented code now helps it.
some try-and-benchmark cycles might be necessary given that
vm_page <-> paddr conversion could be performace critical.

YAMAMOTO Takashi


Home | Main Index | Thread Index | Old Index