Re: [Qemu-devel] [PATCH 5/5] s390x/ccs: add ccw-tester emulated device

2017-09-27 Thread Dong Jia Shi
* Dong Jia Shi [2017-09-26 15:48:56 +0800]: [...] > > > > > > Tried to test with the following method: > > > 1. Start g1 (first level guest on kvm a host) with a virtio blk device > > >defined: > > > -drive > > > file=/dev/disk/by-path/ccw-0.0.3f3e,if=none,id=drive-virtio-disk1,format=raw

Re: [Qemu-devel] [PATCH 5/5] s390x/ccs: add ccw-tester emulated device

2017-09-26 Thread Dong Jia Shi
* Cornelia Huck [2017-09-21 10:54:02 +0200]: > On Thu, 21 Sep 2017 16:45:47 +0800 > Dong Jia Shi wrote: > > > * Cornelia Huck [2017-09-07 10:08:17 +0200]: > > > > [...] > > > > > > I'm thinking of a method these days: > > > > Could passing through an fully emulated ccw device (e.g. 3270), or

Re: [Qemu-devel] [PATCH 5/5] s390x/ccs: add ccw-tester emulated device

2017-09-21 Thread Cornelia Huck
On Thu, 21 Sep 2017 16:45:47 +0800 Dong Jia Shi wrote: > * Cornelia Huck [2017-09-07 10:08:17 +0200]: > > [...] > > > > I'm thinking of a method these days: > > > Could passing through an fully emulated ccw device (e.g. 3270), or a > > > virtio ccw device, in the level 1 kvm guest to a level 2

Re: [Qemu-devel] [PATCH 5/5] s390x/ccs: add ccw-tester emulated device

2017-09-21 Thread Dong Jia Shi
* Cornelia Huck [2017-09-07 10:08:17 +0200]: [...] > > I'm thinking of a method these days: > > Could passing through an fully emulated ccw device (e.g. 3270), or a > > virtio ccw device, in the level 1 kvm guest to a level 2 guest be a test > > method for this? > > > > All of the CCWs will be

Re: [Qemu-devel] [PATCH 5/5] s390x/ccs: add ccw-tester emulated device

