Date: Fri, 08 Aug 2014 16:15:56 -0600 From: Ian Lepore <ian@FreeBSD.org> To: Fabien Thomas <fabient@freebsd.org> Cc: Tim Kientzle <tim@kientzle.com>, freebsd-arm@freebsd.org Subject: Re: [CFR] mge driver / elf reloc Message-ID: <1407536156.56408.412.camel@revolution.hippie.lan> In-Reply-To: <BDFA451C-D74E-4EA7-B1BA-F818E98AB372@freebsd.org> References: <14D22EA6-B73C-47BA-9A86-A957D24F23B8@freebsd.org> <1405810447.85788.41.camel@revolution.hippie.lan> <20140720220514.GP45513@funkthat.com> <F6D53A17-FED0-4F08-BB5B-9F66C5AF5EF6@kientzle.com> <20140720231056.GQ45513@funkthat.com> <BDFA451C-D74E-4EA7-B1BA-F818E98AB372@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, 2014-07-21 at 10:40 +0200, Fabien Thomas wrote: > On 21 Jul 2014, at 01:10, John-Mark Gurney <jmg@funkthat.com> wrote: > > > Tim Kientzle wrote this message on Sun, Jul 20, 2014 at 15:25 -0700: > >> > >> On Jul 20, 2014, at 3:05 PM, John-Mark Gurney <jmg@funkthat.com> wrote: > >> > >>> Ian Lepore wrote this message on Sat, Jul 19, 2014 at 16:54 -0600: > >>>> Sorry to take so long to reply to this, I'm trying to get caught up. I > >>>> see you've already committed the mge fixes. I think the ELF alignment > >>>> fix looks good and should also be committed. > >>> > >>> So, re the elf alignment... > >>> > >>> I think we should get a set of macros that handle load/stores to/from > >>> unaligned addresses that are transparent to the caller.... I need > >>> these for some other code I'm writing... > >>> > >>> I thought Open/Net had these available, but I can't seem to find them > >>> right now... > >> > >> $ man 9 byteorder > >> > >> is most of what you want, lacking only some aliases to pick > >> the correct macro for native byte order. > > > > Um, those doesn't help if you want native endian order... > > > > Also, only the enc/dec functions are documented to work on non-aligned > > address, so that doesn't help in most cases... > > Yes, having an API to read unaligned pointer is better than than using local fix like in the patch. > Tell me if you add one and I can adapt the patch. > > Fabien > So can we just get this patch committed as-is, and then maybe convert it later to the much-desired "something better" that this discussion deteriorated into? http://people.freebsd.org/~fabient/ARM/patch-arm_elf_alignfix If you've ever wondered whether there are real costs to bikeshedding, this is an example of such. This relocation patch is the fix to the kernel module loading problem HPS has been having, and who knows how much time he had to waste debugging it. I know I spent 4 hours on it today before discovering that the problem was known and fixed, only the fix isn't committed yet. -- Ian
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1407536156.56408.412.camel>