Received: by 10.223.185.116 with SMTP id b49csp8393457wrg; Fri, 2 Mar 2018 00:33:43 -0800 (PST) X-Google-Smtp-Source: AG47ELtahIURxb3p2Ya6NkkOke4NGnUwYrHrmy0dSfGfoYxy2g5K6B2Mno3X9AQaUxiwb1Y3MCn6 X-Received: by 10.99.182.1 with SMTP id j1mr3982955pgf.62.1519979622941; Fri, 02 Mar 2018 00:33:42 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1519979622; cv=none; d=google.com; s=arc-20160816; b=m+bTQu8fF+RZlZ+lxuaElELw5I3B0DNFtwIYASupdpbqG3smaiv3abJrqwXzbN31J1 UNMpnjF2nHRwLuTYqadF6FPDVlBy4aiqmy9qyDsnFCcSdJpoI4jXAd7azDs2TiJKC/5/ NwwJHGUp8VewI8xZLyvBKA3mgepNmwZ9TWk1F0sMVtsp9e0DQAGn9rk8ORcioHzxZU93 hS2qyZgUl8OWAvHUo3bpHuoKpv9nqxwQQXM5ePHrHwSuJMVAgkpCwFQvmyO/WD5OzMnO bXzkZH6QaqaMYNUVC1NJ1iXQYUCEsnlmspLJyszz89EG41p7WQ4Wym4ksMEoUXBckCFd h/qA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:arc-authentication-results; bh=83HdTM1sod/R0qLMMnJSzvjmAhFnldoQvjMK+hHDBGg=; b=mTYsTGzaL4mm/ZSWPOEUhVZNuSl3lpNOOwJjft3O3tr/wFHGBHqt8V0+Xbvm4iUVPa PySkmYCgfVzJ7wU8aswlnBl5p8MdnUercFkr/ZPqS8egJnMMeuocCHQGm918UNf30N3R tsEtyyrB+djyzI42fmbWA5CghTxB5BTbaulz1FS4O2J53pns40RtZvQOujetSOlLYH9/ ImZbGR89Kj3Z/i8I/ofiwC0d/N2jLWqaU1rgi6MgCypn/vleMqZvwhVd0SaQaG8Wk1BL jqBXhnnaXlnRaChBg6rKEIFUby6pK2AlOo1bFp0/KrlUMYM0uCimLYZ9wzVp1uS0Vllf WY+A== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id c10si3710387pgf.582.2018.03.02.00.33.28; Fri, 02 Mar 2018 00:33:42 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1422953AbeCBIbh (ORCPT + 99 others); Fri, 2 Mar 2018 03:31:37 -0500 Received: from mail.linuxfoundation.org ([140.211.169.12]:43448 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1422873AbeCBIbd (ORCPT ); Fri, 2 Mar 2018 03:31:33 -0500 Received: from localhost (clnet-b04-243.ikbnet.co.at [83.175.124.243]) by mail.linuxfoundation.org (Postfix) with ESMTPSA id 230B9118E; Fri, 2 Mar 2018 08:31:32 +0000 (UTC) Date: Fri, 2 Mar 2018 09:31:32 +0100 From: Greg Kroah-Hartman To: Philippe Ombredanne Cc: Miroslav Benes , Joe Lawrence , live-patching@vger.kernel.org, LKML , Josh Poimboeuf , Jessica Yu , Jiri Kosina , Petr Mladek , Jason Baron , Evgenii Shatokhin Subject: Re: [PATCH v0 1/3] livepatch: add sample cumulative patch Message-ID: <20180302083132.GA9636@kroah.com> References: <1519421630-12025-1-git-send-email-joe.lawrence@redhat.com> <1519421630-12025-2-git-send-email-joe.lawrence@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.9.3 (2018-01-21) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Mar 01, 2018 at 05:19:28PM -0800, Philippe Ombredanne wrote: > Miroslav, > > On Tue, Feb 27, 2018 at 3:54 AM, Miroslav Benes wrote: > > On Sat, 24 Feb 2018, Philippe Ombredanne wrote: > > > >> Joe, > >> > >> On Fri, Feb 23, 2018 at 1:33 PM, Joe Lawrence wrote: > >> > Add a simple atomic replace / cumulative livepatch example. > >> > > >> > Signed-off-by: Joe Lawrence > >> > --- > >> > samples/livepatch/Makefile | 1 + > >> > samples/livepatch/livepatch-cumulative.c | 216 +++++++++++++++++++++++++++++++ > >> > 2 files changed, 217 insertions(+) > >> > create mode 100644 samples/livepatch/livepatch-cumulative.c > >> > > >> > diff --git a/samples/livepatch/Makefile b/samples/livepatch/Makefile > >> > index 2472ce39a18d..dd0e2a8af1af 100644 > >> > --- a/samples/livepatch/Makefile > >> > +++ b/samples/livepatch/Makefile > >> > @@ -5,3 +5,4 @@ obj-$(CONFIG_SAMPLE_LIVEPATCH) += livepatch-shadow-fix2.o > >> > obj-$(CONFIG_SAMPLE_LIVEPATCH) += livepatch-callbacks-demo.o > >> > obj-$(CONFIG_SAMPLE_LIVEPATCH) += livepatch-callbacks-mod.o > >> > obj-$(CONFIG_SAMPLE_LIVEPATCH) += livepatch-callbacks-busymod.o > >> > +obj-$(CONFIG_SAMPLE_LIVEPATCH) += livepatch-cumulative.o > >> > diff --git a/samples/livepatch/livepatch-cumulative.c b/samples/livepatch/livepatch-cumulative.c > >> > new file mode 100644 > >> > index 000000000000..ab036439e08c > >> > --- /dev/null > >> > +++ b/samples/livepatch/livepatch-cumulative.c > >> > @@ -0,0 +1,216 @@ > >> > +/* > >> > + * Copyright (C) 2018 Joe Lawrence > >> > + * > >> > + * This program is free software; you can redistribute it and/or > >> > + * modify it under the terms of the GNU General Public License > >> > + * as published by the Free Software Foundation; either version 2 > >> > + * of the License, or (at your option) any later version. > >> > + * > >> > + * This program is distributed in the hope that it will be useful, > >> > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > >> > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > >> > + * GNU General Public License for more details. > >> > + * > >> > + * You should have received a copy of the GNU General Public License > >> > + * along with this program; if not, see . > >> > + */ > >> > >> May be you could use the new SPDX tags instead of this fine but long > >> legalese? [1] > >> This would replace ~12 lines of comment by a single line with the same effect. > >> Thanks! > > > > I don't know about that. How come it is perceived as equivalent? I mean, > > we have a well-established way how to say that a particular source > > code/file is distributed with GPL license. Well-established means that > > it's been tested in court AFAIK many times. Even the license itself (found > > in COPYING file) mentions this as way how to attach the license to a file. > > > > Now you want it to be replaced with a tag. Does it say the same? It might. > > It might not. Do we know? Have you got a court ruling which would say that > > this is also a way how to attach a license to a file? I doubt it. It may > > seem trivially clear, but there are no such things in the legal world. > > > > Don't make me wrong. I don't like that copyright thingie much. I don't > > like that you can find even different versions of the text in the kernel > > source code (and not only there). > > > > However I'd prefer to leave at least a note there that the file is still > > distributed under the terms of GPL found in COPYING file. The tag can be > > there too, if it makes someone happy. > > > > Regards, > > Miroslav > > > >> [1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/license-rules.rst > >> -- > >> Philippe > >> > > > > To the best of my knowledge, this has been debated in person and on > list among maintainers and agreed to. > This has also been reviewed by the LF lawyers. The result of is the > documentation in [1] > You are welcomed not to agree of course, but this would make your > contributions stand out with its legalese boilerplate when we are > trying to get of it. > > Greg, anything else to add? Yes, do not add new "boiler plate" license code in new files, otherwise you will just have to rip them out again later on. If you have questions about this, please contact your company's lawyers, as they know all about this issue and last I heard, agreed with it. thanks, greg k-h