On Samstag, 31. Oktober 2020 21:34:31 CET Peter Maydell wrote: > On Sat, 31 Oct 2020 at 13:20, Christian Schoenebeck > > <qemu_...@crudebyte.com> wrote: > > On Freitag, 30. Oktober 2020 13:07:03 CET Christian Schoenebeck wrote: > > > Use mkdtemp() to generate a unique directory for the 9p 'local' tests. > > > > > > This fixes occasional 9p test failures when running 'make check -jN' if > > > QEMU was compiled for multiple target architectures, because the > > > individual > > > architecture's test suites would run in parallel and interfere with each > > > other's data as the test directory was previously hard coded and hence > > > the > > > same directory was used by all of them simultaniously. > > > > > > This also requires a change how the test directory is created and > > > deleted: > > > As the test path is now randomized and virtio_9p_register_nodes() being > > > called in a somewhat undeterministic way, that's no longer an > > > appropriate > > > place to create and remove the test directory. Use a constructor and > > > destructor function for creating and removing the test directory > > > instead. > > > Unfortunately libqos currently does not support setup/teardown callbacks > > > to handle this more cleanly. > > > > Peter, please ignore this PR. This patch needs rework: > OK. As a general rule you need to make "please drop this PR" > requests as replies to the top level cover letter, though -- > otherwise it's pot luck whether I happen to notice them or not. > > thanks > -- PMM
Okay, got it. Best regards, Christian Schoenebeck