mbox series

[kvm-unit-tests,0/2] riscv: sbi: Clean up multiple report prefix popping

Message ID 20240910150842.156949-1-jamestiotio@gmail.com
Headers show
Series riscv: sbi: Clean up multiple report prefix popping | expand

Message

James Raphael Tiovalen Sept. 10, 2024, 3:08 p.m. UTC 
The first patch of this series adds a helper method to clear multiple
prefixes at once. The second patch then uses this new helper method to
tidy up the report prefix pops in the RISC-V SBI tests.

James Raphael Tiovalen (2):
  lib/report: Add helper method to clear multiple prefixes
  riscv: sbi: Tidy up report prefix pops

 lib/libcflat.h |  1 +
 lib/report.c   | 21 +++++++++++++++------
 riscv/sbi.c    | 17 +++++------------
 3 files changed, 21 insertions(+), 18 deletions(-)

--
2.43.0

Comments

Andrew Jones Sept. 11, 2024, 9:58 a.m. UTC | #1 
On Tue, Sep 10, 2024 at 11:08:40PM GMT, James Raphael Tiovalen wrote:
> The first patch of this series adds a helper method to clear multiple
> prefixes at once. The second patch then uses this new helper method to
> tidy up the report prefix pops in the RISC-V SBI tests.
> 
> James Raphael Tiovalen (2):
>   lib/report: Add helper method to clear multiple prefixes
>   riscv: sbi: Tidy up report prefix pops
> 
>  lib/libcflat.h |  1 +
>  lib/report.c   | 21 +++++++++++++++------
>  riscv/sbi.c    | 17 +++++------------
>  3 files changed, 21 insertions(+), 18 deletions(-)
> 
> --
> 2.43.0
>

Looks good to me. There are no longer any double pops in riscv. Out of
curiosity, I checked the other architectures. arm and ppc both had one
instance of double pop, x86 didn't have any, but s390x had several. We
can leave the adoption of popn to the respective maintainer's preferences
though.

Applied to riscv/sbi, https://gitlab.com/jones-drew/kvm-unit-tests/-/commits/riscv/sbi

Thanks,
drew