diff --git a/.github/workflows/debug.yml b/.github/workflows/debug.yml index 29ca5b2aa..7b6beb4ff 100644 --- a/.github/workflows/debug.yml +++ b/.github/workflows/debug.yml @@ -1,6 +1,9 @@ -on: pull_request +on: + pull_request: + paths: + - 'debug/**' -name: Check Code Style (checkpatch) +name: Check Debug Code Style (pylint) jobs: check: diff --git a/debug/gdbserver.py b/debug/gdbserver.py index 53eeffafe..3b9a661ca 100755 --- a/debug/gdbserver.py +++ b/debug/gdbserver.py @@ -1562,6 +1562,7 @@ def test(self): # assertIn("0xbead", output) class PrivTest(GdbSingleHartTest): + """Base class for a few tests that change privilege levels.""" compile_args = ("programs/priv.S", ) def setup(self): # pylint: disable=attribute-defined-outside-init @@ -1588,8 +1589,8 @@ def setup(self): pass class PrivRw(PrivTest): + """Test reading/writing priv.""" def test(self): - """Test reading/writing priv.""" self.write_nop_program(4) for privilege in range(4): self.gdb.p(f"$priv={privilege}") @@ -1600,9 +1601,9 @@ def test(self): assertEqual(actual, privilege) class PrivChange(PrivTest): + """Test that the core's privilege level actually changes when the debugger + writes it.""" def test(self): - """Test that the core's privilege level actually changes.""" - if 0 not in self.supported: raise TestNotApplicable