On Fri, 2014-03-21 at 00:16 +0100, Paolo Bonzini wrote: > Il 20/03/2014 23:33, Marcel Apfelbaum ha scritto: > > I've seen something like that somewhere, but I didn't quite like it. > > I was looking for something more elegant as I was *almost* sure > > this kind of solution will not pass the reviews :) > > > > But maybe I'll try this, let's see what happens, > > If all you're looking for is bigendian (disabling iasl disassembly on > bigendian makes sense), your patch v2 is fine.
Thanks Paolo, I thought so too, but Michael NACKED it having a valid point: We should check this at configure time... . Anyway, if only iasl dis-assembler has issues, this can be seen as acpi test's problem and handled inside the test. Thanks, Marcel > > Assembling ASL on bigendian is supported by at least Fedora and Debian > (and hence Ubuntu). > > Paolo