2023-10-10 14:28:00

by Thomas Hellström

[permalink] [raw]
Subject: [PATCH] drm/gpuvm: Dual-licence the drm_gpuvm code GPL-2.0 OR MIT

Dual-licence in order to make it possible for other non-GPL os'es
to re-implement the code. The use of EXPORT_SYMBOL_GPL() is intentionally
left untouched to prevent use of drm_gpuvm as a proxy for non-GPL drivers
to access GPL-only kernel symbols.

Much of the ideas and algorithms used in the drm_gpuvm code is already
present in one way or another in MIT-licensed code.

Cc: Danilo Krummrich <[email protected]>
Cc: [email protected]
Cc: [email protected]
Cc: [email protected]
Signed-off-by: Thomas Hellström <[email protected]>
---
drivers/gpu/drm/drm_gpuvm.c | 2 +-
include/drm/drm_gpuvm.h | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/drm_gpuvm.c b/drivers/gpu/drm/drm_gpuvm.c
index 02ce6baacdad..08c088319652 100644
--- a/drivers/gpu/drm/drm_gpuvm.c
+++ b/drivers/gpu/drm/drm_gpuvm.c
@@ -1,4 +1,4 @@
-// SPDX-License-Identifier: GPL-2.0-only
+// SPDX-License-Identifier: GPL-2.0 OR MIT
/*
* Copyright (c) 2022 Red Hat.
*
diff --git a/include/drm/drm_gpuvm.h b/include/drm/drm_gpuvm.h
index 361fea5cb849..21bbf11415b3 100644
--- a/include/drm/drm_gpuvm.h
+++ b/include/drm/drm_gpuvm.h
@@ -1,4 +1,4 @@
-/* SPDX-License-Identifier: GPL-2.0-only */
+/* SPDX-License-Identifier: GPL-2.0 OR MIT */

#ifndef __DRM_GPUVM_H__
#define __DRM_GPUVM_H__
--
2.41.0


2023-10-17 07:55:56

by Danilo Krummrich

[permalink] [raw]
Subject: Re: [PATCH] drm/gpuvm: Dual-licence the drm_gpuvm code GPL-2.0 OR MIT

On Tue, Oct 10, 2023 at 04:27:24PM +0200, Thomas Hellstr?m wrote:
> Dual-licence in order to make it possible for other non-GPL os'es
> to re-implement the code. The use of EXPORT_SYMBOL_GPL() is intentionally
> left untouched to prevent use of drm_gpuvm as a proxy for non-GPL drivers
> to access GPL-only kernel symbols.
>
> Much of the ideas and algorithms used in the drm_gpuvm code is already
> present in one way or another in MIT-licensed code.
>
> Cc: Danilo Krummrich <[email protected]>
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Signed-off-by: Thomas Hellstr?m <[email protected]>

Acked-by: Danilo Krummrich <[email protected]>

> ---
> drivers/gpu/drm/drm_gpuvm.c | 2 +-
> include/drm/drm_gpuvm.h | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/drm_gpuvm.c b/drivers/gpu/drm/drm_gpuvm.c
> index 02ce6baacdad..08c088319652 100644
> --- a/drivers/gpu/drm/drm_gpuvm.c
> +++ b/drivers/gpu/drm/drm_gpuvm.c
> @@ -1,4 +1,4 @@
> -// SPDX-License-Identifier: GPL-2.0-only
> +// SPDX-License-Identifier: GPL-2.0 OR MIT
> /*
> * Copyright (c) 2022 Red Hat.
> *
> diff --git a/include/drm/drm_gpuvm.h b/include/drm/drm_gpuvm.h
> index 361fea5cb849..21bbf11415b3 100644
> --- a/include/drm/drm_gpuvm.h
> +++ b/include/drm/drm_gpuvm.h
> @@ -1,4 +1,4 @@
> -/* SPDX-License-Identifier: GPL-2.0-only */
> +/* SPDX-License-Identifier: GPL-2.0 OR MIT */
>
> #ifndef __DRM_GPUVM_H__
> #define __DRM_GPUVM_H__
> --
> 2.41.0
>

2023-10-17 08:15:13

by Thomas Hellström

[permalink] [raw]
Subject: Re: [PATCH] drm/gpuvm: Dual-licence the drm_gpuvm code GPL-2.0 OR MIT


