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

[Dmitry V. Levin]

On Wed, Mar 04, 2020 at 04:21:21PM +0100, Philippe De Muyter wrote:
> 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

Another option is to follow e.g. ffmpeg's example [1] and introduce
a configure option that would enable GPL-2.0-only code and effectively
make the result distributable under GPL-2.0-only terms.

[1] https://git.ffmpeg.org/gitweb/ffmpeg.git/blob_plain/HEAD:/LICENSE.md


-- 
ldv