X-Git-Url: https://gerrit.fd.io/r/gitweb?a=blobdiff_plain;f=src%2Fvppinfra%2Fmaplog.c;h=4e59631bdbe502f857b7be50850b68cca9fd56c1;hb=f8b61fb538046e287063864349d43e98b62b5faf;hp=3e08973bc8c3fa8e1ff7bf5df0b09492f9c6d4a8;hpb=55c79e9c7e14b501baa72bc8b415e0a66752ed01;p=vpp.git diff --git a/src/vppinfra/maplog.c b/src/vppinfra/maplog.c index 3e08973bc8c..4e59631bdbe 100644 --- a/src/vppinfra/maplog.c +++ b/src/vppinfra/maplog.c @@ -24,10 +24,10 @@ * @param[in/out] a init args structure * @return 0 => success, <0 => failure */ -int -clib_maplog_init (clib_maplog_init_args_t * a) +__clib_export int +clib_maplog_init (clib_maplog_init_args_t *a) { - int i, fd; + int i, fd, limit; int rv = 0; u8 zero = 0; u32 record_size_in_cache_lines; @@ -42,7 +42,7 @@ clib_maplog_init (clib_maplog_init_args_t * a) if (mm->flags & CLIB_MAPLOG_FLAG_INIT) return (-2); - memset (mm, 0, sizeof (*mm)); + clib_memset (mm, 0, sizeof (*mm)); record_size_in_cache_lines = (a->record_size_in_bytes + CLIB_CACHE_LINE_BYTES - @@ -54,7 +54,6 @@ clib_maplog_init (clib_maplog_init_args_t * a) /* Round up file size in records to a power of 2, for speed... */ mm->log2_file_size_in_records = max_log2 (file_size_in_records); file_size_in_records = 1ULL << (mm->log2_file_size_in_records); - a->file_size_in_bytes = file_size_in_records * record_size_in_cache_lines * CLIB_CACHE_LINE_BYTES; @@ -68,9 +67,19 @@ clib_maplog_init (clib_maplog_init_args_t * a) mm->file_size_in_records = file_size_in_records; mm->flags |= CLIB_MAPLOG_FLAG_INIT; mm->record_size_in_cachelines = record_size_in_cache_lines; + limit = 2; + if (a->maplog_is_circular) + { + mm->log2_file_size_in_records = 63; + mm->flags |= CLIB_MAPLOG_FLAG_CIRCULAR; + limit = 1; + } - /* Map two files */ - for (i = 0; i < 2; i++) + /* + * Map the one and only file for a circular log, + * two files for a normal log. + */ + for (i = 0; i < limit; i++) { mm->filenames[i] = format (0, "%v_%d", mm->file_basename, mm->current_file_index++); @@ -104,7 +113,7 @@ clib_maplog_init (clib_maplog_init_args_t * a) (void) close (fd); } - memset (h, 0, sizeof (*h)); + clib_memset (h, 0, sizeof (*h)); h->maplog_major_version = MAPLOG_MAJOR_VERSION; h->maplog_minor_version = MAPLOG_MINOR_VERSION; h->maplog_patch_version = MAPLOG_PATCH_VERSION; @@ -117,6 +126,7 @@ clib_maplog_init (clib_maplog_init_args_t * a) h->file_size_in_records = file_size_in_records; h->number_of_records = ~0ULL; h->number_of_files = ~0ULL; + h->maplog_flag_circular = a->maplog_is_circular; memcpy (h->file_basename, mm->file_basename, vec_len (mm->file_basename)); mm->header_filename = format (0, "%v_header", mm->file_basename); @@ -143,7 +153,7 @@ fail: if (fd >= 0) (void) close (fd); - for (i = 0; i < 2; i++) + for (i = 0; i < limit; i++) { if (mm->file_baseva[i]) (void) munmap ((u8 *) mm->file_baseva[i], a->file_size_in_bytes); @@ -161,8 +171,8 @@ fail: /* slow path: unmap a full log segment, and replace it */ -u8 * -_clib_maplog_get_entry_slowpath (clib_maplog_main_t * mm, u64 my_record_index) +__clib_export u8 * +_clib_maplog_get_entry_slowpath (clib_maplog_main_t *mm, u64 my_record_index) { int fd; u8 *rv; @@ -171,6 +181,9 @@ _clib_maplog_get_entry_slowpath (clib_maplog_main_t * mm, u64 my_record_index) u64 file_size_in_bytes = mm->file_size_in_records * mm->record_size_in_cachelines * CLIB_CACHE_LINE_BYTES; + /* This should never happen */ + ASSERT ((mm->flags & CLIB_MAPLOG_FLAG_CIRCULAR) == 0); + /* * Kill some time by calling format before we make the previous log * segment disappear. Obviously it won't do to call clib_maplog_get_entry(), @@ -256,6 +269,7 @@ clib_maplog_update_header (clib_maplog_main_t * mm) /* Fix the header... */ h->number_of_records = mm->next_record_index; h->number_of_files = mm->current_file_index; + h->maplog_flag_wrapped = (mm->flags & CLIB_MAPLOG_FLAG_WRAPPED) ? 1 : 0; /* Back to the beginning of the log header... */ if (lseek (fd, 0, SEEK_SET) < 0) @@ -281,10 +295,10 @@ out: * * @param[in/out] mm mapped log object */ -void -clib_maplog_close (clib_maplog_main_t * mm) +__clib_export void +clib_maplog_close (clib_maplog_main_t *mm) { - int i; + int i, limit; u64 file_size_in_bytes; if (!(mm->flags & CLIB_MAPLOG_FLAG_INIT)) @@ -296,8 +310,10 @@ clib_maplog_close (clib_maplog_main_t * mm) mm->file_size_in_records * mm->record_size_in_cachelines * CLIB_CACHE_LINE_BYTES; + limit = (mm->flags & CLIB_MAPLOG_FLAG_CIRCULAR) ? 1 : 2; + /* unmap current + next segments */ - for (i = 0; i < 2; i++) + for (i = 0; i < limit; i++) { (void) munmap ((u8 *) mm->file_baseva[i], file_size_in_bytes); vec_free (mm->filenames[i]); @@ -305,7 +321,7 @@ clib_maplog_close (clib_maplog_main_t * mm) vec_free (mm->file_basename); vec_free (mm->header_filename); - memset (mm, 0, sizeof (*mm)); + clib_memset (mm, 0, sizeof (*mm)); } /** @@ -315,8 +331,8 @@ clib_maplog_close (clib_maplog_main_t * mm) * @param [in] h clib_maplog_header_t pointer * @param [in] verbose self-explanatory */ -u8 * -format_maplog_header (u8 * s, va_list * args) +__clib_export u8 * +format_maplog_header (u8 *s, va_list *args) { clib_maplog_header_t *h = va_arg (*args, clib_maplog_header_t *); int verbose = va_arg (*args, int); @@ -324,13 +340,15 @@ format_maplog_header (u8 * s, va_list * args) if (!verbose) goto brief; s = format (s, "basename %s ", h->file_basename); - s = format (s, "log ver %d.%d.%d app id %u ver %d.%d.%d\n", + s = format (s, "log ver %d.%d.%d app id %u ver %d.%d.%d %s %s\n", h->maplog_major_version, h->maplog_minor_version, h->maplog_patch_version, h->application_id, h->application_major_version, - h->application_minor_version, h->application_patch_version); + h->application_minor_version, h->application_patch_version, + h->maplog_flag_circular ? "circular" : "linear", + h->maplog_flag_wrapped ? "wrapped" : "not wrapped"); s = format (s, " records are %d %d-byte cachelines\n", h->record_size_in_cachelines, h->cacheline_size); s = format (s, " files are %lld records long, %lld files\n", @@ -362,7 +380,7 @@ brief: * @param [in] file_basename Same basename supplied to clib_maplog_init * @param [in] fp_arg Callback function pointer */ -int +__clib_export int clib_maplog_process (char *file_basename, void *fp_arg) { clib_maplog_header_t _h, *h = &_h; @@ -394,6 +412,7 @@ clib_maplog_process (char *file_basename, void *fp_arg) goto out; } (void) close (fd); + fd = -1; file_size_in_bytes = h->file_size_in_records * h->record_size_in_cachelines * CLIB_CACHE_LINE_BYTES; @@ -415,6 +434,7 @@ clib_maplog_process (char *file_basename, void *fp_arg) file_baseva = mmap (0, file_size_in_bytes, PROT_READ, MAP_SHARED, fd, 0); (void) close (fd); + fd = -1; if (file_baseva == (u8 *) MAP_FAILED) { clib_unix_warning ("mmap"); @@ -425,7 +445,27 @@ clib_maplog_process (char *file_basename, void *fp_arg) records_this_file = (records_left > h->file_size_in_records) ? h->file_size_in_records : records_left; - (*fp) (h, file_baseva, records_this_file); + /* + * Normal log, or a circular non-wrapped log, or a circular + * wrapped log which happens to be exactly linear + */ + if (h->maplog_flag_circular == 0 || h->maplog_flag_wrapped == 0 || + ((h->number_of_records % h->file_size_in_records) == 0)) + (*fp) (h, file_baseva, records_this_file); + else + { + /* "Normal" wrapped circular log */ + u64 first_chunk_record_index = h->number_of_records & + (h->file_size_in_records - 1); + u64 first_chunk_number_of_records = records_this_file - + first_chunk_record_index; + u8 *chunk_baseva = file_baseva + + first_chunk_record_index * h->record_size_in_cachelines * + h->cacheline_size; + (*fp) (h, chunk_baseva, first_chunk_number_of_records); + (*fp) (h, file_baseva, + records_this_file - first_chunk_number_of_records); + } if (munmap (file_baseva, file_size_in_bytes) < 0) { @@ -439,7 +479,7 @@ clib_maplog_process (char *file_basename, void *fp_arg) } out: - if (fd > 0) + if (fd >= 0) (void) close (fd); vec_free (this_filename);