Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 06 Feb 2008 18:34:25 +0200
From:      Andriy Gapon <avg@icyb.net.ua>
To:        pav@FreeBSD.org
Cc:        Bruce Evans <bde@zeta.org.au>, freebsd-hackers@FreeBSD.org, scottl@FreeBSD.org, freebsd-fs@FreeBSD.org, Julian Elischer <julian@elischer.org>, Remko Lodder <remko@FreeBSD.org>
Subject:   Re: fs/udf: vm pages "overlap" while reading  large dir [patch]
Message-ID:  <47A9E191.8000607@icyb.net.ua>
In-Reply-To: <1202235368.68281.12.camel@ikaros.oook.cz>
References:  <200612221824.kBMIOhfM049471@freefall.freebsd.org>	 <47A2EDB0.8000801@icyb.net.ua> <47A2F404.7010208@icyb.net.ua>	 <47A735A4.3060506@icyb.net.ua> <47A75B47.2040604@elischer.org>	 <1202155663.62432.0.camel@ikaros.oook.cz> <47A8754C.5010607@icyb.net.ua> <1202235368.68281.12.camel@ikaros.oook.cz>

next in thread | previous in thread | raw e-mail | index | archive | help
on 05/02/2008 20:16 Pav Lucistnik said the following:
> Andriy Gapon píše v út 05. 02. 2008 v 16:40 +0200:
> 
>>> Yay, and can you fix the sequential read performance while you're at it?
>>> Kthx!
> 
>> this was almost trivial :-)
>> See the attached patch, first hunk is just for consistency.
>> The code was borrowed from cd9660, only field/variable names are adjusted.
> 
> Omg looks good. Can't wait for it to bubble throught to 6-STABLE :)
> 
> Thanks million for working on this.
> 

Actually the patch is not entirely correct. max_size returned from
udf_bmap_internal should be used to calculate number of continuous
sectors for read-ahead (as opposed to file size in the patch).

-- 
Andriy Gapon



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