[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PULL v2 01/16] tests/9pfs: fix test dir for parallel tests
From: |
Christian Schoenebeck |
Subject: |
Re: [PULL v2 01/16] tests/9pfs: fix test dir for parallel tests |
Date: |
Sat, 31 Oct 2020 22:15:23 +0100 |
On Samstag, 31. Oktober 2020 21:34:31 CET Peter Maydell wrote:
> On Sat, 31 Oct 2020 at 13:20, Christian Schoenebeck
>
> <qemu_oss@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
- [PULL v2 00/16] 9p queue (previous 2020-10-23), Christian Schoenebeck, 2020/10/30
- [PULL v2 02/16] tests/9pfs: fix coverity error in create_local_test_dir(), Christian Schoenebeck, 2020/10/30
- [PULL v2 03/16] tests/9pfs: Force removing of local 9pfs test directory, Christian Schoenebeck, 2020/10/30
- [PULL v2 04/16] tests/9pfs: Factor out do_version() helper, Christian Schoenebeck, 2020/10/30
- [PULL v2 05/16] tests/9pfs: Set alloc in fs_create_dir(), Christian Schoenebeck, 2020/10/30
- [PULL v2 06/16] tests/9pfs: Factor out do_attach() helper, Christian Schoenebeck, 2020/10/30
- [PULL v2 07/16] tests/9pfs: Turn fs_readdir_split() into a helper, Christian Schoenebeck, 2020/10/30
- [PULL v2 08/16] tests/9pfs: Turn fs_mkdir() into a helper, Christian Schoenebeck, 2020/10/30
- [PULL v2 09/16] tests/9pfs: simplify do_mkdir(), Christian Schoenebeck, 2020/10/30
- [PULL v2 10/16] tests/9pfs: add local Tunlinkat directory test, Christian Schoenebeck, 2020/10/30
- [PULL v2 11/16] tests/9pfs: add local Tlcreate test, Christian Schoenebeck, 2020/10/30