Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 26 May 2007 02:03:02 +0200
From:      Attilio Rao <attilio@FreeBSD.org>
To:        Alan Cox <alc@cs.rice.edu>
Cc:        alc@freebsd.org, arch@freebsd.org, Jeff Roberson <jeff@freebsd.org>, Bruce Evans <bde@optusnet.com.au>
Subject:   Re: sched_lock && thread_lock()
Message-ID:  <46577936.9060804@FreeBSD.org>
In-Reply-To: <46568A72.3020406@cs.rice.edu>
References:  <20070520155103.K632@10.0.0.1> <20070521113648.F86217@besplex.bde.org> <20070520213132.K632@10.0.0.1> <4651CAB8.8070007@FreeBSD.org> <4651CE2F.8080908@FreeBSD.org> <20070521022847.D679@10.0.0.1> <20070521195811.G56785@delplex.bde.org> <4651FCB5.7070604@FreeBSD.org> <20070521225032.C57233@delplex.bde.org> <20070522162819.N5249@besplex.bde.org> <20070522201336.C87981@besplex.bde.org> <46533CAD.8030104@FreeBSD.org> <4655C67A.9060000@FreeBSD.org> <46568A72.3020406@cs.rice.edu>

next in thread | previous in thread | raw e-mail | index | archive | help
Alan Cox wrote:
> Attilio Rao wrote:
> 
>> Attilio Rao wrote:
>>
>>> Bruce Evans wrote:
>>>
>>>> 4 more translation errors breaking 8 counters altogether (v_vnodepgsin
>>>> is broken twice):
>>>
>>>
>>> Thanks a lot for the revision, there will be a pending patch in the 
>>> next hour.
>>
>>
>> Hello,
>> Let me know if this patch is right for you and if you have feedbacks, 
>> comments, etc:
>> http://users.gufi.org/~rookie/works/patches/schedlock/vmmeter3.diff
>>
>> This should fix translation errors Bruce has found and switching the 
>> _SET() method in order to being a simple assignment (as Bruce has 
>> suggested).
> 
> 
> I think there are still translation errors in vnode_pager.c:
> 
> This increment by 1 of cnt.v_vnodepgsin is incorrect:

ok, following the last suggestions I updated the patch. The URL is above.

Thanks,
Attilio




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?46577936.9060804>