From owner-freebsd-fs@FreeBSD.ORG Thu Feb 28 19:04:05 2008 Return-Path: Delivered-To: freebsd-fs@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 8B3821065679; Thu, 28 Feb 2008 19:04:05 +0000 (UTC) (envelope-from scottl@samsco.org) Received: from pooker.samsco.org (pooker.samsco.org [168.103.85.57]) by mx1.freebsd.org (Postfix) with ESMTP id 2A88C8FC2E; Thu, 28 Feb 2008 19:04:05 +0000 (UTC) (envelope-from scottl@samsco.org) Received: from phobos.samsco.home (phobos.samsco.home [192.168.254.11]) (authenticated bits=0) by pooker.samsco.org (8.13.8/8.13.8) with ESMTP id m1SJ42cS041119; Thu, 28 Feb 2008 12:04:02 -0700 (MST) (envelope-from scottl@samsco.org) Message-ID: <47C705A2.9030502@samsco.org> Date: Thu, 28 Feb 2008 12:04:02 -0700 From: Scott Long User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X; en-US; rv:1.8.1.11) Gecko/20071128 SeaMonkey/1.1.7 MIME-Version: 1.0 To: Andriy Gapon 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> <1204053786.1799.0.camel@ikaros.oook.cz> <47C671CF.9020101@icyb.net.ua> In-Reply-To: <47C671CF.9020101@icyb.net.ua> X-Enigmail-Version: 0.95.6 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-1.4 required=5.4 tests=ALL_TRUSTED autolearn=failed version=3.1.8 X-Spam-Checker-Version: SpamAssassin 3.1.8 (2007-02-13) on pooker.samsco.org Cc: Bruce Evans , freebsd-hackers@FreeBSD.org, freebsd-fs@FreeBSD.org, pav@FreeBSD.org, Julian Elischer , Remko Lodder Subject: Re: fs/udf: vm pages "overlap" while reading large dir [patch] X-BeenThere: freebsd-fs@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Filesystems List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 28 Feb 2008 19:04:07 -0000 Andriy Gapon wrote: > on 26/02/2008 21:23 Pav Lucistnik said the following: >> Pav Lucistnik píše v út 05. 02. 2008 v 19:16 +0100: >>> 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. >> Just tested it with my shiny new Bluray drive, and it work wonders. >> Finally seamless playback of media files off UDF carrying media. >> >> So, how does it look WRT committing it? >> > > Pav, > > thank you for the feedback/reminder. > In my personal option the latest patch posted and described at the > following like is a very good candidate for commit: > http://docs.FreeBSD.org/cgi/mid.cgi?47AA43B9.1040608 > It might have a couple of style related issues, but it should improve > things a bit even if some larger VM/VFS/GEOM issues remain. > > And by the way, a patch from the following PR would be a good > "side-dish" for the above patch: > http://www.freebsd.org/cgi/query-pr.cgi?pr=kern/78987 > I think it is simple and obvious enough. > I will commit both of these to CVS today. Thanks again. Scott