Re: [PATCH v2 6/8] test/pdx: add PDX compression unit tests

2025-06-25 Thread Roger Pau Monné
On Tue, Jun 24, 2025 at 03:37:34PM +0200, Anthony PERARD wrote: > On Fri, Jun 20, 2025 at 01:11:28PM +0200, Roger Pau Monne wrote: > > +.PHONY: run > > +run: $(TARGETS) > > +ifeq ($(CC),$(HOSTCC)) > > + for test in $? ; do \ > > + ./$$test ; \ > > + done > > You need to add `set -e`

Re: [PATCH v2 6/8] test/pdx: add PDX compression unit tests

2025-06-25 Thread Anthony PERARD
On Fri, Jun 20, 2025 at 01:11:28PM +0200, Roger Pau Monne wrote: > +.PHONY: run > +run: $(TARGETS) > +ifeq ($(CC),$(HOSTCC)) > + for test in $? ; do \ > + ./$$test ; \ > + done You need to add `set -e` or the exit value from the tested binary might be ignored. This `run` targe

[PATCH v2 6/8] test/pdx: add PDX compression unit tests

2025-06-20 Thread Roger Pau Monne
Introduce a set of unit tests for PDX compression. The unit tests contains both real and crafted memory maps that are then compressed using the selected PDX algorithm. Note the build system for the unit tests has been done in a way to support adding new compression algorithms easily. That requir