From: Juan Quintela <quintela@redhat.com>
In-Reply-To: <4A8E61D1.80003@redhat.com> (Gerd Hoffmann's message of "Fri, 21
	Aug 2009 10:58:57 +0200")
References: <cover.1250788880.git.quintela@redhat.com>
	<4A8E61D1.80003@redhat.com>
Date: Fri, 21 Aug 2009 11:12:14 +0200
Message-ID: <m3my5twmxd.fsf@neno.mitica>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Subject: [Qemu-devel] Re: [PATCH 00/23] New VMState table based load/save
	infrastructure
To: Gerd Hoffmann <kraxel@redhat.com>
Cc: qemu-devel@nongnu.org

Gerd Hoffmann <kraxel@redhat.com> wrote:
> On 08/20/09 19:42, Juan Quintela wrote:
>> - i440fx_load()
>>    Can we put pci_irq_levels[] array in any struct?  This is the only driver
>>    that I have found that uses a global variable.  Gerd, can you take a look?
>>    It is not possible to have the PCIDevice struct inside a PIIXDevice or similar?
>
> Stuffing a pointer to pci_irq_levels into a struct should work without
> problems and should be good enougth to get vmstate save/load going ;)
>
> Sticking the array itself into a struct would be even better, that
> might need more cleanup work though.

Do you mean to do any of the tow approachs?
I promise to study your patch for other places that I need :)

Later, Juan.