coredump: fix ruff warnings on parser scripts

This fixes ruff warnings on coredump parser scripts.

Note SIM115 is not fixed yet as it requires more intrusive
changes to handle opened file with context manager.

Signed-off-by: Daniel Leung <[email protected]>

Author: dcpleung

.ruff-excludes.toml

@@ -129,18 +129,10 @@
     "UP015",    # https://docs.astral.sh/ruff/rules/redundant-open-modes
 ]
 "./scripts/coredump/coredump_parser/elf_parser.py" = [
-    "E501",     # https://docs.astral.sh/ruff/rules/line-too-long
-    "I001",     # https://docs.astral.sh/ruff/rules/unsorted-imports
     "SIM115",   # https://docs.astral.sh/ruff/rules/open-file-with-context-handler
-    "UP031",    # https://docs.astral.sh/ruff/rules/printf-string-formatting
-    "UP039",    # https://docs.astral.sh/ruff/rules/unnecessary-class-parentheses
 ]
 "./scripts/coredump/coredump_parser/log_parser.py" = [
-    "I001",     # https://docs.astral.sh/ruff/rules/unsorted-imports
     "SIM115",   # https://docs.astral.sh/ruff/rules/open-file-with-context-handler
-    "UP030",    # https://docs.astral.sh/ruff/rules/format-literals
-    "UP031",    # https://docs.astral.sh/ruff/rules/printf-string-formatting
-    "UP032",    # https://docs.astral.sh/ruff/rules/f-string
 ]
 "./scripts/coredump/coredump_serial_log_parser.py" = [
     "SIM115",   # https://docs.astral.sh/ruff/rules/open-file-with-context-handler

scripts/coredump/coredump_parser/elf_parser.py

@@ -6,11 +6,10 @@
 
 import logging
 import struct
+from enum import IntEnum
 
 import elftools
 from elftools.elf.elffile import ELFFile
-from enum import IntEnum
-
 
 # ELF section flags
 SHF_WRITE = 0x1
@@ -144,8 +143,8 @@ def parse(self):
             if store:
                 mem_region = {"start": sec_start, "end": sec_end, "data": section.data()}
                 logger.info(
-                    "ELF Section: 0x%x to 0x%x of size %d (%s)"
-                    % (mem_region["start"], mem_region["end"], len(mem_region["data"]), sect_desc)
+                    f'ELF Section: 0x{mem_region["start"]:x} to 0x{mem_region["end"]:x} '
+                    f'of size {mem_region["data"]:d} ({sect_desc:s})'
                 )
 
                 self.memory_regions.append(mem_region)

scripts/coredump/coredump_parser/log_parser.py

@@ -7,7 +7,6 @@
 import logging
 import struct
 
-
 # Note: keep sync with C code
 COREDUMP_HDR_ID = b'ZE'
 COREDUMP_HDR_VER = 2
@@ -126,7 +125,7 @@ def parse_memory_section(self):
         mem = {"start": saddr, "end": eaddr, "data": data}
         self.memory_regions.append(mem)
 
-        logger.info("Memory: 0x%x to 0x%x of size %d" % (saddr, eaddr, size))
+        logger.info(f"Memory: 0x{saddr:x} to 0x{eaddr:x} of size {size:d}")
 
         return True
 
@@ -156,7 +155,7 @@ def parse(self):
             "reason": reason,
         }
 
-        logger.info("Reason: {0}".format(reason_string(reason)))
+        logger.info(f"Reason: {reason_string(reason)}")
         logger.info(f"Pointer size {ptr_size}")
 
         del id1, id2, hdr_ver, tgt_code, ptr_size, flags, reason