Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 14 Jan 2002 02:03:06 -0800 (PST)
From:      John Baldwin <jhb@FreeBSD.org>
To:        Alfred Perlstein <bright@mu.org>
Cc:        cvs-all@FreeBSD.org, cvs-committers@FreeBSD.org, Alan Cox <alc@FreeBSD.org>
Subject:   Re: cvs commit: src/sys/kern vfs_aio.c
Message-ID:  <XFMail.020114020306.jhb@FreeBSD.org>
In-Reply-To: <20020113233830.E7984@elvis.mu.org>

next in thread | previous in thread | raw e-mail | index | archive | help

On 14-Jan-02 Alfred Perlstein wrote:
> * Alan Cox <alc@FreeBSD.org> [020113 23:26] wrote:
>> alc         2002/01/13 23:26:33 PST
>> 
>>   Modified files:
>>     sys/kern             vfs_aio.c 
>>   Log:
>>    o Correct the initialization of aiolio_zone: Each entry was 16 times
>>    larger
>>      than necessary.
>>    o Move a rarely-used goto label inside a critical section so that we
>>    don't
>>      perform an splnet() for which there is no corresponding splx().
>>    o Remove unnecessary splnet()/splx() around accesses to
>>    kaioinfo::kaio_jobdone
>>      in aio_return().
>>    o Use TAILQ_FOREACH for simple cases of iteration over
>>    kaioinfo::kaio_jobdone.
> 
> I skipped over AIO for the fdlocking patches.  Any chance you can
> take a look at it?

Erm, in the future can you please do this before committing?  Or at least send
out a HEADS UP saying that AIO is broken after your commit?

-- 

John Baldwin <jhb@FreeBSD.org>  <><  http://www.FreeBSD.org/~jhb/
"Power Users Use the Power to Serve!"  -  http://www.FreeBSD.org/

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




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