[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] aspeed: Add an AST2600 eval board
From: |
Peter Maydell |
Subject: |
Re: [PATCH] aspeed: Add an AST2600 eval board |
Date: |
Tue, 22 Oct 2019 17:43:49 +0100 |
On Thu, 17 Oct 2019 at 14:54, Peter Maydell <address@hidden> wrote:
>
> On Thu, 17 Oct 2019 at 07:33, Joel Stanley <address@hidden> wrote:
> >
> > On Wed, 16 Oct 2019 at 09:08, Cédric Le Goater <address@hidden> wrote:
> > >
> > > Define the board with 1 GiB of RAM but some boards can have up to 2
> > > GiB.
> > >
> > > Signed-off-by: Cédric Le Goater <address@hidden>
> > > Reviewed-by: Joel Stanley <address@hidden>
> > > ---
> > >
> > > Changes since AST2600 patchset:
> > >
> > > - lowered the RAM size to 1 GiB as it was breaking the tests on some
> > > hosts.
> >
> > Peter,
> >
> > After chatting with Cédric I agree we should merge this patch.
> >
> > As it turns out the EVBs have differing amounts of RAM; his has 1GB
> > while mine has 2GB. So we are not being inaccurate by setting 1GB as
> > the default here.
>
> That's convenient, means we don't have to figure out how to
> special-case the test infrastructure for it :-)
This is now OK on the 32-bit boxes, but still fails 'make check'
on my OSX system:
manooth$ QTEST_QEMU_BINARY=aarch64-softmmu/qemu-system-aarch64
tests/qom-test -p /aarch64/qom/ast2600-evb
/aarch64/qom/ast2600-evb: Broken pipe
/Users/pm215/src/qemu-for-merges/tests/libqtest.c:149: kill_qemu()
detected QEMU death from signal 6 (Abort trap: 6)
Abort trap: 6
Dropping from the pullreq again :-(
thanks
-- PMM