Subject: Re: CVS commit: src/sys/sys
To: Izumi Tsutsui <tsutsui@ceres.dti.ne.jp>
From: Christos Zoulas <christos@zoulas.com>
List: source-changes
Date: 06/06/2005 11:52:11
On Jun 7, 12:31am, tsutsui@ceres.dti.ne.jp (Izumi Tsutsui) wrote:
-- Subject: Re: CVS commit: src/sys/sys

| christos@zoulas.com wrote:
| 
| > | Hmm, bufpages is actually used by some files:
| > | sys/arch/powerpc/oea/oea_machdep.c
| Sorry, this doesn't refer it (only in comments).
| 
| > | sys/arch/sun2/sun2/machdep.c
| > | sys/compat/irix/irix_sysmp.c (local declaration added already)
| > | 
| > | Shouldn't they refer it?
| > 
| > It is always 0 these days. If they actually need the count, they should
| > use the new uvm variable.
| 
| But accroding to sys/kern/vfs_bio.c, we could still override it
| for bufmem_hiwater, coundn't it?
| (though sun2 value (37) seems overrided again by BUFMEM_HIWMMIN)

Yes, that is true. For that case they can still declare it locally
as an extern, instead of making it visible to the whole kernel.

christos