[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/2] s390x/ccs: add ccw-tester emulated device
From: |
Halil Pasic |
Subject: |
Re: [Qemu-devel] [PATCH 1/2] s390x/ccs: add ccw-tester emulated device |
Date: |
Mon, 25 Sep 2017 17:06:01 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 |
On 09/19/2017 04:20 PM, Pierre Morel wrote:
> On 15/09/2017 19:01, Cornelia Huck wrote:
>> On Fri, 15 Sep 2017 09:27:58 +0200
>> Cornelia Huck <address@hidden> wrote:
>>
>>> On Thu, 14 Sep 2017 18:50:29 +0200
>>> Halil Pasic <address@hidden> wrote:
>>>
>>>> On 09/14/2017 04:26 PM, Cornelia Huck wrote:
>>
>>>>> There's quite some potential in this. We may want to make this a
>>>>> permanent addition.
>>>>>
>>>>
>>>> I'm happy to contribute! I'm not sure how shall we proceed though.
>>>> Maybe with making a todo list?
>>>
>>> I think the first step would be to figure out the ids so we don't step
>>> on anyone's toes. Then maybe refactor a bit so that other testers can
>>> be added easily.
>>>
>>> For ideas about things to be tested, maybe put a list into the wiki?
>>
>> OK, I've created https://wiki.qemu.org/Testing/CCWTestDevice - let me
>> know if you need an account.
>>
>
> Great!
> I just saw your email, it was waiting in my inbox.
> strange things happens some time. A good serie. strange things.
> neverless...
>
> Seriously.
> Yes, I definitively need an account!
>
>
> Pierre
>
@Connie
I would also like to have an account. I would also like to do a
v2 of this somewhere in the not too distant future. I intend
to address the issues pointed out by you here (chpid_type,
cu_type). Another question is whether this should go under
hw/misc/ like the pci-testdev? ccw-testdev would also probably
be a better file name (that ccw-tester) regardless of in which
folder does this belong. One more thing I am considering for
v2 is this make it extensible argument. I had something like
mode on my mind from the very beginning (fib would be one mode).
The idea was to provide a control ccw for setting/getting the mode
(something like virtio) as well as a device property for setting
the initial mode (so that guest does not have to know about it).
Regards,
Halil
- [Qemu-devel] [PATCH 0/2] tests for CCW IDA, Halil Pasic, 2017/09/13
- [Qemu-devel] [PATCH 1/2] s390x/ccs: add ccw-tester emulated device, Halil Pasic, 2017/09/13
- Re: [Qemu-devel] [PATCH 1/2] s390x/ccs: add ccw-tester emulated device, Cornelia Huck, 2017/09/14
- Re: [Qemu-devel] [PATCH 1/2] s390x/ccs: add ccw-tester emulated device, Halil Pasic, 2017/09/14
- Re: [Qemu-devel] [PATCH 1/2] s390x/ccs: add ccw-tester emulated device, Cornelia Huck, 2017/09/15
- Re: [Qemu-devel] [PATCH 1/2] s390x/ccs: add ccw-tester emulated device, Cornelia Huck, 2017/09/15
- Re: [Qemu-devel] [PATCH 1/2] s390x/ccs: add ccw-tester emulated device, Pierre Morel, 2017/09/19
- Re: [Qemu-devel] [PATCH 1/2] s390x/ccs: add ccw-tester emulated device,
Halil Pasic <=
- Re: [Qemu-devel] [PATCH 1/2] s390x/ccs: add ccw-tester emulated device, Cornelia Huck, 2017/09/25
- Re: [Qemu-devel] [PATCH 1/2] s390x/ccs: add ccw-tester emulated device, Christian Borntraeger, 2017/09/18
- Re: [Qemu-devel] [PATCH 1/2] s390x/ccs: add ccw-tester emulated device, Cornelia Huck, 2017/09/18
- Re: [Qemu-devel] [PATCH 1/2] s390x/ccs: add ccw-tester emulated device, Pierre Morel, 2017/09/19
[Qemu-devel] [PATCH 2/2 NOT QEMU] a tester device for ccw I/O, Halil Pasic, 2017/09/13