On Tue, 16 Feb 2021 at 17:31, Philippe Mathieu-Daudé <phi...@redhat.com> wrote: > > On 2/16/21 4:00 PM, Alexander Bulekov wrote: > > On 210216 1325, Peter Maydell wrote: > >> Hi; I just noticed this, but why does this test case pass > >> "-trace pci*" to QEMU? It doesn't look like it does anything to > >> try to capture the trace output, which thus winds up just going > >> to the stdout/stderr of the "make check" run. Similarly I'm not > >> sure '-d guest_errors' is going to do anything helpful unless > >> you take steps to capture the output and check it. > > I suppose Alex took it from commit 4177b062fc5 ("hw/isa/lpc_ich9: > Ignore reserved/invalid SCI IRQ") where it is used to show the > problem the fuzzer reproducer triggers. Not useful in regular testing. > > > Ah, I sometimes include those in the initial report, if the device has > > trace-events. I can remove this from fuzz-test.c, if it is slowing > > things down. > > I doubt it is slowing things down, but it probably make it harder to > find other problems (I suppose Peter got confused when looking for > another failure and found these traces).
Not even that; it just looks a bit weird in the 'make check' output. If there's no particular reason for them to be in there we might as well remove them. -- PMM