Source-Changes archive

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

Re: CVS commit: src/sys/uvm



On Apr 15,  6:38pm, yamt%mwd.biglobe.ne.jp@localhost (YAMAMOTO Takashi) wrote:
-- Subject: Re: CVS commit: src/sys/uvm

| 
| --NextPart-20060415183815-0229801
| Content-Type: Text/Plain; charset=us-ascii
| 
| > Right, what I am saying is that we should either be using:
| > 
| >     if (entry->object.uvm_obj)
| > 
| > or:
| >     if (UVM_ET_ISOBJ(entry))
| > 
| > consistently or even add the null test as a debugging aid inside the macro.
| > Using both is confusing.
| 
| then, how about the attached diff?
| 
| YAMAMOTO Takashi

I like it, but then I am not a uvm expert.

christos



Home | Main Index | Thread Index | Old Index