Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix(driver): remove some inconsistencies in our event tables #936

Merged
merged 5 commits into from
Mar 7, 2023
Merged

fix(driver): remove some inconsistencies in our event tables #936

merged 5 commits into from
Mar 7, 2023

Commits on Mar 6, 2023

  1. fix: solve some inconsistencies with PPM_SC_UMOUNT and `PPM_SC_LSTA… 

    …T64`

before this commit these two syscalls were considered "generic" but they had an associate event in the event table. Now these two syscalls use specific events and are no more generic. Please note that we need to craft new events for `PPM_SC_UMOUNT` and `PPM_SC_UMOUNT2` since `PPM_SC_UMOUNT` cannot use `PPME_SYSCALL_UMOUNT_E`, this is just a tmp patch.

Signed-off-by: Andrea Terzolo <andrea.terzolo@polito.it>
    @Andreagit97
    Andreagit97 committed Mar 6, 2023
    Configuration menu
    Copy the full SHA
    a9408ab View commit details
    Browse the repository at this point in the history

  2. fix: add 2 new syscalls PPM_SC_RECV and PPM_SC_SEND 

    Before this commit we just had event pairs to instrument these syscalls but not a real code to identify them, for this reason their event pairs were associated to `PPM_SC_UNKNOWN`.

Signed-off-by: Andrea Terzolo <andrea.terzolo@polito.it>
    @Andreagit97
    Andreagit97 committed Mar 6, 2023
    Configuration menu
    Copy the full SHA
    96174ef View commit details
    Browse the repository at this point in the history

  3. cleanup: use "NA" as unknown event name 

    Signed-off-by: Andrea Terzolo <andrea.terzolo@polito.it>
    @Andreagit97
    Andreagit97 committed Mar 6, 2023
    Configuration menu
    Copy the full SHA
    8f66fc6 View commit details
    Browse the repository at this point in the history

  4. update: support new umount event pair 

    Signed-off-by: Andrea Terzolo <andrea.terzolo@polito.it>
    @Andreagit97
    Andreagit97 committed Mar 6, 2023
    Configuration menu
    Copy the full SHA
    8b4b8b8 View commit details
    Browse the repository at this point in the history

  5. tests: remove PPME_CONTAINER_X event 

    remove `PPME_CONTAINER_X` event from `event_set_to_names_no_generic_events1` test since after this fix it is associated with `NA` name instead of `container`

Signed-off-by: Andrea Terzolo <andrea.terzolo@polito.it>
    @Andreagit97
    Andreagit97 committed Mar 6, 2023
    Configuration menu
    Copy the full SHA
    0f33f1d View commit details
    Browse the repository at this point in the history