Hi Jingbo,

On 2023/2/3 11:53, Jingbo Xu wrote:
As new flags introduced, the corresponding print symbols for trace are
not added accordingly.  Add these missing print symbols for these flags.

Signed-off-by: Jingbo Xu <[email protected]>
---
  include/trace/events/erofs.h | 14 ++++++++++----
  1 file changed, 10 insertions(+), 4 deletions(-)

diff --git a/include/trace/events/erofs.h b/include/trace/events/erofs.h
index 4f4c44ea3a65..c0dbe9c4f656 100644
--- a/include/trace/events/erofs.h
+++ b/include/trace/events/erofs.h
@@ -19,12 +19,18 @@ struct erofs_map_blocks;
                { 1,            "DIR" })
#define show_map_flags(flags) __print_flags(flags, "|", \
-       { EROFS_GET_BLOCKS_RAW, "RAW" })
+       { EROFS_GET_BLOCKS_RAW,         "RAW" },      \

Initially, it was just used for reading compressed (raw) data
for compressed files.  However, it's never used actually.

Could we just gid of this flag (EROFS_GET_BLOCKS_RAW)?

Thanks,
Gao Xiang

+       { EROFS_GET_BLOCKS_FIEMAP,      "FIEMAP" },   \
+       { EROFS_GET_BLOCKS_READMORE,    "READMORE" }, \
+       { EROFS_GET_BLOCKS_FINDTAIL,    "FINDTAIL" })
#define show_mflags(flags) __print_flags(flags, "", \
-       { EROFS_MAP_MAPPED,     "M" },                        \
-       { EROFS_MAP_META,       "I" },                        \
-       { EROFS_MAP_ENCODED,    "E" })
+       { EROFS_MAP_MAPPED,             "M" },                \
+       { EROFS_MAP_META,               "I" },                \
+       { EROFS_MAP_ENCODED,            "E" },                \
+       { EROFS_MAP_FULL_MAPPED,        "F" },                \
+       { EROFS_MAP_FRAGMENT,           "R" },                \
+       { EROFS_MAP_PARTIAL_REF,        "P" })
TRACE_EVENT(erofs_lookup,

Reply via email to