mbox series

[v4,0/2] hw: Remove dynamic field width from trace events

Message ID 20191118222746.31467-1-philmd@redhat.com (mailing list archive)
Headers show
Series hw: Remove dynamic field width from trace events | expand

Message

Philippe Mathieu-Daudé Nov. 18, 2019, 10:27 p.m. UTC
This series fixes LP#1844817 [2].

(Eric noted in [1] the dtrace via stap backend can not support
the dynamic '*' width format.)

If they are trivial/block/tracing pull in preparation, this
series will be happy to be taken, else it will go via mips-next.

Thanks,

Phil.

[1] https://lists.gnu.org/archive/html/qemu-devel/2019-09/msg04720.html
[2] https://bugs.launchpad.net/qemu/+bug/1844817

Philippe Mathieu-Daudé (2):
  hw/block/pflash: Remove dynamic field width from trace events
  hw/mips/gt64xxx: Remove dynamic field width from trace events

 hw/block/pflash_cfi01.c |  8 ++++----
 hw/block/pflash_cfi02.c |  8 ++++----
 hw/mips/gt64xxx_pci.c   | 16 ++++++++--------
 hw/block/trace-events   |  8 ++++----
 hw/mips/trace-events    |  4 ++--
 5 files changed, 22 insertions(+), 22 deletions(-)

Comments

Philippe Mathieu-Daudé Nov. 18, 2019, 10:29 p.m. UTC | #1
This is material for 4.2-rc2.

On 11/18/19 11:27 PM, Philippe Mathieu-Daudé wrote:
> This series fixes LP#1844817 [2].
> 
> (Eric noted in [1] the dtrace via stap backend can not support
> the dynamic '*' width format.)
> 
> If they are trivial/block/tracing pull in preparation, this
> series will be happy to be taken, else it will go via mips-next.
> 
> Thanks,
> 
> Phil.
> 
> [1] https://lists.gnu.org/archive/html/qemu-devel/2019-09/msg04720.html
> [2] https://bugs.launchpad.net/qemu/+bug/1844817
Stefan Hajnoczi Nov. 19, 2019, 4:21 p.m. UTC | #2
On Mon, Nov 18, 2019 at 11:27:44PM +0100, Philippe Mathieu-Daudé wrote:
> This series fixes LP#1844817 [2].
> 
> (Eric noted in [1] the dtrace via stap backend can not support
> the dynamic '*' width format.)
> 
> If they are trivial/block/tracing pull in preparation, this
> series will be happy to be taken, else it will go via mips-next.
> 
> Thanks,
> 
> Phil.
> 
> [1] https://lists.gnu.org/archive/html/qemu-devel/2019-09/msg04720.html
> [2] https://bugs.launchpad.net/qemu/+bug/1844817
> 
> Philippe Mathieu-Daudé (2):
>   hw/block/pflash: Remove dynamic field width from trace events
>   hw/mips/gt64xxx: Remove dynamic field width from trace events
> 
>  hw/block/pflash_cfi01.c |  8 ++++----
>  hw/block/pflash_cfi02.c |  8 ++++----
>  hw/mips/gt64xxx_pci.c   | 16 ++++++++--------
>  hw/block/trace-events   |  8 ++++----
>  hw/mips/trace-events    |  4 ++--
>  5 files changed, 22 insertions(+), 22 deletions(-)
> 
> -- 
> 2.21.0
> 

Will send a pull request today.

Thanks, applied to my tracing tree:
https://github.com/stefanha/qemu/commits/tracing

Stefan
Philippe Mathieu-Daudé Nov. 19, 2019, 4:24 p.m. UTC | #3
On 11/19/19 5:21 PM, Stefan Hajnoczi wrote:
> On Mon, Nov 18, 2019 at 11:27:44PM +0100, Philippe Mathieu-Daudé wrote:
>> This series fixes LP#1844817 [2].
>>
>> (Eric noted in [1] the dtrace via stap backend can not support
>> the dynamic '*' width format.)
>>
>> If they are trivial/block/tracing pull in preparation, this
>> series will be happy to be taken, else it will go via mips-next.
>>
>> Thanks,
>>
>> Phil.
>>
>> [1] https://lists.gnu.org/archive/html/qemu-devel/2019-09/msg04720.html
>> [2] https://bugs.launchpad.net/qemu/+bug/1844817
>>
>> Philippe Mathieu-Daudé (2):
>>    hw/block/pflash: Remove dynamic field width from trace events
>>    hw/mips/gt64xxx: Remove dynamic field width from trace events
>>
>>   hw/block/pflash_cfi01.c |  8 ++++----
>>   hw/block/pflash_cfi02.c |  8 ++++----
>>   hw/mips/gt64xxx_pci.c   | 16 ++++++++--------
>>   hw/block/trace-events   |  8 ++++----
>>   hw/mips/trace-events    |  4 ++--
>>   5 files changed, 22 insertions(+), 22 deletions(-)
>>
>> -- 
>> 2.21.0
>>
> 
> Will send a pull request today.

Oh thanks Stefan, but it was getting late for the RC tag so I sent a 
pull for these patches via mips-next a pair of hours ago:
https://lists.gnu.org/archive/html/qemu-devel/2019-11/msg02894.html