On Tue, Oct 15, 2019 at 01:16:35PM +0300, Jarkko Sakkinen wrote: > On Mon, Oct 07, 2019 at 09:46:07PM -0700, Sean Christopherson wrote: > > Use the recently added assertion framework to report errors and exit > > instead of propagating the error back up the stack. Using assertions > > reduces code and provides more detailed error messages, and has no > > downsides as all errors lead to exit(1) anyways, i.e. an assertion > > isn't blocking forward progress. > > > > Signed-off-by: Sean Christopherson <sean.j.christopherson@xxxxxxxxx> > > I'm also dropping all of this. Was too hazy with it because of rush last > week. > > You shoud use EXCEPT_* macros instead of your home baked ones: > > https://www.kernel.org/doc/html/v4.15/dev-tools/kselftest.html > > I don't know what you are talking about in this commit message. > "Recently added" tells me absolutely nothing. All I see that you > are adding your own ad hoc crap. E.g. 1. WTH the new thing is. 2. Why is it overriding the macros already defined for kselftest (see the documentation). 3. Before vDSO commits please provide a patch set that does the migration with clear explanation what is going on. /Jarkko
