[libvirt] [PATCH 1/7] cpu: Add support for loading and storing CPU data
Jiri Denemark
jdenemar at redhat.com
Tue Jul 23 16:50:52 UTC 2013
On Tue, Jul 23, 2013 at 17:27:41 +0100, Daniel Berrange wrote:
> On Tue, Jul 23, 2013 at 06:11:30PM +0200, Jiri Denemark wrote:
>
> Could use a more verbose commit message.
>
> Looking at later patches, I see cpuDataFormat used in the
> test suite, but I don't see cpuDataParse used anywhere ?
> If these are only for the test suite, then perhaps adding
> them to a cpupriv.h header is preferrable.
Yeah, cpupriv.h might be better since both APIs are really there just
for our testsuite. Although I guess marking them as testsuite only in
cpu.h would work too. And you're right, cpuDataParse is not used
anywhere but I plan to use it in other cpu related tests in the future
and having them both in a single patch seemed best to me.
Jirka
More information about the libvir-list
mailing list