lkml.org 
[lkml]   [2015]   [Apr]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectIntel FSGSBASE support (was: Re: [PATCH 6/8] x86: Enumerate kernel FSGS capability in AT_HWCAP2)

* Borislav Petkov <bp@alien8.de> wrote:

> On Sat, Apr 11, 2015 at 01:08:48AM +0200, Andi Kleen wrote:

> > > > +/* HWCAP2 supplies kernel enabled CPU feature, so that the application
> > > > + can know that it can safely use them. The bits are defined in
> > > > + uapi/asm/hwcap.h. */
> > >
> > > Comments formatting.
> >
> > The formatting matches all the other comments in the file.
>
> That doesn't mean you need to add new comments with the old
> formatting which we're trying to get rid of.

Exactly, and the thing is, I've seen this behavior before, so I'm also
going to ignore all these Intel FSGSBASE patches from Andi Kleen, for
the following technical reasons:

- they are poorly written,

- a necessary precondition of such features is the thorough (and
constructively conducted) clean-up of the underlying code,

- the series exposes a new user-space ABI that is going to be exposed
forever and has to be done right on the first attempt,

- unacceptable passive-aggressive behavior was directed by Andi
against constructive, technical feedback from reviewers and
maintainers.

So consider Intel FSGSBASE support NACK-ed on these four technical
grounds. All four problems have to be properly addressed (beyond
addressing all the other feedback that was given) before the NACK is
lifted.

Maybe someone else has the time to pick up and deobfuscate (or
entirely rewrite) these patches into a properly written series?

The Intel FSGSBASE hardware feature itself looks potentially useful,
so I'm not opposed to the concept itself.

Thanks,

Ingo


\
 
 \ /
  Last update: 2015-04-11 11:01    [W:0.096 / U:5.800 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site