[PATCH 0/7] some cleanups around CPU related code

Daniel Henrique Barboza posted 7 patches 2 weeks, 3 days ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20211118135944.272979-1-danielhb413@gmail.com
src/conf/domain_conf.h       |   1 +
src/cpu/cpu_ppc64.c          |  26 +-------
src/cpu/cpu_x86.c            |  38 +-----------
src/qemu/qemu_capabilities.c | 116 +++++++++++++++--------------------
src/qemu/qemu_process.c      |  15 ++---
5 files changed, 57 insertions(+), 139 deletions(-)

[PATCH 0/7] some cleanups around CPU related code

Posted by Daniel Henrique Barboza 2 weeks, 3 days ago
Hi,

Here are some cleanups I ended up making while investigating a
test issue with ppc64 hosts recently.

Daniel Henrique Barboza (7):
  qemu_capabilities.c: use g_autofree in virQEMUCapsLoadCache()
  cpu_x86.c: remove 'guest' param from x86Compute()
  cpu_ppc64.c: remove 'guest' param from ppc64Compute()
  qemu_capabilities.c: use g_autofree in virQEMUCapsLoadHostCPUModelInfo
  domain_conf.h: add autoptr cleanup func to virDomainXMLOptionPtr
  qemu_process.c: use g_autoptr() in qemuProcessQMPInitMonitor
  qemu_capabilities.c: use g_autoptr() in virQEMUCapsInitHostCPUModel()

 src/conf/domain_conf.h       |   1 +
 src/cpu/cpu_ppc64.c          |  26 +-------
 src/cpu/cpu_x86.c            |  38 +-----------
 src/qemu/qemu_capabilities.c | 116 +++++++++++++++--------------------
 src/qemu/qemu_process.c      |  15 ++---
 5 files changed, 57 insertions(+), 139 deletions(-)

-- 
2.31.1

Re: [PATCH 0/7] some cleanups around CPU related code

Posted by Daniel Henrique Barboza 2 weeks, 3 days ago
I'll resend this series because I somehow messed with my own email in the CC.
Sorry for the span.



Daniel

On 11/18/21 10:59, Daniel Henrique Barboza wrote:
> Hi,
> 
> Here are some cleanups I ended up making while investigating a
> test issue with ppc64 hosts recently.
> 
> Daniel Henrique Barboza (7):
>    qemu_capabilities.c: use g_autofree in virQEMUCapsLoadCache()
>    cpu_x86.c: remove 'guest' param from x86Compute()
>    cpu_ppc64.c: remove 'guest' param from ppc64Compute()
>    qemu_capabilities.c: use g_autofree in virQEMUCapsLoadHostCPUModelInfo
>    domain_conf.h: add autoptr cleanup func to virDomainXMLOptionPtr
>    qemu_process.c: use g_autoptr() in qemuProcessQMPInitMonitor
>    qemu_capabilities.c: use g_autoptr() in virQEMUCapsInitHostCPUModel()
> 
>   src/conf/domain_conf.h       |   1 +
>   src/cpu/cpu_ppc64.c          |  26 +-------
>   src/cpu/cpu_x86.c            |  38 +-----------
>   src/qemu/qemu_capabilities.c | 116 +++++++++++++++--------------------
>   src/qemu/qemu_process.c      |  15 ++---
>   5 files changed, 57 insertions(+), 139 deletions(-)
> 

Re: [PATCH 0/7] some cleanups around CPU related code

Posted by Peter Krempa 2 weeks, 3 days ago
On Thu, Nov 18, 2021 at 11:20:11 -0300, Daniel Henrique Barboza wrote:
> I'll resend this series because I somehow messed with my own email in the CC.
> Sorry for the span.

Is that really necessary? You'll still get the replies from the mailing
list.

Re: [PATCH 0/7] some cleanups around CPU related code

Posted by Daniel Henrique Barboza 2 weeks, 3 days ago

On 11/18/21 11:21, Peter Krempa wrote:
> On Thu, Nov 18, 2021 at 11:20:11 -0300, Daniel Henrique Barboza wrote:
>> I'll resend this series because I somehow messed with my own email in the CC.
>> Sorry for the span.
> 
> Is that really necessary? You'll still get the replies from the mailing
> list.
> 

I wanted to avoid the people using "reply-all" and getting hit with "recipient
does not exist" messages because the email didn't exist. There's also the
possibility that the misspelt email actually exists and you're messaging
someone else.


Daniel