Re: [PATCH v8-RESEND 00/33] Fix CONFIG_DRM_USE_DYNAMIC_DEBUG=y regression

From: jim . cromie
Date: Wed Jun 05 2024 - 13:06:13 EST


>
>
> > Im thinking that a combo of 1, 2 would suffice.
> > And we could probably drop the config constraints,
> > especially if the test avoids failing on configs where a failure is expected.
> > IOW - if no test-modules/builtins, run only 1st 3 tests, ending with PERCENT
> >
> > That said, I wonder if the exit 127 should be success instead ?
> >
>
> I don't follow you here. Could you please elaborate why exit 127
> should be a success ?
>

well, exit 127 is sort-of a monkey-patch.
(this) monkey, sees it used by git-bisect to not be a normal fail,
but a special signal that bisection itself failed. (maybe..)

I have no good reason to think its useful for selftests. RFC.

WRT success, if /proc/dynamic_debug/control doesnt exist,
thats probably intended, and we shouldnt complain.

>
> > thanks Lukas,
> > ~jimc
> >
> > > Thanks,
> > > Lukasz