From owner-svn-src-all@FreeBSD.ORG Wed Jul 21 09:04:59 2010 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 3761A1065675; Wed, 21 Jul 2010 09:04:59 +0000 (UTC) (envelope-from ivoras@gmail.com) Received: from mail-ww0-f51.google.com (mail-ww0-f51.google.com [74.125.82.51]) by mx1.freebsd.org (Postfix) with ESMTP id 46A348FC1C; Wed, 21 Jul 2010 09:04:57 +0000 (UTC) Received: by wwb22 with SMTP id 22so613483wwb.8 for ; Wed, 21 Jul 2010 02:04:57 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:mime-version:sender:received :in-reply-to:references:from:date:x-google-sender-auth:message-id :subject:to:cc:content-type:content-transfer-encoding; bh=NKF26/icjuF+uinYRejPl7MeooPmpOfk2+SJ3hsJLLs=; b=AtIf8rxm2CsvWt8TwUqkW8swuJvBicP465sQ6RTnVVwgCBMWGzHGG8Ni1crxEsGQbv 8WhdzFhONr8d0CfdRXBHPsMdFJnGKg4yqv46nGZ+zhz5Vy7Ee/ikQHxO8V8WxMCcLtWP q/7iNKs9Pk5iQqhCDZOT77TBG8utifWp1/pEQ= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:sender:in-reply-to:references:from:date :x-google-sender-auth:message-id:subject:to:cc:content-type :content-transfer-encoding; b=NrOD+5QCb8WJSz/lbs/QdcmVd7XWeScleFL8Cu4qEErKOVbYGHKJLbed2TeXkS+Uwo haLCM7bI3rabunCf6EakoSukIhxmYu5GPY51h/6QGhPB2/ZgEVc73zGffDynAFvKgMs0 jp4r/hbnjMF4C2FjLExB8yAPUyVd5LsF3kiQ8= Received: by 10.216.145.99 with SMTP id o77mr6443268wej.113.1279703097118; Wed, 21 Jul 2010 02:04:57 -0700 (PDT) MIME-Version: 1.0 Sender: ivoras@gmail.com Received: by 10.216.87.206 with HTTP; Wed, 21 Jul 2010 02:04:37 -0700 (PDT) In-Reply-To: <20100721134319.E7228@delplex.bde.org> References: <201007181015.o6IAFXvK018739@svn.freebsd.org> <201007190829.21995.jhb@freebsd.org> <20100721134319.E7228@delplex.bde.org> From: Ivan Voras Date: Wed, 21 Jul 2010 11:04:37 +0200 X-Google-Sender-Auth: GxH2EOCxwPk7RNzTKM0rjVjrFHM Message-ID: To: Bruce Evans Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, John Baldwin Subject: Re: svn commit: r210217 - head/sys/kern X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 21 Jul 2010 09:04:59 -0000 On 21 July 2010 06:18, Bruce Evans wrote: > On Mon, 19 Jul 2010, John Baldwin wrote: > >>> Log: >>> =C2=A0In keeping with the Age-of-the-fruitbat theme, scale up hirunning= space >>> on >>> =C2=A0machines which can clearly afford the memory. >>> >>> =C2=A0This is a somewhat conservative version of the patch - more fine = tuning >>> may be >>> =C2=A0necessary. >>> >>> =C2=A0Idea from: Thread on hackers@ >>> =C2=A0Discussed with: alc > > Sorry I didn't look at the thread, but I wonder if you should increase > lorunningspace similarly. The previous ratio of lorunningspace to hirunningspace was 1/2 - is this still a good target? > There is a possibly related problem with writing through file systems to > high-latency disk devices like dvds. =C2=A0getblk() often takes many seco= nds, > and occasionally takes a couple of _minutes_. =C2=A0dvd's aren't quite th= at > slow, and can easily write hirunningspace =3D 1MB in 1 second, except > possibly if the file system is not designed for high-latency devices > (like most including cd9660 and udf are :-() so it does lots of random > i/o). > > The above is mostly for 1 active file system... It does seem like there would be more benefitial to hang these variables per mount-point or something similar but I'm content that they are tunable and that the new values help high-end machines, probably in cooperation with tagged (NCQ-like) IO. >> Presumably you could use 'lmax(lmin(..., 16 * 1024 * 1024), 1024 * >> 1024))'? > > Better. > Normal formatting is sometimes broken to avoid lines longer than 80 > characters. =C2=A0This works here. =C2=A0I don't like this. Yes, this was the reason for the original patch's formatting :)