mbox series

[0/3] plugins/execlog: add data address match and address range support

Message ID 20240229150729.1620410-1-svens@stackframe.org (mailing list archive)
Headers show
Series plugins/execlog: add data address match and address range support | expand

Message

Sven Schnelle Feb. 29, 2024, 3:07 p.m. UTC
Hi List,

this patchset adds a new -dfilter option and address range matching. With this
execlog can match only a certain range of address for both instruction and
data adresses.

Example usage:

qemu-system-xxx <other options> -d plugin -plugin libexeclog.so,afilter=0x1000-0x2000,dfilter=0x388

This would only log instruction in the address range 0x1000 to 0x2000
and accessing data at address 0x388.

Changes in v2:
- rebased on top of latest master

Sven Schnelle (3):
  plugins/execlog: pass matches array to parse_vaddr_match
  plugins/execlog: add data address match
  plugins/execlog: add address range matching

 contrib/plugins/execlog.c | 95 ++++++++++++++++++++++++++++++++-------
 1 file changed, 79 insertions(+), 16 deletions(-)

Comments

Alex Bennée Feb. 29, 2024, 5:08 p.m. UTC | #1
Sven Schnelle <svens@stackframe.org> writes:

> Hi List,
>
> this patchset adds a new -dfilter option and address range matching. With this
> execlog can match only a certain range of address for both instruction and
> data adresses.
>
> Example usage:
>
> qemu-system-xxx <other options> -d plugin -plugin libexeclog.so,afilter=0x1000-0x2000,dfilter=0x388
>
> This would only log instruction in the address range 0x1000 to 0x2000
> and accessing data at address 0x388.
>
> Changes in v2:
> - rebased on top of latest master
>
> Sven Schnelle (3):
>   plugins/execlog: pass matches array to parse_vaddr_match

I think we've lost a patch in the re-posting. patchew hasn't seen it
either:

  https://patchew.org/QEMU/20240229150729.1620410-1-svens@stackframe.org/

>   plugins/execlog: add data address match
>   plugins/execlog: add address range matching
>
>  contrib/plugins/execlog.c | 95 ++++++++++++++++++++++++++++++++-------
>  1 file changed, 79 insertions(+), 16 deletions(-)
Sven Schnelle Feb. 29, 2024, 7:40 p.m. UTC | #2
Hi Alex,

Alex Bennée <alex.bennee@linaro.org> writes:

> Sven Schnelle <svens@stackframe.org> writes:
> I think we've lost a patch in the re-posting. patchew hasn't seen it
> either:
>
>   https://patchew.org/QEMU/20240229150729.1620410-1-svens@stackframe.org/
>
>>   plugins/execlog: add data address match
>>   plugins/execlog: add address range matching
>>
>>  contrib/plugins/execlog.c | 95 ++++++++++++++++++++++++++++++++-------
>>  1 file changed, 79 insertions(+), 16 deletions(-)

Yes, i got a 550 mail. But i'll look into the qemu_set_dfilter_ranges()
before resending.