lkml.org 
[lkml]   [2014]   [Jan]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 06/12] tools lib traceevent: Unregister handler when kvm plugin unloaded
    Date
    The kvm handlers should be unregistered when the plugin unloaded
    otherwise they'll try to access invalid memory.

    Acked-by: Steven Rostedt <rostedt@goodmis.org>
    Reviewed-by: Jiri Olsa <jolsa@redhat.com>
    Signed-off-by: Namhyung Kim <namhyung@kernel.org>
    ---
    tools/lib/traceevent/plugin_kvm.c | 29 +++++++++++++++++++++++++++++
    1 file changed, 29 insertions(+)

    diff --git a/tools/lib/traceevent/plugin_kvm.c b/tools/lib/traceevent/plugin_kvm.c
    index a0e282c6b967..9e0e8c61b43b 100644
    --- a/tools/lib/traceevent/plugin_kvm.c
    +++ b/tools/lib/traceevent/plugin_kvm.c
    @@ -434,3 +434,32 @@ int PEVENT_PLUGIN_LOADER(struct pevent *pevent)
    PEVENT_FUNC_ARG_VOID);
    return 0;
    }
    +
    +void PEVENT_PLUGIN_UNLOADER(struct pevent *pevent)
    +{
    + pevent_unregister_event_handler(pevent, -1, "kvm", "kvm_exit",
    + kvm_exit_handler, NULL);
    +
    + pevent_unregister_event_handler(pevent, -1, "kvm", "kvm_emulate_insn",
    + kvm_emulate_insn_handler, NULL);
    +
    + pevent_unregister_event_handler(pevent, -1, "kvmmmu", "kvm_mmu_get_page",
    + kvm_mmu_get_page_handler, NULL);
    +
    + pevent_unregister_event_handler(pevent, -1, "kvmmmu", "kvm_mmu_sync_page",
    + kvm_mmu_print_role, NULL);
    +
    + pevent_unregister_event_handler(pevent, -1,
    + "kvmmmu", "kvm_mmu_unsync_page",
    + kvm_mmu_print_role, NULL);
    +
    + pevent_unregister_event_handler(pevent, -1, "kvmmmu", "kvm_mmu_zap_page",
    + kvm_mmu_print_role, NULL);
    +
    + pevent_unregister_event_handler(pevent, -1, "kvmmmu",
    + "kvm_mmu_prepare_zap_page", kvm_mmu_print_role,
    + NULL);
    +
    + pevent_unregister_print_function(pevent, process_is_writable_pte,
    + "is_writable_pte");
    +}
    --
    1.7.11.7


    \
     
     \ /
      Last update: 2014-01-16 04:21    [W:4.354 / U:0.012 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site