2011/7/19 Marcel Moolenaar <mar...@xcllnt.net>: > > On Jul 18, 2011, at 8:19 AM, Attilio Rao wrote: > >> Author: attilio >> Date: Mon Jul 18 15:19:40 2011 >> New Revision: 224187 >> URL: http://svn.freebsd.org/changeset/base/224187 >> >> Log: >> - Remove the eintrcnt/eintrnames usage and introduce the concept of >> sintrcnt/sintrnames which are symbols containing the size of the 2 >> tables. >> - For amd64/i386 remove the storage of intr* stuff from assembly files. >> This area can be widely improved by applying the same to other >> architectures and likely finding an unified approach among them and >> move the whole code to be MI. More work in this area is expected to >> happen fairly soon. >> >> No MFC is previewed for this patch. > > You just broke ia64 and possibly other 64-bit architectures: > > ".word" declares a 16-bit integral on ia64 and the size symbols > are of type size_t (=64 bit). We'll be having misaligned loads > (= kernel panics) and/or reading garbage...
I'm a bit surprised of this though. .hword was supposed to be the 16-bit integral, while .word was supposed to be the 32-bits one, if I read my "info as" on amd64. Anyway, what do you think about this patch? (I still need to test it): http://www.freebsd.org/~attilio/64bits-fixup.diff Thanks, Attilio -- Peace can only be achieved by understanding - A. Einstein _______________________________________________ svn-src-all@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/svn-src-all To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"
