[igt-dev] [PATCH i-g-t v3 2/2] tests/gem_ctx_param: Update invalid param

Chris Wilson chris at chris-wilson.co.uk
Tue Jan 23 21:08:10 UTC 2018


Quoting Antonio Argenziano (2018-01-23 19:15:23)
> 
> 
> On 19/01/18 01:31, Chris Wilson wrote:
> > Still expecting a
> > 
> > if (!(flags & NICE))
> >       capset(~CAP_SYS_NICE);
> > 
> > pass
> > 
> 
> Sorry, I missed the suggestion from previous reviews. I am not sure if 
> it would be best to have one single patch (like 
> http://paste.debian.net/1006666/) or split out the introduction of libcap.

Fair enough, if its going to be more than a simple call and you want to
introduce some infrastructure (configure/meson), definitely split it out
to another patch or 10.
-Chris


More information about the igt-dev mailing list