Date: Tue, 05 Feb 2008 22:15:01 -0800 From: Julian Elischer <julian@elischer.org> To: Andriy Gapon <avg@icyb.net.ua> Cc: Bruce Evans <bde@zeta.org.au>, freebsd-hackers@FreeBSD.org, scottl@FreeBSD.org, freebsd-fs@FreeBSD.org, pav@FreeBSD.org, Remko Lodder <remko@FreeBSD.org> Subject: Re: fs/udf: vm pages "overlap" while reading large dir [patch] Message-ID: <47A95065.3060703@elischer.org> In-Reply-To: <47A8754C.5010607@icyb.net.ua> 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>
next in thread | previous in thread | raw e-mail | index | archive | help
Andriy Gapon wrote: > on 04/02/2008 22:07 Pav Lucistnik said the following: >> Julian Elischer píše v po 04. 02. 2008 v 10:36 -0800: >>> Andriy Gapon wrote: >>>> More on the problem with reading big directories on UDF. >>> You do realise that you have now made yourself the official >>> maintainer of the UDF file system by submitting a competent >>> and insightful analysis of the problem? >> Yay, and can you fix the sequential read performance while you're at it? >> Kthx! >> > > Pav, > > this was almost trivial :-) Ok that settles it.. it's yours. > See the attached patch, first hunk is just for consistency. > The code was borrowed from cd9660, only field/variable names are adjusted. > > But there is another issue that I also mentioned in the email about > directory reading. It is UDF_INVALID_BMAP case of udf_bmap_internal, > i.e. the case when file data is embedded into a file entry. > This is a special case that needs to be handled differently. > udf_readatoffset() handles it, but the latest udf_read code doesn't. > I have a real UDF filesystem where this type of allocation is used for > small files and those files can not be read. > > This is described in Part 4, section 14.6.8 of ECMA-167. > >
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?47A95065.3060703>