From 4d454d3759497ede0e59e3ba9d56bfa1c5082b38 Mon Sep 17 00:00:00 2001 From: Saya Sugiura Date: Tue, 28 Jul 2020 09:31:58 +0900 Subject: logstorage: Add debug logs This commit adds debug logs for opening and scanning DLT log files in Logstorage device. Signed-off-by: Saya Sugiura Signed-off-by: Bui Nguyen Quoc Thanh --- src/offlinelogstorage/dlt_offline_logstorage.c | 7 ++++++ .../dlt_offline_logstorage_behavior.c | 27 +++++++++++++++++++--- 2 files changed, 31 insertions(+), 3 deletions(-) diff --git a/src/offlinelogstorage/dlt_offline_logstorage.c b/src/offlinelogstorage/dlt_offline_logstorage.c index 94364d6..2bf5fad 100644 --- a/src/offlinelogstorage/dlt_offline_logstorage.c +++ b/src/offlinelogstorage/dlt_offline_logstorage.c @@ -2110,6 +2110,13 @@ int dlt_logstorage_write(DltLogStorage *handle, } /* prepare log file (create and/or open)*/ + if (config[i]->ecuid == NULL) + dlt_vlog(LOG_DEBUG, "%s: ApId-CtId-EcuId [%s]-[%s]-[]\n", __func__, + config[i]->apids, config[i]->ctids); + else + dlt_vlog(LOG_DEBUG, "%s: ApId-CtId-EcuId [%s]-[%s]-[%s]\n", __func__, + config[i]->apids, config[i]->ctids, config[i]->ecuid); + ret = config[i]->dlt_logstorage_prepare(config[i], uconfig, handle->device_mount_point, diff --git a/src/offlinelogstorage/dlt_offline_logstorage_behavior.c b/src/offlinelogstorage/dlt_offline_logstorage_behavior.c index 75be657..fa6aba5 100644 --- a/src/offlinelogstorage/dlt_offline_logstorage_behavior.c +++ b/src/offlinelogstorage/dlt_offline_logstorage_behavior.c @@ -279,6 +279,7 @@ int dlt_logstorage_storage_dir_info(DltLogStorageUserConfig *file_config, char *path, DltLogStorageFilterConfig *config) { + int check = 0; int i = 0; int cnt = 0; int ret = 0; @@ -297,11 +298,12 @@ int dlt_logstorage_storage_dir_info(DltLogStorageUserConfig *file_config, cnt = scandir(path, &files, 0, alphasort); if (cnt < 0) { - dlt_log(LOG_ERR, - "dlt_logstorage_storage_dir_info: Failed to scan directory\n"); + dlt_vlog(LOG_ERR, "%s: Failed to scan directory\n", __func__); return -1; } + dlt_vlog(LOG_DEBUG, "%s: Scanned [%d] files from %s\n", __func__, cnt, path); + /* In order to have a latest status of file list, * the existing records must be deleted before updating */ @@ -357,9 +359,13 @@ int dlt_logstorage_storage_dir_info(DltLogStorageUserConfig *file_config, (*tmp)->name = strdup(files[i]->d_name); (*tmp)->idx = current_idx; (*tmp)->next = NULL; + check++; } } + dlt_vlog(LOG_DEBUG, "%s: After dir scan: [%d] files of [%s]\n", __func__, + check, config->file_name); + if (ret == 0) { max_idx = dlt_logstorage_sort_file_name(&config->records); @@ -480,9 +486,13 @@ int dlt_logstorage_open_log_file(DltLogStorageFilterConfig *config, } config->working_file_name = strdup((*newest)->name); } - strcat(absolute_file_path, config->working_file_name); + dlt_vlog(LOG_DEBUG, + "%s: Number of log files-newest file-wrap_id [%d]-[%s]-[%d]\n", + __func__, num_log_files, config->working_file_name, + config->wrap_id); + ret = stat(absolute_file_path, &s); /* if size is enough, open it */ @@ -529,10 +539,17 @@ int dlt_logstorage_open_log_file(DltLogStorageFilterConfig *config, if (config->wrap_id && stat(absolute_file_path, &s) == 0) { remove(absolute_file_path); num_log_files -= 1; + dlt_vlog(LOG_DEBUG, + "%s: Remove '%s' (num_log_files: %d, config->num_files:%d)\n", + __func__, absolute_file_path, num_log_files, config->num_files); } config->log = fopen(absolute_file_path, "a+"); + dlt_vlog(LOG_DEBUG, + "%s: Filename and Index after updating [%s]-[%d]\n", + __func__, file_name, idx); + /* Add file to file list */ *tmp = malloc(sizeof(DltLogStorageFileList)); @@ -557,6 +574,10 @@ int dlt_logstorage_open_log_file(DltLogStorageFilterConfig *config, sizeof(absolute_file_path) / sizeof(char)); strcat(absolute_file_path, storage_path); strcat(absolute_file_path, (*head)->name); + dlt_vlog(LOG_DEBUG, + "%s: Remove '%s' (num_log_files: %d, config->num_files:%d)\n", + __func__, absolute_file_path, num_log_files, + config->num_files); remove(absolute_file_path); free((*head)->name); (*head)->name = NULL; -- cgit v1.2.1