Messages in this thread | | | From | Schspa Shi <> | Subject | Re: [PATCH 2/2] debugobject: add unit test for static debug object | Date | Thu, 23 Mar 2023 16:44:21 +0800 |
| |
Thomas Gleixner <tglx@linutronix.de> writes:
> On Thu, Mar 23 2023 at 11:16, Schspa Shi wrote: >> Schspa Shi <schspa@gmail.com> writes: >>> MAINTAINERS | 5 ++ >>> lib/Kconfig.debug | 14 ++++ >>> lib/Makefile | 2 + >>> lib/test_static_debug_object.c | 125 +++++++++++++++++++++++++++++++++ >>> 4 files changed, 146 insertions(+) >>> create mode 100644 lib/test_static_debug_object.c >> >> What do you think about this test case? Should we need it ? There are >> some platform compatibility issues here that need a little optimization. > > What does it buy over the existing self test. Nothing AFACIT aside of > extra code. >
It checks the race of the is_static_object() call in the previous BUG. This test can used to make sure the new fix patch works. The existing self test have no ability to check this.
> Thanks, > > tglx
-- BRs Schspa Shi
| |