[RFC PATCH 0/4] kunit: more assertion reworking

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



This is a follow up to these three series:
https://lore.kernel.org/all/20220113165931.451305-1-dlatypov@xxxxxxxxxx/
https://lore.kernel.org/all/20220118223506.1701553-1-dlatypov@xxxxxxxxxx/
https://lore.kernel.org/all/20220125210011.3817742-1-dlatypov@xxxxxxxxxx/

The two goals of those series were
a) reduce the size of struct kunit_assert and friends.
   (struct kunit_assert went from 48 => 8 bytes on UML.)
b) simplify the internal code, mostly by deleting macros

This series goes further
a) sizeof(struct kunit_assert) = 0 now
b) e.g. we delete another class of macros (KUNIT_INIT_*_ASSERT_STRUCT)

Note: this does change the function signature of
kunit_do_failed_assertion, so we'd need to update the rust wrapper in https://github.com/Rust-for-Linux/linux/blob/rust/rust/kernel/kunit.rs
But otherwise, I don't think this series changes anything on the
rust-side.

Daniel Latypov (4):
  rfc: kunit: remove format func from struct kunit_assert, get it to 0
    bytes
  rfc: kunit: rename base KUNIT_ASSERTION macro to _KUNIT_FAILED
  rfc: kunit: eliminate KUNIT_INIT_*_ASSERT_STRUCT macros
  kunit: delcare kunit_assert structs as const

 include/kunit/assert.h |  74 ++-----------------------
 include/kunit/test.h   | 123 ++++++++++++++++++++++-------------------
 lib/kunit/test.c       |   7 ++-
 3 files changed, 76 insertions(+), 128 deletions(-)


base-commit: e7eaffce47b7db72b077630dbe836f0c4132496d
-- 
2.36.1.124.g0e6072fb45-goog




[Index of Archives]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux