From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Jones Subject: Re: [PATCH 13/17] arm: initial drop Date: Mon, 3 Feb 2014 16:55:48 +0100 Message-ID: <20140203155548.GG6832@hawk.usersys.redhat.com> References: <1390321323-1855-1-git-send-email-drjones@redhat.com> <1390321323-1855-14-git-send-email-drjones@redhat.com> <20140202022826.GW3570@cbox> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: kvmarm@lists.cs.columbia.edu, kvm@vger.kernel.org To: Christoffer Dall Return-path: Received: from mx1.redhat.com ([209.132.183.28]:34670 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751999AbaBCP4E (ORCPT ); Mon, 3 Feb 2014 10:56:04 -0500 Content-Disposition: inline In-Reply-To: <20140202022826.GW3570@cbox> Sender: kvm-owner@vger.kernel.org List-ID: On Sat, Feb 01, 2014 at 06:28:26PM -0800, Christoffer Dall wrote: > On Tue, Jan 21, 2014 at 05:21:59PM +0100, Andrew Jones wrote: > > This is the initial arm test framework and a first simple test that > > checks some bootinfo. kvm isn't needed to run this test. This patch > > also adds a common build environment variable, $QEMU_BIN, which > > allows makefiles to call on qemu when needed. > > > > Try it out with > > yum install gcc-arm-linux-gnu dtc > > export QEMU=[qemu with mach-virt and virtio-testdev] > > ./configure --cross-prefix=arm-linux-gnu- --arch=arm > > make > > ./run_tests.sh > > > > Signed-off-by: Andrew Jones > > > > --- > > [...] > > > diff --git a/lib/arm/setup.c b/lib/arm/setup.c > > new file mode 100644 > > index 0000000000000..1db249a1eb94c > > --- /dev/null > > +++ b/lib/arm/setup.c > > @@ -0,0 +1,46 @@ > > +#include "libcflat.h" > > +#include "devicetree.h" > > +#include "arm/sysinfo.h" > > +#include "heap.h" > > + > > +extern void io_init(void); > > +extern void setup_args(char *args); > > + > > +extern unsigned long stacktop; > > + > > +void *mem_start; > > +size_t mem_size; > > +char *bootargs; > > const char *bootargs? ok > > > + > > +static void read_bootinfo(const void *fdt) > > +{ > > + int ret; > > + > > [...] > > Reviewed-by: Christoffer Dall