2017-09-18 Thread Dong Jia Shi
* Halil Pasic [2017-09-08 12:28:03 +0200]: [snip] > > What Halil pointed out is that the ccw_cb implementation of 3270 > > emulation does not take care of IDALs. This is true. > > > > I can also do that right after this series, if Halil agrees. > > (The 3270 emulation authors are busy of other

Re: [Qemu-devel] [PATCH 5/5] s390x/ccs: add ccw-tester emulated device

2017-09-08 Thread Halil Pasic
On 09/08/2017 04:01 AM, Dong Jia Shi wrote: > * Cornelia Huck [2017-09-07 12:52:20 +0200]: > >> On Thu, 7 Sep 2017 12:21:50 +0200 >> Halil Pasic wrote: >> >>> On 09/07/2017 10:08 AM, Cornelia Huck wrote: On Thu, 7 Sep 2017 15:31:09 +0800 Dong Jia Shi wrote: >> > I'm thinking of

Re: [Qemu-devel] [PATCH 5/5] s390x/ccs: add ccw-tester emulated device

2017-09-07 Thread Dong Jia Shi
* Cornelia Huck [2017-09-07 12:52:20 +0200]: > On Thu, 7 Sep 2017 12:21:50 +0200 > Halil Pasic wrote: > > > On 09/07/2017 10:08 AM, Cornelia Huck wrote: > > > On Thu, 7 Sep 2017 15:31:09 +0800 > > > Dong Jia Shi wrote: > > > >> I'm thinking of a method these days: > > >> Could passing through

Re: [Qemu-devel] [PATCH 5/5] s390x/ccs: add ccw-tester emulated device

2017-09-07 Thread Cornelia Huck
On Thu, 7 Sep 2017 11:10:17 +0200 Janosch Frank wrote: > The thing is that I want to test the subchannel and not the device and > therefore I really do not want to have to issue control commands to a > device in order to get data. Having a device that reads and writes > without a lot of overhead

Re: [Qemu-devel] [PATCH 5/5] s390x/ccs: add ccw-tester emulated device

2017-09-07 Thread Cornelia Huck
On Thu, 7 Sep 2017 12:21:50 +0200 Halil Pasic wrote: > On 09/07/2017 10:08 AM, Cornelia Huck wrote: > > On Thu, 7 Sep 2017 15:31:09 +0800 > > Dong Jia Shi wrote: > >> I'm thinking of a method these days: > >> Could passing through an fully emulated ccw device (e.g. 3270), or a > >> virtio ccw d

Re: [Qemu-devel] [PATCH 5/5] s390x/ccs: add ccw-tester emulated device

2017-09-07 Thread Halil Pasic
On 09/07/2017 10:08 AM, Cornelia Huck wrote: > On Thu, 7 Sep 2017 15:31:09 +0800 > Dong Jia Shi wrote: > >> * Cornelia Huck [2017-09-06 15:18:21 +0200]: >> >>> On Tue, 5 Sep 2017 13:16:45 +0200 >>> Halil Pasic wrote: >>> Add a fake device meant for testing the correctness of our css

Re: [Qemu-devel] [PATCH 5/5] s390x/ccs: add ccw-tester emulated device

2017-09-07 Thread Janosch Frank
[...] > The main problem here is that you want to exercise a middle layer (the > css code) and need to write boilerplate code on both host and guest > side in order to be able to do so. > Nod. > In general, a device that accepts arbitrary channel programs l

Re: [Qemu-devel] [PATCH 5/5] s390x/ccs: add ccw-tester emulated device

2017-09-07 Thread Cornelia Huck
On Wed, 6 Sep 2017 18:16:29 +0200 Halil Pasic wrote: > On 09/06/2017 05:20 PM, Cornelia Huck wrote: > > On Wed, 6 Sep 2017 16:24:13 +0200 > > Halil Pasic wrote: > > > >> On 09/06/2017 03:18 PM, Cornelia Huck wrote: > >>> On Tue, 5 Sep 2017 13:16:45 +0200 > >>> Halil Pasic wrote: > >>>

Re: [Qemu-devel] [PATCH 5/5] s390x/ccs: add ccw-tester emulated device

2017-09-07 Thread Cornelia Huck
On Thu, 7 Sep 2017 15:31:09 +0800 Dong Jia Shi wrote: > * Cornelia Huck [2017-09-06 15:18:21 +0200]: > > > On Tue, 5 Sep 2017 13:16:45 +0200 > > Halil Pasic wrote: > > > > > Add a fake device meant for testing the correctness of our css emulation. > > > > > > What we currently have is wri

Re: [Qemu-devel] [PATCH 5/5] s390x/ccs: add ccw-tester emulated device

2017-09-07 Thread Dong Jia Shi
* Cornelia Huck [2017-09-06 15:18:21 +0200]: > On Tue, 5 Sep 2017 13:16:45 +0200 > Halil Pasic wrote: > > > Add a fake device meant for testing the correctness of our css emulation. > > > > What we currently have is writing a Fibonacci sequence of uint32_t to the > > device via ccw write. The

Re: [Qemu-devel] [PATCH 5/5] s390x/ccs: add ccw-tester emulated device

2017-09-06 Thread Halil Pasic
On 09/06/2017 05:20 PM, Cornelia Huck wrote: > On Wed, 6 Sep 2017 16:24:13 +0200 > Halil Pasic wrote: > >> On 09/06/2017 03:18 PM, Cornelia Huck wrote: >>> On Tue, 5 Sep 2017 13:16:45 +0200 >>> Halil Pasic wrote: >>> Add a fake device meant for testing the correctness of our css emula

Re: [Qemu-devel] [PATCH 5/5] s390x/ccs: add ccw-tester emulated device

2017-09-06 Thread Cornelia Huck
On Wed, 6 Sep 2017 16:24:13 +0200 Halil Pasic wrote: > On 09/06/2017 03:18 PM, Cornelia Huck wrote: > > On Tue, 5 Sep 2017 13:16:45 +0200 > > Halil Pasic wrote: > > > >> Add a fake device meant for testing the correctness of our css emulation. > >> > >> What we currently have is writing a Fi

Re: [Qemu-devel] [PATCH 5/5] s390x/ccs: add ccw-tester emulated device

2017-09-06 Thread Halil Pasic
On 09/06/2017 03:18 PM, Cornelia Huck wrote: > On Tue, 5 Sep 2017 13:16:45 +0200 > Halil Pasic wrote: > >> Add a fake device meant for testing the correctness of our css emulation. >> >> What we currently have is writing a Fibonacci sequence of uint32_t to the >> device via ccw write. The writ

Re: [Qemu-devel] [PATCH 5/5] s390x/ccs: add ccw-tester emulated device

2017-09-06 Thread Cornelia Huck
On Tue, 5 Sep 2017 13:16:45 +0200 Halil Pasic wrote: > Add a fake device meant for testing the correctness of our css emulation. > > What we currently have is writing a Fibonacci sequence of uint32_t to the > device via ccw write. The write is going to fail if it ain't a Fibonacci > and indicat

[Qemu-devel] [PATCH 5/5] s390x/ccs: add ccw-tester emulated device

2017-09-05 Thread Halil Pasic
Add a fake device meant for testing the correctness of our css emulation. What we currently have is writing a Fibonacci sequence of uint32_t to the device via ccw write. The write is going to fail if it ain't a Fibonacci and indicate a device exception in scsw together with the proper residual cou