[Top][All Lists]

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

Re: [Qemu-devel] Re: [PATCH] pci: initialize header type register.

From: Markus Armbruster
Subject: Re: [Qemu-devel] Re: [PATCH] pci: initialize header type register.
Date: Tue, 09 Feb 2010 09:21:44 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/23.1 (gnu/linux)

Gerd Hoffmann <address@hidden> writes:

>   Hi,
>> This still means we have two copies of same data
>> and need to maintain code that keeps them in sync,
>> even if that is called just at init time.
> No.  There is nothing to keep in sync.  And there is no extra copy of data.
> Today you have pci_set_*() calls somewhere in PCIDeviceInfo->init().
> I'd like to see them replaced with PCIDeviceInfo->$field + setup in
> common code.  The information that device $foo has vendor id 42 and
> device id 4711 (and other properties) just moves from code to data.
> It is static information, it should be static data.  And having the
> information in a well defined place in a data structure instead of
> hidden somewhere in the ->init() code makes it alot easier to reuse
> the information for something else.  That is the whole point.


reply via email to

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