Re: [PATCH 0/3] hv: vmbus: add fuzz testing to hv devices

2019-08-02 Thread Branden Bonaby
On Fri, Aug 02, 2019 at 09:30:18AM +0200, Vitaly Kuznetsov wrote:
> Branden Bonaby  writes:
> 
> > This patchset introduces a testing framework for Hyper-V drivers.
> > This framework allows us to introduce delays in the packet receive
> > path on a per-device basis. While the current code only supports 
> > introducing arbitrary delays in the host/guest communication path,
> > we intend to expand this to support error injection in the future.
> >
> > Branden Bonaby (3):
> >   drivers: hv: vmbus: Introduce latency testing
> >   drivers: hv: vmbus: add fuzz test attributes to sysfs
> >   tools: hv: add vmbus testing tool
> >
> >  Documentation/ABI/stable/sysfs-bus-vmbus |  22 ++
> >  drivers/hv/connection.c  |   5 +
> >  drivers/hv/ring_buffer.c |  10 +
> >  drivers/hv/vmbus_drv.c   |  97 ++-
> 
> Can we have something like CONFIG_HYPERV_TESTING and put this new 
> code under #ifdef?
> 
> >  include/linux/hyperv.h   |  14 +
> >  tools/hv/vmbus_testing   | 326 +++
> >  6 files changed, 473 insertions(+), 1 deletion(-)
> >  create mode 100644 tools/hv/vmbus_testing
> 
> -- 
> Vitaly

You're right, it would be better to do it that way with ifdef's.
Will edit my patches and resend.

Branden Bonaby


Re: [PATCH 0/3] hv: vmbus: add fuzz testing to hv devices

2019-08-02 Thread Vitaly Kuznetsov
Branden Bonaby  writes:

> This patchset introduces a testing framework for Hyper-V drivers.
> This framework allows us to introduce delays in the packet receive
> path on a per-device basis. While the current code only supports 
> introducing arbitrary delays in the host/guest communication path,
> we intend to expand this to support error injection in the future.
>
> Branden Bonaby (3):
>   drivers: hv: vmbus: Introduce latency testing
>   drivers: hv: vmbus: add fuzz test attributes to sysfs
>   tools: hv: add vmbus testing tool
>
>  Documentation/ABI/stable/sysfs-bus-vmbus |  22 ++
>  drivers/hv/connection.c  |   5 +
>  drivers/hv/ring_buffer.c |  10 +
>  drivers/hv/vmbus_drv.c   |  97 ++-

Can we have something like CONFIG_HYPERV_TESTING and put this new 
code under #ifdef?

>  include/linux/hyperv.h   |  14 +
>  tools/hv/vmbus_testing   | 326 +++
>  6 files changed, 473 insertions(+), 1 deletion(-)
>  create mode 100644 tools/hv/vmbus_testing

-- 
Vitaly


[PATCH 0/3] hv: vmbus: add fuzz testing to hv devices

2019-08-01 Thread Branden Bonaby
This patchset introduces a testing framework for Hyper-V drivers.
This framework allows us to introduce delays in the packet receive
path on a per-device basis. While the current code only supports 
introducing arbitrary delays in the host/guest communication path,
we intend to expand this to support error injection in the future.

Branden Bonaby (3):
  drivers: hv: vmbus: Introduce latency testing
  drivers: hv: vmbus: add fuzz test attributes to sysfs
  tools: hv: add vmbus testing tool

 Documentation/ABI/stable/sysfs-bus-vmbus |  22 ++
 drivers/hv/connection.c  |   5 +
 drivers/hv/ring_buffer.c |  10 +
 drivers/hv/vmbus_drv.c   |  97 ++-
 include/linux/hyperv.h   |  14 +
 tools/hv/vmbus_testing   | 326 +++
 6 files changed, 473 insertions(+), 1 deletion(-)
 create mode 100644 tools/hv/vmbus_testing

-- 
2.17.1