[PATCH] drm/i915: Update license on selftest lists to MIT

[PATCH] drm/i915: Update license on selftest lists to MIT

[Ed Maste]

These two files (i915_live_selftests.h, i915_mock_selftests.h) were
introduced in commit 953c7f82eb89 ("drm/i915: Provide a hook for
selftests") and are effectively just a list of selftests.

The selftest implementation itself is largely in i915_selftest.c, and
uses a MIT license.  Graphics drivers are shared with other operating
systems and have long used a permissive license (or dual license) to
facilitate this.

The two selftest list files carried no license when introduced in
953c7f82eb89, presumably as they were considered trivial.  Notably the
general selftest header i915_selftest.h (which does have non-trivial
content) also has an MIT license.

The GPL-2.0 SPDX tag in these two files came from b24413180f56, where
Greg Kroah-Hartman added the tag to all files that had no license.  This
makes sense in general, but it is clear from the context of the original
selftest commit here that these files are a trivial part of an otherwise
MIT-licensed patch to a MIT-licensed component, and should have an MIT
license.

Signed-off-by: Ed Maste <emaste@FreeBSD.org>
---
 drivers/gpu/drm/i915/selftests/i915_live_selftests.h | 2 +-
 drivers/gpu/drm/i915/selftests/i915_mock_selftests.h | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/i915/selftests/i915_live_selftests.h b/drivers/gpu/drm/i915/selftests/i915_live_selftests.h
index 5aee6c9a8295..264b951c4f77 100644
--- a/drivers/gpu/drm/i915/selftests/i915_live_selftests.h
+++ b/drivers/gpu/drm/i915/selftests/i915_live_selftests.h
@@ -1,4 +1,4 @@
-/* SPDX-License-Identifier: GPL-2.0 */
+/* SPDX-License-Identifier: MIT */
 
 #ifndef selftest
 #define selftest(x, y)
diff --git a/drivers/gpu/drm/i915/selftests/i915_mock_selftests.h b/drivers/gpu/drm/i915/selftests/i915_mock_selftests.h
index 0c22e0fc9059..18935e311748 100644
--- a/drivers/gpu/drm/i915/selftests/i915_mock_selftests.h
+++ b/drivers/gpu/drm/i915/selftests/i915_mock_selftests.h
@@ -1,4 +1,4 @@
-/* SPDX-License-Identifier: GPL-2.0 */
+/* SPDX-License-Identifier: MIT */
 
 #ifndef selftest
 #define selftest(x, y)
-- 
2.46.2

✓ i915.CI.BAT: success for drm/i915: Update license on selftest lists to MIT

[Patchwork]

== Series Details ==

Series: drm/i915: Update license on selftest lists to MIT
URL   : https://patchwork.freedesktop.org/series/142266/
State : success

== Summary ==

CI Bug Log - changes from CI_DRM_15804 -> Patchwork_142266v1
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

  External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142266v1/index.html

Participating hosts (42 -> 41)
------------------------------

  Missing    (1): fi-snb-2520m 

Known issues
------------

  Here are the changes found in Patchwork_142266v1 that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@i915_selftest@live:
    - bat-arls-5:         NOTRUN -> [ABORT][1] ([i915#12061])
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142266v1/bat-arls-5/igt@i915_selftest@live.html

  * igt@i915_selftest@live@workarounds:
    - bat-arls-5:         [PASS][2] -> [ABORT][3] ([i915#12061])
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15804/bat-arls-5/igt@i915_selftest@live@workarounds.html
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142266v1/bat-arls-5/igt@i915_selftest@live@workarounds.html
    - bat-mtlp-6:         [PASS][4] -> [ABORT][5] ([i915#12061]) +1 other test abort
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15804/bat-mtlp-6/igt@i915_selftest@live@workarounds.html
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142266v1/bat-mtlp-6/igt@i915_selftest@live@workarounds.html

  * igt@kms_pipe_crc_basic@read-crc-frame-sequence:
    - bat-dg2-11:         [PASS][6] -> [SKIP][7] ([i915#9197])
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15804/bat-dg2-11/igt@kms_pipe_crc_basic@read-crc-frame-sequence.html
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142266v1/bat-dg2-11/igt@kms_pipe_crc_basic@read-crc-frame-sequence.html

  
#### Possible fixes ####

  * igt@i915_pm_rpm@module-reload:
    - bat-dg2-11:         [FAIL][8] ([i915#12903]) -> [PASS][9]
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15804/bat-dg2-11/igt@i915_pm_rpm@module-reload.html
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142266v1/bat-dg2-11/igt@i915_pm_rpm@module-reload.html
    - bat-dg1-7:          [FAIL][10] ([i915#12903]) -> [PASS][11]
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15804/bat-dg1-7/igt@i915_pm_rpm@module-reload.html
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142266v1/bat-dg1-7/igt@i915_pm_rpm@module-reload.html

  * igt@i915_selftest@live@workarounds:
    - {bat-mtlp-9}:       [ABORT][12] ([i915#12061]) -> [PASS][13] +1 other test pass
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15804/bat-mtlp-9/igt@i915_selftest@live@workarounds.html
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142266v1/bat-mtlp-9/igt@i915_selftest@live@workarounds.html

  
  {name}: This element is suppressed. This means it is ignored when computing
          the status of the difference (SUCCESS, WARNING, or FAILURE).

  [i915#12061]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12061
  [i915#12903]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12903
  [i915#9197]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/9197


Build changes
-------------

  * Linux: CI_DRM_15804 -> Patchwork_142266v1

  CI-20190529: 20190529
  CI_DRM_15804: 21f6d267c4a3e02cd75de5641f8786e427ce8fa8 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_8143: 515351a02a01f459212197f47ff91053551dcfab @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
  Patchwork_142266v1: 21f6d267c4a3e02cd75de5641f8786e427ce8fa8 @ git://anongit.freedesktop.org/gfx-ci/linux

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142266v1/index.html

Re: [PATCH] drm/i915: Update license on selftest lists to MIT

[Rodrigo Vivi]

On Sun, Dec 08, 2024 at 03:17:27PM +0000, Ed Maste wrote:
> These two files (i915_live_selftests.h, i915_mock_selftests.h) were
> introduced in commit 953c7f82eb89 ("drm/i915: Provide a hook for
> selftests") and are effectively just a list of selftests.
> 
> The selftest implementation itself is largely in i915_selftest.c, and
> uses a MIT license.  Graphics drivers are shared with other operating
> systems and have long used a permissive license (or dual license) to
> facilitate this.
> 
> The two selftest list files carried no license when introduced in
> 953c7f82eb89, presumably as they were considered trivial.  Notably the
> general selftest header i915_selftest.h (which does have non-trivial
> content) also has an MIT license.
> 
> The GPL-2.0 SPDX tag in these two files came from b24413180f56, where
> Greg Kroah-Hartman added the tag to all files that had no license.  This
> makes sense in general, but it is clear from the context of the original
> selftest commit here that these files are a trivial part of an otherwise
> MIT-licensed patch to a MIT-licensed component, and should have an MIT
> license.
> 

Cc: Chris Wilson <chris@chris-wilson.co.uk>

Chris, do you confirm your intention of MIT and not GPL for these files?

Cc: Greg KH <gregkh@linuxfoundation.org>

Greg, is this acceptable?

> Signed-off-by: Ed Maste <emaste@FreeBSD.org>
> ---
>  drivers/gpu/drm/i915/selftests/i915_live_selftests.h | 2 +-
>  drivers/gpu/drm/i915/selftests/i915_mock_selftests.h | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/selftests/i915_live_selftests.h b/drivers/gpu/drm/i915/selftests/i915_live_selftests.h
> index 5aee6c9a8295..264b951c4f77 100644
> --- a/drivers/gpu/drm/i915/selftests/i915_live_selftests.h
> +++ b/drivers/gpu/drm/i915/selftests/i915_live_selftests.h
> @@ -1,4 +1,4 @@
> -/* SPDX-License-Identifier: GPL-2.0 */
> +/* SPDX-License-Identifier: MIT */
>  
>  #ifndef selftest
>  #define selftest(x, y)
> diff --git a/drivers/gpu/drm/i915/selftests/i915_mock_selftests.h b/drivers/gpu/drm/i915/selftests/i915_mock_selftests.h
> index 0c22e0fc9059..18935e311748 100644
> --- a/drivers/gpu/drm/i915/selftests/i915_mock_selftests.h
> +++ b/drivers/gpu/drm/i915/selftests/i915_mock_selftests.h
> @@ -1,4 +1,4 @@
> -/* SPDX-License-Identifier: GPL-2.0 */
> +/* SPDX-License-Identifier: MIT */
>  
>  #ifndef selftest
>  #define selftest(x, y)
> -- 
> 2.46.2
> 

Re: [PATCH] drm/i915: Update license on selftest lists to MIT

[Greg KH]

On Tue, Dec 10, 2024 at 06:19:00PM -0500, Rodrigo Vivi wrote:
> On Sun, Dec 08, 2024 at 03:17:27PM +0000, Ed Maste wrote:
> > These two files (i915_live_selftests.h, i915_mock_selftests.h) were
> > introduced in commit 953c7f82eb89 ("drm/i915: Provide a hook for
> > selftests") and are effectively just a list of selftests.
> > 
> > The selftest implementation itself is largely in i915_selftest.c, and
> > uses a MIT license.  Graphics drivers are shared with other operating
> > systems and have long used a permissive license (or dual license) to
> > facilitate this.
> > 
> > The two selftest list files carried no license when introduced in
> > 953c7f82eb89, presumably as they were considered trivial.  Notably the
> > general selftest header i915_selftest.h (which does have non-trivial
> > content) also has an MIT license.
> > 
> > The GPL-2.0 SPDX tag in these two files came from b24413180f56, where
> > Greg Kroah-Hartman added the tag to all files that had no license.  This
> > makes sense in general, but it is clear from the context of the original
> > selftest commit here that these files are a trivial part of an otherwise
> > MIT-licensed patch to a MIT-licensed component, and should have an MIT
> > license.

No, that is not clear, by default, anything without a license gets the
GPL2 license as that is the license of the entire body of code.

> Cc: Chris Wilson <chris@chris-wilson.co.uk>
> 
> Chris, do you confirm your intention of MIT and not GPL for these files?
> 
> Cc: Greg KH <gregkh@linuxfoundation.org>
> 
> Greg, is this acceptable?

You need to get a signed-off-by from everyone who has touched these
files, which is not what you have done here :(

good luck!

greg k-h

Re: [PATCH] drm/i915: Update license on selftest lists to MIT

[Ed Maste]

On Wed, 11 Dec 2024 at 02:25, Greg KH <gregkh@linuxfoundation.org> wrote:
>
> On Tue, Dec 10, 2024 at 06:19:00PM -0500, Rodrigo Vivi wrote:
> > On Sun, Dec 08, 2024 at 03:17:27PM +0000, Ed Maste wrote:
> > > These two files (i915_live_selftests.h, i915_mock_selftests.h) were
> > > introduced in commit 953c7f82eb89 ("drm/i915: Provide a hook for
> > > selftests") and are effectively just a list of selftests.
> > >
> > > The selftest implementation itself is largely in i915_selftest.c, and
> > > uses a MIT license.  Graphics drivers are shared with other operating
> > > systems and have long used a permissive license (or dual license) to
> > > facilitate this.
> > >
> > > The two selftest list files carried no license when introduced in
> > > 953c7f82eb89, presumably as they were considered trivial.  Notably the
> > > general selftest header i915_selftest.h (which does have non-trivial
> > > content) also has an MIT license.
> > >
> > > The GPL-2.0 SPDX tag in these two files came from b24413180f56, where
> > > Greg Kroah-Hartman added the tag to all files that had no license.  This
> > > makes sense in general, but it is clear from the context of the original
> > > selftest commit here that these files are a trivial part of an otherwise
> > > MIT-licensed patch to a MIT-licensed component, and should have an MIT
> > > license.
>
> No, that is not clear, by default, anything without a license gets the
> GPL2 license as that is the license of the entire body of code.

I mean the intent is clear from looking at the original patch in 953c7f82eb89:
- drivers/gpu/drm/i915 is generally MIT licensed
- 953c7f82eb89 added these two trivial files with no license text
- that patch also added nontrivial new files with an MIT license

The same sort of issue affected drm/radeon, fixed in b7019ac550eb:

    drm/radeon: fix incorrrect SPDX-License-Identifiers

    radeon is MIT.  This were incorrectly changed in
    commit b24413180f56 ("License cleanup: add SPDX GPL-2.0 license
identifier to files with no license")
    and
    commit d198b34f3855 (".gitignore: add SPDX License Identifier")
    and:
    commit ec8f24b7faaf ("treewide: Add SPDX license identifier -
Makefile/Kconfig")

    Fixes: d198b34f3855 (".gitignore: add SPDX License Identifier")
    Fixes: ec8f24b7faaf ("treewide: Add SPDX license identifier -
Makefile/Kconfig")
    Fixes: b24413180f56 ("License cleanup: add SPDX GPL-2.0 license
identifier to files with no license")
    Bug: https://gitlab.freedesktop.org/drm/amd/-/issues/2053
    Reviewed-by: Christian König <christian.koenig@amd.com>
    Signed-off-by: Alex Deucher <alexander.deucher@amd.com>

and drm/noveau, fixed in b7019ac550eb:

    drm/nouveau: fix bogus GPL-2 license header

    The bulk SPDX addition made all these files into GPL-2.0 licensed files.
    However the remainder of the project is MIT-licensed, these files
    (primarily header files) were simply missing the boiler plate and got
    caught up in the global update.

    Fixes: b24413180f5 (License cleanup: add SPDX GPL-2.0 license
identifier to files with no license)
    Signed-off-by: Ilia Mirkin <imirkin@alum.mit.edu>
    Acked-by: Emil Velikov <emil.l.velikov@gmail.com>
    Acked-by: Karol Herbst <kherbst@redhat.com>
    Signed-off-by: Ben Skeggs <bskeggs@redhat.com>

> > Chris, do you confirm your intention of MIT and not GPL for these files?
> >
> > Cc: Greg KH <gregkh@linuxfoundation.org>
> >
> > Greg, is this acceptable?
>
> You need to get a signed-off-by from everyone who has touched these
> files, which is not what you have done here :(

Most of the contributions to these files were done by 13 Intel
employees. Presumably a signed-off-by from someone at Intel is
sufficient for all of them? I've CC'd other contributors to these two
files for a signed-off-by.

Re: [PATCH] drm/i915: Update license on selftest lists to MIT

[Greg KH]

On Thu, Dec 12, 2024 at 10:10:54AM -0500, Ed Maste wrote:
> On Wed, 11 Dec 2024 at 02:25, Greg KH <gregkh@linuxfoundation.org> wrote:
> >
> > On Tue, Dec 10, 2024 at 06:19:00PM -0500, Rodrigo Vivi wrote:
> > > On Sun, Dec 08, 2024 at 03:17:27PM +0000, Ed Maste wrote:
> > > > These two files (i915_live_selftests.h, i915_mock_selftests.h) were
> > > > introduced in commit 953c7f82eb89 ("drm/i915: Provide a hook for
> > > > selftests") and are effectively just a list of selftests.
> > > >
> > > > The selftest implementation itself is largely in i915_selftest.c, and
> > > > uses a MIT license.  Graphics drivers are shared with other operating
> > > > systems and have long used a permissive license (or dual license) to
> > > > facilitate this.
> > > >
> > > > The two selftest list files carried no license when introduced in
> > > > 953c7f82eb89, presumably as they were considered trivial.  Notably the
> > > > general selftest header i915_selftest.h (which does have non-trivial
> > > > content) also has an MIT license.
> > > >
> > > > The GPL-2.0 SPDX tag in these two files came from b24413180f56, where
> > > > Greg Kroah-Hartman added the tag to all files that had no license.  This
> > > > makes sense in general, but it is clear from the context of the original
> > > > selftest commit here that these files are a trivial part of an otherwise
> > > > MIT-licensed patch to a MIT-licensed component, and should have an MIT
> > > > license.
> >
> > No, that is not clear, by default, anything without a license gets the
> > GPL2 license as that is the license of the entire body of code.
> 
> I mean the intent is clear from looking at the original patch in 953c7f82eb89:
> - drivers/gpu/drm/i915 is generally MIT licensed
> - 953c7f82eb89 added these two trivial files with no license text
> - that patch also added nontrivial new files with an MIT license
> 
> The same sort of issue affected drm/radeon, fixed in b7019ac550eb:
> 
>     drm/radeon: fix incorrrect SPDX-License-Identifiers
> 
>     radeon is MIT.  This were incorrectly changed in
>     commit b24413180f56 ("License cleanup: add SPDX GPL-2.0 license
> identifier to files with no license")
>     and
>     commit d198b34f3855 (".gitignore: add SPDX License Identifier")
>     and:
>     commit ec8f24b7faaf ("treewide: Add SPDX license identifier -
> Makefile/Kconfig")
> 
>     Fixes: d198b34f3855 (".gitignore: add SPDX License Identifier")
>     Fixes: ec8f24b7faaf ("treewide: Add SPDX license identifier -
> Makefile/Kconfig")
>     Fixes: b24413180f56 ("License cleanup: add SPDX GPL-2.0 license
> identifier to files with no license")
>     Bug: https://gitlab.freedesktop.org/drm/amd/-/issues/2053
>     Reviewed-by: Christian König <christian.koenig@amd.com>
>     Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
> 
> and drm/noveau, fixed in b7019ac550eb:
> 
>     drm/nouveau: fix bogus GPL-2 license header
> 
>     The bulk SPDX addition made all these files into GPL-2.0 licensed files.
>     However the remainder of the project is MIT-licensed, these files
>     (primarily header files) were simply missing the boiler plate and got
>     caught up in the global update.
> 
>     Fixes: b24413180f5 (License cleanup: add SPDX GPL-2.0 license
> identifier to files with no license)
>     Signed-off-by: Ilia Mirkin <imirkin@alum.mit.edu>
>     Acked-by: Emil Velikov <emil.l.velikov@gmail.com>
>     Acked-by: Karol Herbst <kherbst@redhat.com>
>     Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
> 
> > > Chris, do you confirm your intention of MIT and not GPL for these files?
> > >
> > > Cc: Greg KH <gregkh@linuxfoundation.org>
> > >
> > > Greg, is this acceptable?
> >
> > You need to get a signed-off-by from everyone who has touched these
> > files, which is not what you have done here :(
> 
> Most of the contributions to these files were done by 13 Intel
> employees. Presumably a signed-off-by from someone at Intel is
> sufficient for all of them? I've CC'd other contributors to these two
> files for a signed-off-by.

Yes, if you get a signed-off-by from a lawyer at Intel, that should
cover their contributions, but you also need it for the other
contributors as well.

Good luck!

greg k-h