From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59138 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725846AbgFQIZd (ORCPT ); Wed, 17 Jun 2020 04:25:33 -0400 Date: Wed, 17 Jun 2020 10:24:59 +0200 From: Peter Zijlstra Subject: Re: [v3 PATCH 1/2] lockdep: Split header file into lockdep and lockdep_types Message-ID: <20200617082459.GC2531@hirez.programming.kicks-ass.net> References: <20200617071524.GA3055@gondor.apana.org.au> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: linux-s390-owner@vger.kernel.org List-ID: To: Herbert Xu Cc: Petr Mladek , Sergey Senozhatsky , Steven Rostedt , Linux Kernel Mailing List , Ingo Molnar , Will Deacon , Heiko Carstens , Vasily Gorbik , Christian Borntraeger , linux-s390@vger.kernel.org On Wed, Jun 17, 2020 at 05:17:19PM +1000, Herbert Xu wrote: > There is a header file inclusion loop between asm-generic/bug.h > and linux/kernel.h. This causes potential compile failurs depending > on the which file is included first. One way of breaking this loop > is to stop spinlock_types.h from including lockdep.h. This patch > splits lockdep.h into two files for this purpose. > > Signed-off-by: Herbert Xu > Reviewed-by: Andy Shevchenko > Reviewed-by: Sergey Senozhatsky > Acked-by: Petr Mladek > Acked-by: Steven Rostedt (VMware) Looks good. Petr, how about I stick this in tip/locking/header for us both to share?