[PATCH 1/2] ioctl: add symbolic translations for Nvidia ioctl codes

[Philippe De Muyter]

On Wed, Mar 04, 2020 at 02:14:55AM +0100, Eugene Syromiatnikov wrote:
> On Tue, Feb 11, 2020 at 09:33:18AM +0100, Philippe De Muyter wrote:
> > added header files were retrieved from
> > 
> > git://nv-tegra.nvidia.com/linux-nvgpu.git
> > git://nv-tegra.nvidia.com/linux-nvhost.git
> > git://nv-tegra.nvidia.com/linux-nvmap.git
> > 
> > Signed-off-by: Philippe De Muyter <phdm at qel.be>
> > ---
> 
> > diff --git a/nvgpu.h b/nvgpu.h
> > new file mode 100644
> > index 0000000..5f02a31
> > --- /dev/null
> > +++ b/nvgpu.h
> > @@ -0,0 +1,2157 @@
> > +/*
> > + * NVGPU Public Interface Header
> > + *
> > + * Copyright (c) 2011-2017, NVIDIA CORPORATION.  All rights reserved.
> > + *
> > + * This program is free software; you can redistribute it and/or modify it
> > + * under the terms and conditions of the GNU General Public License,
> > + * version 2, as published by the Free Software Foundation.
> 
> This is problematic, as it's incompatible with the current strace's
> license (LGPL 2.1+).

Is that so problematic ?  These are only headers, not programs.  IANAL,
but that's fair use, needed for interoperability.

Do I need to retype the constants myself ?

Another option is to compile that extension only on machines where those
headers are installed, but that would limit the usefullness of pre-built
binaries in the distributions.

One could also add a message at run-time telling the user to recompile
from sources, or add a way to check if those files are available and
"compile" them at the first usage of strace, but that's stupid extra work

Best regards

Philippe

-- 
Philippe De Muyter +32 2 6101532 Macq SA rue de l'Aeronef 2 B-1140 Bruxelles