plugins: Move declarations around and rename 'hwaddr' argument

[PATCH 0/3] plugins: Move declarations around and rename 'hwaddr' argument

Philippe Mathieu-Daudé posted 3 patches 3 years, 12 months ago

Download series mbox

Test docker-mingw@fedora passed

Test checkpatch passed

Test asan passed

Test docker-quick@centos7 passed

Test FreeBSD passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20200510171119.20827-1-f4bug@amsat.org

Maintainers: Laurent Vivier <laurent@vivier.eu>, Michael Tokarev <mjt@tls.msk.ru>

include/qemu/plugin.h      | 65 ++++++++++++++++++--------------------
include/qemu/qemu-plugin.h |  2 +-
plugins/api.c              |  4 +--
3 files changed, 34 insertions(+), 37 deletions(-)

Expand all Fold all

[PATCH 0/3] plugins: Move declarations around and rename 'hwaddr' argument

Posted by Philippe Mathieu-Daudé 3 years, 12 months ago

Rename qemu_plugin_hwaddr_is_io() 'hwaddr' argument to later
allow declaration of the 'hwaddr' type to be poisoned (see [*]).

[*] https://lists.gnu.org/archive/html/qemu-devel/2020-05/msg02254.html
    "exec/cpu: Poison 'hwaddr' type in user-mode emulation"

Philippe Mathieu-Daudé (3):
  qemu/plugin: Trivial code movement
  qemu/plugin: Move !CONFIG_PLUGIN stubs altogether
  qemu/qemu-plugin: Make qemu_plugin_hwaddr_is_io() hwaddr argument
    const

 include/qemu/plugin.h      | 65 ++++++++++++++++++--------------------
 include/qemu/qemu-plugin.h |  2 +-
 plugins/api.c              |  4 +--
 3 files changed, 34 insertions(+), 37 deletions(-)

-- 
2.21.3

Re: [PATCH 0/3] plugins: Move declarations around and rename 'hwaddr' argument

Posted by Emilio G. Cota 3 years, 12 months ago

On Sun, May 10, 2020 at 19:11:16 +0200, Philippe Mathieu-Daudé wrote:
> Rename qemu_plugin_hwaddr_is_io() 'hwaddr' argument to later
> allow declaration of the 'hwaddr' type to be poisoned (see [*]).
> 
> [*] https://lists.gnu.org/archive/html/qemu-devel/2020-05/msg02254.html
>     "exec/cpu: Poison 'hwaddr' type in user-mode emulation"
> 
> Philippe Mathieu-Daudé (3):
>   qemu/plugin: Trivial code movement
>   qemu/plugin: Move !CONFIG_PLUGIN stubs altogether
>   qemu/qemu-plugin: Make qemu_plugin_hwaddr_is_io() hwaddr argument
>     const

Reviewed-by: Emilio G. Cota <cota@braap.org>
for the series.

Thanks,
		E.

Re: [PATCH 0/3] plugins: Move declarations around and rename 'hwaddr' argument

Posted by Alex Bennée 3 years, 12 months ago

Philippe Mathieu-Daudé <f4bug@amsat.org> writes:

> Rename qemu_plugin_hwaddr_is_io() 'hwaddr' argument to later
> allow declaration of the 'hwaddr' type to be poisoned (see [*]).
>
> [*] https://lists.gnu.org/archive/html/qemu-devel/2020-05/msg02254.html
>     "exec/cpu: Poison 'hwaddr' type in user-mode emulation"


Queued to plugins/next, thanks.

>
> Philippe Mathieu-Daudé (3):
>   qemu/plugin: Trivial code movement
>   qemu/plugin: Move !CONFIG_PLUGIN stubs altogether
>   qemu/qemu-plugin: Make qemu_plugin_hwaddr_is_io() hwaddr argument
>     const
>
>  include/qemu/plugin.h      | 65 ++++++++++++++++++--------------------
>  include/qemu/qemu-plugin.h |  2 +-
>  plugins/api.c              |  4 +--
>  3 files changed, 34 insertions(+), 37 deletions(-)


-- 
Alex Bennée