Date: Wed, 23 Apr 2008 09:26:10 +1000 From: Andrew Snow <andrew@modulus.org> To: Zaphod Beeblebrox <zbeeble@gmail.com> Cc: freebsd-stable@freebsd.org, Pete French <petefrench@ticketswitch.com> Subject: Re: Dreadful gmirror performance, though each half works fine Message-ID: <480E7412.2040802@modulus.org> In-Reply-To: <5f67a8c40804221136s2c1893c0tdd00c627ab813c59@mail.gmail.com> References: <E1JnuQf-000LaE-5n@dilbert.ticketswitch.com> <480D7F58.1080203@modulus.org> <5f67a8c40804221136s2c1893c0tdd00c627ab813c59@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
Zaphod Beeblebrox wrote: > I use this patch for sbin/geom/class/mirror/geom_mirror.c > > Change: > md.md_priority = i - 1; > To: > md.md_priority = i - 1 + 100; > > > I hate to ask for the "right" solution, but shouldn't we be patching > the gmirror userland to accept a priority argument to label and make the > kernel part listen to that? This patch does make sense --- but it > doesn't go far enough. > > Also, it seems sensible that you should be able to modify the priority > values of a running disk. Both of those are good ideas. But for years, no one can be bothered making a patch. At least my patch is only one line, and solves 90% of the problem, and still no one can be bothered committing it. Maybe we should apply my patch for now, until someone works on the rest. - Andrew
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?480E7412.2040802>