Subject: Re: CVS commit: src/sys/uvm
To: YAMAMOTO Takashi <yamt@mwd.biglobe.ne.jp>
From: Christos Zoulas <christos@zoulas.com>
List: source-changes
Date: 04/15/2006 12:25:37
On Apr 15,  6:38pm, yamt@mwd.biglobe.ne.jp (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