From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S964847AbcALOyO (ORCPT ); Tue, 12 Jan 2016 09:54:14 -0500 Received: from mail.kernel.org ([198.145.29.136]:56300 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757488AbcALOyK (ORCPT ); Tue, 12 Jan 2016 09:54:10 -0500 Date: Tue, 12 Jan 2016 11:54:06 -0300 From: Arnaldo Carvalho de Melo To: Borislav Petkov Cc: Josh Poimboeuf , Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , x86@kernel.org, linux-kernel@vger.kernel.org, live-patching@vger.kernel.org, Michal Marek , Peter Zijlstra , Andy Lutomirski , Linus Torvalds , Andi Kleen , Pedro Alves , Namhyung Kim , Bernd Petrovitsch , Chris J Arges , Andrew Morton , Jiri Slaby Subject: Re: [PATCH v15 02/25] tools: Make list.h self-sufficient Message-ID: <20160112145406.GZ18367@kernel.org> References: <59cdc19c6589d1b5ef43d83b0e2d5a4a40301374.1450442274.git.jpoimboe@redhat.com> <20160112123540.GA22699@pd.tnic> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20160112123540.GA22699@pd.tnic> X-Url: http://acmel.wordpress.com User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Em Tue, Jan 12, 2016 at 01:35:40PM +0100, Borislav Petkov escreveu: > On Fri, Dec 18, 2015 at 06:39:16AM -0600, Josh Poimboeuf wrote: > > Commit 7f5f873c6a07 added the use of the WRITE_ONCE macro to the kernel > > nitpick: we like to do this commit quoting style: > > 7f5f873c6a07 ("rculist: Use WRITE_ONCE() when deleting from reader-visible list") > > Checkpatch catches that: > > ERROR: Please use git commit description style 'commit <12+ chars of sha1> ("")' - ie: 'Commit 7f5f873c6a07 ("rculist: Use WRITE_ONCE() when deleting from reader-visible list")' > #21: > Commit 7f5f873c6a07 added the use of the WRITE_ONCE macro to the kernel I'm fixing this up and taking this patch, ok? - Arnaldo > > version of list.h, which broke the stacktool build because the tools > > list.h includes the kernel list.h. > > > > Avoid this type of situation in the future and make list.h > > self-sufficient by copying the kernel list.h routines directly into > > tools list.h. > > > > This is a straight copy except for adjustments to the include statements > > and copying of the tools-specific list routines (list_del_range and > > list_for_each_from). > > > > Signed-off-by: Josh Poimboeuf <jpoimboe@redhat.com> > > --- > > tools/include/linux/list.h | 753 ++++++++++++++++++++++++++++++++++++++++++++- > > 1 file changed, 747 insertions(+), 6 deletions(-) > > ... > > -- > Regards/Gruss, > Boris. > > ECO tip #101: Trim your mails when you reply.