Date: Mon, 13 Jun 2005 14:26:12 -0600 From: Scott Long <scottl@samsco.org> To: Jeff Roberson <jroberson@chesapeake.net> Cc: cvs-src@FreeBSD.org, Jeff Roberson <jeff@FreeBSD.org>, src-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/kern subr_disk.c Message-ID: <42ADEBE4.2020308@samsco.org> In-Reply-To: <20050612203027.B16943@mail.chesapeake.net> References: <200506122232.j5CMWTIb020093@repoman.freebsd.org> <42ACBADB.70109@samsco.org> <20050612202915.C16943@mail.chesapeake.net> <20050612203027.B16943@mail.chesapeake.net>
next in thread | previous in thread | raw e-mail | index | archive | help
Jeff Roberson wrote: > On Sun, 12 Jun 2005, Jeff Roberson wrote: > > >>On Sun, 12 Jun 2005, Scott Long wrote: >> >> >>>Jeff Roberson wrote: >>> >>> >>>>jeff 2005-06-12 22:32:29 UTC >>>> >>>> FreeBSD src repository >>>> >>>> Modified files: >>>> sys/kern subr_disk.c >>>> Log: >>>> - Dramatically simplify bioqdisksort(). We no longer do ordered bios so >>>> most of the code to deal with them has been dead for sometime. Simplify >>>> the code by doing an insert sort hinted by the current head position. >>>> >>>> Met with apathy by: arch@ >>>> >>>> Revision Changes Path >>>> 1.84 +40 -85 src/sys/kern/subr_disk.c >>> >>>Not exactly apathy, just lack of time ;-( I trust that this has been >>>adequately tested on many different workloads? Also, how exactly does >>>this fall under your blanket approval to fix VFS? >> >>I was not aware of any current code freeze. Is there one? > > > I see, sorry. I missed by 24hrs. I was waiting for feedback on arch, but > I've been running with this patch for some time. > > Speaking of the freeze, I'd really like to get the interrupt thread perf > fixes in before 6.0. I think it should be a priority to fix ithread > performance. > This sounds fine, just send an email to re@ about it since it falls outside of your existing blanket approval. Scott
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?42ADEBE4.2020308>