Antoine, thanks for the suggestion and the additional review. V2 patch
series forthcoming.
-- Ben
On Sat, Mar 10, 2018 at 11:58 PM, Antoine Martin
wrote:
> On 09/03/18 05:11, Ben Crocker wrote:
> > Sent that a little too soon; please consider "Reviewed-by: Ben Crocker
> > mailto:bcroc...@redh
On 09/03/18 05:11, Ben Crocker wrote:
> Sent that a little too soon; please consider "Reviewed-by: Ben Crocker
> mailto:bcroc...@redhat.com>>" to be
> added after each of the "Signed-off-by: Nicolai Haehnle
> mailto:nicolai.haeh...@amd.com>>" lines.
You can add:
Reviewed-by: Antoine Martin
You sh
Sent that a little too soon; please consider "Reviewed-by: Ben Crocker <
bcroc...@redhat.com>" to be
added after each of the "Signed-off-by: Nicolai Haehnle <
nicolai.haeh...@amd.com>" lines.
Also please note that I rebased Nicolai's original patch,
https://patchwork.freedesktop.org/series/18684/,
From: Nicolai Hähnle
From: Nicolai Hähnle
Having different types of code all trying to check for elevated privileges
is a bad idea. This implementation is the most thorough one.
Signed-off-by: Nicolai Hähnle
---
hw/xfree86/common/xf86Init.c | 59 +
inc
From: Nicolai Hähnle
Having different types of code all trying to check for elevated privileges
is a bad idea. This implementation is the most thorough one.
Signed-off-by: Nicolai Hähnle
---
hw/xfree86/common/xf86Init.c | 59 +
include/os.h