Date: Wed, 1 Oct 2008 15:11:13 +0200 From: Ulf Lilleengen <lulf@freebsd.org> To: "Rick C. Petty" <rick-freebsd@kiwi-computer.com> Cc: freebsd-geom@freebsd.org Subject: Re: [patch] geom_vinum platform fixes Message-ID: <20081001131113.GB1419@carrot.studby.ntnu.no> In-Reply-To: <20081001130842.GA1419@carrot.studby.ntnu.no> References: <20080310052711.GA49676@keira.kiwi-computer.com> <20080313153551.82wlu8iio4088c44@webmail.ntnu.no> <20080313182257.GB14969@keira.kiwi-computer.com> <20080316090554.GA1230@carrot.studby.ntnu.no> <20080316185055.GA64920@keira.kiwi-computer.com> <20080407075843.GA1391@carrot.studby.ntnu.no> <20080407142713.GA62578@keira.kiwi-computer.com> <20081001070911.GA1626@carrot.studby.ntnu.no> <20081001130842.GA1419@carrot.studby.ntnu.no>
next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, Oct 01, 2008 at 03:09:00PM +0200, Ulf Lilleengen wrote: > On Wed, Oct 01, 2008 at 09:09:12AM +0200, Ulf Lilleengen wrote: > > On man, apr 07, 2008 at 09:27:13am -0500, Rick C. Petty wrote: > > > On Mon, Apr 07, 2008 at 09:58:43AM +0200, Ulf Lilleengen wrote: > > > > On søn, mar 16, 2008 at 12:50:55pm -0600, Rick C. Petty wrote: > > > > > On Sun, Mar 16, 2008 at 10:05:55AM +0100, Ulf Lilleengen wrote: > > > > > > > > > > > > I've reviewed the patch and done some modifications to it. I'll need some > > > > > > testing first though (I don't have a testbed right now since I'm travelling). > > > > > > > > > > I've reviewed your patch but haven't had time to test it yet (hopefully > > > > > this week..) > > > > > > > > > Have you been able to test it? I don't have any 64-bit hardware at the > > > > moment :) > > > > > > Sorry, I've been busy with other projects. I'll try to get to it this > > > week, although I can't see why your patch would break anything. > > > > > > > It has been a long time, but I was able to test the patch on i386, amd64 and > > sparc64, and unfortunately it did not work on sparc64. However, I have now > > modified it to support big/little endian 32/64-bit and it seems to work on > > both sparc64 and i386, and the testing is successful on amd64 as well (after > > I get home today), I will commit the attached patch. > > > > Sorry for the long delay :) > > > > And the patch :) > Hrmf, does not seem to get through. Here then: http://people.freebsd.org/~lulf/patches/gvinum/gvinum_platformfix.diff -- Ulf Lilleengen
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20081001131113.GB1419>