On Thu, Mar 20, 2014 at 10:17:14PM +0000, Peter Maydell wrote: > On 20 March 2014 22:06, Marcel Apfelbaum <marce...@redhat.com> wrote: > > On Thu, 2014-03-20 at 22:57 +0100, Paolo Bonzini wrote: > >> Il 20/03/2014 22:14, Marcel Apfelbaum ha scritto: > >> > +# All known versions of iasl on BE machines are broken. > >> > +# TODO: add detection code once a non-broken version makes an > >> > appearance. > >> > +if ($iasl -h > /dev/null 2>&1) && > >> > + (lscpu | grep "Byte Order" | grep --quiet "Little Endian" ); then > >> > >> lscpu is not portable. > > I am open to suggestions... > > echo "trivial iasl source" | iasl --compile-options | iasl > --disassemble-options | grep "error" > > Fill in the handwaving with actual syntax ;-) > > thanks > -- PMM
True but whatever test we write, it's possible that a buggy iasl passes it by luck. Let's wait for a working iasl to appear on some BE machines. When that happens, we'll be able to write a robust detection script. -- MST
