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 for few compilation warnings and checkpatch errors. #177

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Fix for few compilation warnings and checkpatch errors. #177

wants to merge 7 commits into from

Commits on Aug 19, 2024

  1. rasdaemon: Fix for compilation warning in ras-memory-failure-handler.c 

    Fix for following compilation warning,
ras-memory-failure-handler.c:120:6: warning: implicit declaration of function ‘asprintf’; did you mean ‘vsprintf’? [-Wimplicit-function-declaration]
  if (asprintf(&env[ei++], "PATH=%s", getenv("PATH") ?: "/sbin:/usr/sbin:/bin:/usr/bin") < 0)

Signed-off-by: Shiju Jose <[email protected]>
    @shijujose4
    shijujose4 committed Aug 19, 2024
    Configuration menu
    Copy the full SHA
    3e61563 View commit details
    Browse the repository at this point in the history

  2. rasdaemon: rbtree: removed unused definition for RB_ROOT 

    Removed unused definition for RB_ROOT from rbtree.h

Signed-off-by: Shiju Jose <[email protected]>
    @shijujose4
    shijujose4 committed Aug 19, 2024
    Configuration menu
    Copy the full SHA
    e1852b1 View commit details
    Browse the repository at this point in the history

  3. rasdaemon: ras-mce-handler: Fix checkpatch errors 

    Fix following checkpatch error in  ras-mce-handler.c

Delete below obselte code under #if 0 ... #endif
WARNING: Consider removing the code enclosed by this #if 0 and its #endif

WARNING: Consider removing the code enclosed by this #if 0 and its #endif

Signed-off-by: Shiju Jose <[email protected]>
    @shijujose4
    shijujose4 committed Aug 19, 2024
    Configuration menu
    Copy the full SHA
    9620d88 View commit details
    Browse the repository at this point in the history

  4. rasdaemon: ras-events: removed obselete code under #if 0 

    Remove unused code enclosed under #if 0 to fix the checkpatch
warnings.

Signed-off-by: Shiju Jose <[email protected]>
    @shijujose4
    shijujose4 committed Aug 19, 2024
    Configuration menu
    Copy the full SHA
    bbc69a5 View commit details
    Browse the repository at this point in the history

  5. rasdaemon: ras-arm-handler: Fix checkpatch warning length exceeds 120… 

    … columns

Fix following checkpatch warning in ras-arm-handler.
+			trace_seq_printf(s, " Program execution can be restarted reliably at the PC associated with the error");

Signed-off-by: Shiju Jose <[email protected]>
    @shijujose4
    shijujose4 committed Aug 19, 2024
    Configuration menu
    Copy the full SHA
    ba8957e View commit details
    Browse the repository at this point in the history

  6. rasdaemon: ras-events: Fix warning ‘filter_ras_mc_event’ defined but … 

    …not used

Fix following compilation warning,
ras-events.c:318:12: warning: ‘filter_ras_mc_event’ defined but not used [-Wunused-function]
 static int filter_ras_mc_event(struct ras_events *ras, char *group, char *event,

Signed-off-by: Shiju Jose <[email protected]>
    @shijujose4
    shijujose4 committed Aug 19, 2024
    Configuration menu
    Copy the full SHA
    856e52b View commit details
    Browse the repository at this point in the history

  7. rasdaemon: ras-non-standard-handler: Fix checkpatch warning 

    Fix following checkpatch warning,
CHECK: spaces preferred around that '*' (ctx:WxV)
+	sqlite3_stmt	*stmt_dec_record;

Signed-off-by: Shiju Jose <[email protected]>
    @shijujose4
    shijujose4 committed Aug 19, 2024
    Configuration menu
    Copy the full SHA
    5edc835 View commit details
    Browse the repository at this point in the history