mbox series

[0/3] Simplify and improve fpga test suites using deferred actions

Message ID 20240725125031.308195-1-marpagan@redhat.com (mailing list archive)
Headers show
Series Simplify and improve fpga test suites using deferred actions | expand

Message

Marco Pagani July 25, 2024, 12:50 p.m. UTC
Use deferred actions to simplify fpga test suites and avoid potential
memory leaks when test cases fail. Deferred actions are executed when a
test is completed either successfully or unsuccessfully (e.g., due to a
failed assertion).

Marco Pagani (3):
  Simplify and improve the fpga manager test suite using deferred
    actions
  Simplify and improve the fpga bridge test suite using deferred actions
  Simplify and improve the fpga region test suite using deferred actions

 drivers/fpga/tests/fpga-bridge-test.c | 25 +++++++---------
 drivers/fpga/tests/fpga-mgr-test.c    | 28 +++++++++++-------
 drivers/fpga/tests/fpga-region-test.c | 41 ++++++++++++++++-----------
 3 files changed, 52 insertions(+), 42 deletions(-)


base-commit: 3a0fa8e97d3024de199634667900b6a90bbe43d0

Comments

Xu Yilun Aug. 15, 2024, 3:01 a.m. UTC | #1
On Thu, Jul 25, 2024 at 02:50:28PM +0200, Marco Pagani wrote:
> Use deferred actions to simplify fpga test suites and avoid potential
> memory leaks when test cases fail. Deferred actions are executed when a
> test is completed either successfully or unsuccessfully (e.g., due to a
> failed assertion).
> 
> Marco Pagani (3):
>   Simplify and improve the fpga manager test suite using deferred
>     actions
>   Simplify and improve the fpga bridge test suite using deferred actions
>   Simplify and improve the fpga region test suite using deferred actions

Don't forget the prefix in shortlog. I added "fpga: " for them.

Others look good to me.

Acked-by: Xu Yilun <yilun.xu@intel.com>

Applied this series to for-next.

Thanks,
Yilun

> 
>  drivers/fpga/tests/fpga-bridge-test.c | 25 +++++++---------
>  drivers/fpga/tests/fpga-mgr-test.c    | 28 +++++++++++-------
>  drivers/fpga/tests/fpga-region-test.c | 41 ++++++++++++++++-----------
>  3 files changed, 52 insertions(+), 42 deletions(-)
> 
> 
> base-commit: 3a0fa8e97d3024de199634667900b6a90bbe43d0
> -- 
> 2.45.2
> 
>