mbox series

[v2,0/2] fuzz: add virtio-blk fuzz target

Message ID cover.1604920905.git.dimastep@yandex-team.ru
Headers show
Series fuzz: add virtio-blk fuzz target | expand

Message

Dima Stepanov Nov. 9, 2020, 11:25 a.m. UTC 
v1 -> v2:
  - Update meson build with the CONFIG_VIRTIO_BLK case
  - Add "Reviewed-by: Alexander Bulekov" for both patches

Based on the current virtio-scsi-fuzz target implement new virtio-blk
fuzz target. Use a virtio_blk_test.c file as a reference for the block
device initialization.

Also make a small update to the documentation to fix build/run command
lines after meson and build changes in QEMU.

Dima Stepanov (2):
  fuzz: add virtio-blk fuzz target
  docs/fuzz: update make and run command lines

 docs/devel/fuzzing.txt             |   6 +-
 tests/qtest/fuzz/meson.build       |   1 +
 tests/qtest/fuzz/virtio_blk_fuzz.c | 234 +++++++++++++++++++++++++++++++++++++
 3 files changed, 238 insertions(+), 3 deletions(-)
 create mode 100644 tests/qtest/fuzz/virtio_blk_fuzz.c

Comments

Dima Stepanov Nov. 13, 2020, 10:10 a.m. UTC | #1 
On Mon, Nov 09, 2020 at 01:24:20PM +0100, Thomas Huth wrote:
> On 09/11/2020 12.25, Dima Stepanov wrote:

> > The virtio-blk fuzz target sets up and fuzzes the available virtio-blk

> > queues. The implementation is based on two files:

> >   - tests/qtest/fuzz/virtio_scsi_fuzz.c

> >   - tests/qtest/virtio_blk_test.c

> > 

> > Signed-off-by: Dima Stepanov <dimastep@yandex-team.ru>

> > Reviewed-by: Alexander Bulekov <alxndr@bu.edu>

> > ---

> >  tests/qtest/fuzz/meson.build       |   1 +

> >  tests/qtest/fuzz/virtio_blk_fuzz.c | 234 +++++++++++++++++++++++++++++++++++++

> >  2 files changed, 235 insertions(+)

> >  create mode 100644 tests/qtest/fuzz/virtio_blk_fuzz.c

> 

> Thanks, I can add this to my next qtest-related pull request.

> Note: I just spotted your patch by accident ... please put me on CC: if you

> want me to merge qtest/fuzzer related patches.


Hi Thomas,

Thanks for handling it. Got it! Will add you to CC for the next fuzzing
pull requests.

Dima.

> 

>  Thomas

>