qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 04/13] pseries: savevm support for XICS interrup


From: Alexey Kardashevskiy
Subject: Re: [Qemu-devel] [PATCH 04/13] pseries: savevm support for XICS interrupt controller
Date: Wed, 05 Jun 2013 21:17:21 +1000
User-agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:17.0) Gecko/20130328 Thunderbird/17.0.5

On 05.06.2013 19:55, David Gibson wrote:
> On Wed, Jun 05, 2013 at 01:22:02PM +1000, Alexey Kardashevskiy wrote:
>> On 06/04/2013 10:20 PM, Alexey Kardashevskiy wrote:
> [snip]
>>> +static int ics_post_load(void *opaque, int version_id)
>>> +{
>>> +    int i;
>>> +    struct ics_state *ics = opaque;
>>> +
>>> +    for (i = 0; i < ics->nr_irqs; i++) {
>>
>> Obviously here is a bug...
>>
>> -    for (i = 0; i < ics->nr_irqs; i++) {
>> +    for (i = 0; i < ics->icp->nr_servers; i++) {
> 
> Ah.. yes.  Although it would be a little nicer to avoid the duplicate
> calls to ics_resend() that will cause.  Still the simple version
> should be correct for now.  Please resend with this fixed.

I'll wait till Friday (do we expect anyone to comment, besides you?) and
repost the whole series with fixed "from" :)


-- 
With best regards

Alexey Kardashevskiy -- icq: 52150396



reply via email to

[Prev in Thread] Current Thread [Next in Thread]