On 10/17/23 09:54, Danilo Krummrich wrote:
> On Tue, Oct 10, 2023 at 04:27:24PM +0200, Thomas Hellström wrote:
>> Dual-licence in order to make it possible for other non-GPL os'es
>> to re-implement the code. The use of EXPORT_SYMBOL_GPL() is intentionally
>> left untouched to prevent use of drm_gpuvm as a proxy for non-GPL drivers
>> to access GPL-only kernel symbols.
>>
>> Much of the ideas and algorithms used in the drm_gpuvm code is already
>> present in one way or another in MIT-licensed code.
>>
>> Cc: Danilo Krummrich <[email protected]>
>> Cc: [email protected]
>> Cc: [email protected]
>> Cc: [email protected]
>> Signed-off-by: Thomas Hellström <[email protected]>
> Acked-by: Danilo Krummrich <[email protected]>

Thanks, Danilo

Thomas


>
>> ---
>> drivers/gpu/drm/drm_gpuvm.c | 2 +-
>> include/drm/drm_gpuvm.h | 2 +-
>> 2 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/drm_gpuvm.c b/drivers/gpu/drm/drm_gpuvm.c
>> index 02ce6baacdad..08c088319652 100644
>> --- a/drivers/gpu/drm/drm_gpuvm.c
>> +++ b/drivers/gpu/drm/drm_gpuvm.c
>> @@ -1,4 +1,4 @@
>> -// SPDX-License-Identifier: GPL-2.0-only
>> +// SPDX-License-Identifier: GPL-2.0 OR MIT
>> /*
>> * Copyright (c) 2022 Red Hat.
>> *
>> diff --git a/include/drm/drm_gpuvm.h b/include/drm/drm_gpuvm.h
>> index 361fea5cb849..21bbf11415b3 100644
>> --- a/include/drm/drm_gpuvm.h
>> +++ b/include/drm/drm_gpuvm.h
>> @@ -1,4 +1,4 @@
>> -/* SPDX-License-Identifier: GPL-2.0-only */
>> +/* SPDX-License-Identifier: GPL-2.0 OR MIT */
>>
>> #ifndef __DRM_GPUVM_H__
>> #define __DRM_GPUVM_H__
>> --
>> 2.41.0
>>

2023-10-17 08:49:11

by Francois Dugast

[permalink] [raw]
Subject: Re: [Intel-xe] [PATCH] drm/gpuvm: Dual-licence the drm_gpuvm code GPL-2.0 OR MIT

On Tue, Oct 17, 2023 at 10:14:59AM +0200, Thomas Hellstr?m wrote:
>
> On 10/17/23 09:54, Danilo Krummrich wrote:
> > On Tue, Oct 10, 2023 at 04:27:24PM +0200, Thomas Hellstr?m wrote:
> > > Dual-licence in order to make it possible for other non-GPL os'es
> > > to re-implement the code. The use of EXPORT_SYMBOL_GPL() is intentionally
> > > left untouched to prevent use of drm_gpuvm as a proxy for non-GPL drivers
> > > to access GPL-only kernel symbols.
> > >
> > > Much of the ideas and algorithms used in the drm_gpuvm code is already
> > > present in one way or another in MIT-licensed code.
> > >
> > > Cc: Danilo Krummrich <[email protected]>
> > > Cc: [email protected]
> > > Cc: [email protected]
> > > Cc: [email protected]
> > > Signed-off-by: Thomas Hellstr?m <[email protected]>
> > Acked-by: Danilo Krummrich <[email protected]>

As this was acked by author:

Reviewed-by: Francois Dugast <[email protected]>

>
> Thanks, Danilo
>
> Thomas
>
>
> >
> > > ---
> > > drivers/gpu/drm/drm_gpuvm.c | 2 +-
> > > include/drm/drm_gpuvm.h | 2 +-
> > > 2 files changed, 2 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/drivers/gpu/drm/drm_gpuvm.c b/drivers/gpu/drm/drm_gpuvm.c
> > > index 02ce6baacdad..08c088319652 100644
> > > --- a/drivers/gpu/drm/drm_gpuvm.c
> > > +++ b/drivers/gpu/drm/drm_gpuvm.c
> > > @@ -1,4 +1,4 @@
> > > -// SPDX-License-Identifier: GPL-2.0-only
> > > +// SPDX-License-Identifier: GPL-2.0 OR MIT
> > > /*
> > > * Copyright (c) 2022 Red Hat.
> > > *
> > > diff --git a/include/drm/drm_gpuvm.h b/include/drm/drm_gpuvm.h
> > > index 361fea5cb849..21bbf11415b3 100644
> > > --- a/include/drm/drm_gpuvm.h
> > > +++ b/include/drm/drm_gpuvm.h
> > > @@ -1,4 +1,4 @@
> > > -/* SPDX-License-Identifier: GPL-2.0-only */
> > > +/* SPDX-License-Identifier: GPL-2.0 OR MIT */
> > > #ifndef __DRM_GPUVM_H__
> > > #define __DRM_GPUVM_H__
> > > --
> > > 2.41.0
> > >