Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932712AbbFIKGO (ORCPT ); Tue, 9 Jun 2015 06:06:14 -0400 Received: from mail2-relais-roc.national.inria.fr ([192.134.164.83]:51322 "EHLO mail2-relais-roc.national.inria.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932175AbbFIKGH (ORCPT ); Tue, 9 Jun 2015 06:06:07 -0400 X-IronPort-AV: E=Sophos;i="5.13,579,1427752800"; d="scan'208";a="164119273" Date: Tue, 9 Jun 2015 12:05:57 +0200 (CEST) From: Julia Lawall X-X-Sender: jll@hadrien To: Paul Bolle cc: monstr@monstr.eu, Julia Lawall , Kedareswara rao Appana , dan.j.williams@intel.com, vinod.koul@intel.com, michal.simek@xilinx.com, soren.brinkmann@xilinx.com, appanad@xilinx.com, punnaia@xilinx.com, dmaengine@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Srikanth Thokala , Greg Kroah-Hartman , Joe Perches , Fengguang Wu Subject: Re: [PATCH v4 2/2] dma: Add Xilinx AXI Central Direct Memory Access Engine driver support In-Reply-To: <1433837720.16887.42.camel@x220> Message-ID: References: <1432228035-6569-1-git-send-email-appanad@xilinx.com> <1432290653.27695.97.camel@x220> <557681A7.7080202@monstr.eu> <55769132.6070601@monstr.eu> <1433837720.16887.42.camel@x220> User-Agent: Alpine 2.10 (DEB 1266 2009-07-14) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3458 Lines: 74 On Tue, 9 Jun 2015, Paul Bolle wrote: > On Tue, 2015-06-09 at 09:09 +0200, Michal Simek wrote: > > On 06/09/2015 08:10 AM, Julia Lawall wrote: > > > On Tue, 9 Jun 2015, Michal Simek wrote: > > >> Also sort of checking for this will be great. Julia? > > > > > > If this requires checking the contents of comment, Coccinelle currently > > > can't help with that. Perhaps an idea would be to just do a grep on the > > > file. So if I find MODULE_LICENSE("GPL v2") and then grepping for "either > > > vresion 2" gives success, then there is a problem? It's obviously not > > > foolproof, but perhaps it could be helpful. > > > > Having some sort of checking somewhere will be great. checkpatch? > > zero-day testing system? > > Mistakes I've seen made since I started checking this stuff (a few > months ago): > - typos in the license ident, say "GPLv2", "GPL V2", or "BSD": generates > a warning when module is loaded and taints kernel. People still get this > wrong. A test in checkpatch for these typos was submitted a while ago, > but it never got added; > - not adding MODULE_LICENSE() to a module: also generates a warning when > module is loaded and taints kernel. People still get this wrong; > - adding MODULE_LICENSE() to built-in only code: pointless at best, and > annoying for reviewers ("Hey, did the submitter intend to write built-in > only code or modular code?"); > - using "Dual BSD/GPL" but not a trace of the BSD license blurb in > sight, while adding that blurb is one of the very few requirements this > license actually has; > - license mismatch, say comment blurb states "GPL v2 (or later)" but > MODULE_LICENSE() ident states "GPL v2" only (or vice versa): very easy > mistake to make, happens once or twice a week. > > Did I miss anything in that list? > > I'm afraid that most of the above can only be caught reliably by > attention to detail by submitters and reviewers. That's a pity, because > checking for that stuff is about as boring as it gets. (What does that > say about me?) There has been some research in this direction: A sentence-matching method for automatic license identification of source code files Daniel M. German, Yuki Manabe, Katsuro Inoue The reuse of free and open source software (FOSS) components is becoming more prevalent. One of the major challenges in finding the right component is finding one that has a license that is e for its intended use. The license of a FOSS component is determined by the licenses of its source code files. In this paper, we describe the challenges of identifying the license under which source code is made available, and propose a sentence-based matching algorithm to automatically do it. We demonstrate the feasibility of our approach by implementing a tool named Ninka. We performed an evaluation that shows that Ninka outperforms other methods of license identification in precision and speed. We also performed an empirical study on 0.8 million source code files of Debian that highlight interesting facts about the manner in which licenses are used by FOSS. If you google for the paper title, you come across a pdf of the paper. There seems to be a tool available as well: http://ninka.turingmachine.org/ julia -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/