[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/7] s390: sclp base support
From: |
Christian Borntraeger |
Subject: |
Re: [Qemu-devel] [PATCH 3/7] s390: sclp base support |
Date: |
Mon, 30 Jul 2012 16:34:38 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux i686; rv:14.0) Gecko/20120714 Thunderbird/14.0 |
On 30/07/12 14:38, Alexander Graf wrote:
>> +/* There is one SCLP bus per machine */
>> +static SCLPS390Bus *sbus;
>
> ... but there isn't necessarily one machine per qemu instance. Today there
> is,
> but we shouldn't rely on that fact. Please move the bus variable into a
> machine
> struct that only the machine knows about.
Could do. However, we have the same issue with the virtio bus. Would it make
sense to leave it
here as is and the do a cleanup later on?
[...]
>> +void sclp_service_interrupt(uint32_t sccb)
>
> Does this have to be globally visible? If all the sclp source ends up in this
> file, it should only be necessary internal to it, right?
It will be called by the console and quiesce code
>
>> +{
>> + if (!sccb) {
>
> Please add a comment when this would trigger. In fact, I'm not sure I fully
> understand the reason for this function :).
It will be enhanced by patch 4. Will move that hunk completely there. See also
the explanation for patch 4.
- Re: [Qemu-devel] [PATCH 4/7] s390: sclp event support, (continued)
[Qemu-devel] [PATCH 6/7] s390: sclp ascii console support, Christian Borntraeger, 2012/07/24
[Qemu-devel] [PATCH 5/7] s390: sclp signal quiesce support, Christian Borntraeger, 2012/07/24
[Qemu-devel] [PATCH 3/7] s390: sclp base support, Christian Borntraeger, 2012/07/24
[Qemu-devel] [PATCH 1/7] s390: Fix error handling and condition code of service call, Christian Borntraeger, 2012/07/24
[Qemu-devel] [PATCH 2/7] s390: provide interface for service interrupt/introduce interrupt.c, Christian Borntraeger, 2012/07/24
Re: [Qemu-devel] [PATCH 0/7v3] s390: sclp patch set, Christian Borntraeger, 2012/07/30