Re: [PATCH 1/1] drm/shmem: Dual licence the files as GPL-2 and MIT

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, Nov 12, 2022 at 1:44 PM Robert Swindells <rjs@xxxxxxxxxx> wrote:
>
> Contributors to these files are:
>
> Noralf Trønnes <noralf@xxxxxxxxxxx>
> Liu Zixian <liuzixian4@xxxxxxxxxx>
> Dave Airlie <airlied@xxxxxxxxxx>
> Thomas Zimmermann <tzimmermann@xxxxxxx>
> Lucas De Marchi <lucas.demarchi@xxxxxxxxx>
> Gerd Hoffmann <kraxel@xxxxxxxxxx>
> Rob Herring <robh@xxxxxxxxxx>

My contributions are related to the madvise functions. That's largely
lifted or inspired from the MSM code which is GPL only. That in turn
looks inspired from i915 which is MIT (though not much more than the
comment):

$ git grep 'Our goal here is to return as much of the memory'
drivers/gpu/drm/drm_gem_shmem_helper.c: /* Our goal here is to return
as much of the memory as
drivers/gpu/drm/i915/gem/i915_gem_shmem.c:       * Our goal here is to
return as much of the memory as
drivers/gpu/drm/msm/msm_gem.c:  /* Our goal here is to return as much
of the memory as


I imagine this is not the only example in this file. In fact, looking
at the introduction of this file, it looks like it originated from V3D
code as that was the first driver to convert over. V3D is licensed
GPL2+. Of course, its code was not written in a vacuum either and came
from ???

This to me is a problem with the dual licensing in DRM drivers. Code
moves around with little attention paid at the time to licensing. I
wouldn't trust anything claiming MIT license is not GPL
'contaminated'.

OTOH, there's really only one way for the madvise code to work, so
maybe not a copyrightable work on its own.

Rob




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux