[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v7 2/2] Add QTest testcase for the Intel Hexadec

From: Stefan Hajnoczi
Subject: Re: [Qemu-devel] [PATCH v7 2/2] Add QTest testcase for the Intel Hexadecimal Object File Loader.
Date: Mon, 14 May 2018 15:14:31 +0100
User-agent: Mutt/1.9.5 (2018-04-13)

On Mon, May 14, 2018 at 08:57:48AM +0200, Steffen Görtz wrote:
> On 10.05.2018 09:18, Su Hang wrote:
> > +
> > +    QTestState *s = qtest_startf(
> > +        "-M versatilepb -m 128M -nographic -kernel 
> > ../tests/hex-loader-check-data/test.hex");
> > +
> The test binary "text.hex" is not being copied to out-of-tree builds at
> the moment and the test run will therefore fail.
> The acpi tests also use a fixture (acpi-test-data) which is being copied
> to the out-of-tree build in ./configure:L7181 .
> This might also be an option for your test.

To explain how out-of-tree builds work for anyone who hasn't seen them

  $ mkdir /var/tmp/build-dir
  $ cd /var/tmp/build-dir
  $ ~/qemu/configure ... && make

The .o files and other build output will be located in
/var/tmp/build-dir (away from the source code in ~/qemu/).

The build system must ensure that data files are available to the test
program in /var/tmp/build-dir/ if an out-of-tree build is being done.
In the acpi-test-data case that Steffen mentioned the ./configure script
sets up symlinks:

  for test_file in $(find $source_path/tests/acpi-test-data -type f)
      FILES="$FILES tests/acpi-test-data$(echo $test_file | sed -e 
  symlink "$source_path/$f" "$f"

Please add hex-loader-check-data in the same way.


Attachment: signature.asc
Description: PGP signature

reply via email to

[Prev in Thread] Current Thread [Next in Thread]