vl.c: fix trace backend init ordering

[PATCH v2 0/1] vl.c: fix trace backend init ordering

Daniel Henrique Barboza posted 1 patch 4 years, 10 months ago

Download series mbox

Test checkpatch passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20210105181437.538366-1-danielhb413@gmail.com

Maintainers: Paolo Bonzini <pbonzini@redhat.com>

softmmu/vl.c | 18 +++++++++++++-----
1 file changed, 13 insertions(+), 5 deletions(-)

Expand all Fold all

[PATCH v2 0/1] vl.c: fix trace backend init ordering

Posted by Daniel Henrique Barboza 4 years, 10 months ago

changes from v2:
- fixed a typo in the comment block
- added Paolo's R-b


Daniel Henrique Barboza (1):
  vl.c: do not execute trace_init_backends() before daemonizing

 softmmu/vl.c | 18 +++++++++++++-----
 1 file changed, 13 insertions(+), 5 deletions(-)

-- 
2.26.2

Re: [PATCH v2 0/1] vl.c: fix trace backend init ordering

Posted by Daniel Henrique Barboza 4 years, 8 months ago

Ping

There is at least one user in the Libvirt ML experiencing the same problem
this patch is fixing, using QEMU upstream compiling with the trace backend,
and it seems like the daemonization is still not working.

Thanks,

DHB

On 1/5/21 3:14 PM, Daniel Henrique Barboza wrote:
> changes from v2:
> - fixed a typo in the comment block
> - added Paolo's R-b
> 
> 
> Daniel Henrique Barboza (1):
>    vl.c: do not execute trace_init_backends() before daemonizing
> 
>   softmmu/vl.c | 18 +++++++++++++-----
>   1 file changed, 13 insertions(+), 5 deletions(-)
>

Re: [PATCH v2 0/1] vl.c: fix trace backend init ordering

Posted by Paolo Bonzini 4 years, 8 months ago

On 02/03/21 11:51, Daniel Henrique Barboza wrote:
> Ping
> 
> There is at least one user in the Libvirt ML experiencing the same problem
> this patch is fixing, using QEMU upstream compiling with the trace backend,
> and it seems like the daemonization is still not working.
> 
> 
> Thanks,
> 
> DHB
> 
> On 1/5/21 3:14 PM, Daniel Henrique Barboza wrote:
>> changes from v2:
>> - fixed a typo in the comment block
>> - added Paolo's R-b
>>
>>
>> Daniel Henrique Barboza (1):
>>    vl.c: do not execute trace_init_backends() before daemonizing
>>
>>   softmmu/vl.c | 18 +++++++++++++-----
>>   1 file changed, 13 insertions(+), 5 deletions(-)
>>
> 

Sorry, I was expecting Stefan to apply this patch.  I have now queued it.

Paolo