From mboxrd@z Thu Jan 1 00:00:00 1970 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751534AbeAPXGf (ORCPT + 1 other); Tue, 16 Jan 2018 18:06:35 -0500 Received: from mx1.redhat.com ([209.132.183.28]:49704 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750892AbeAPXGe (ORCPT ); Tue, 16 Jan 2018 18:06:34 -0500 Date: Tue, 16 Jan 2018 17:06:29 -0600 From: Josh Poimboeuf To: Borislav Petkov Cc: Peter Zijlstra , David Woodhouse , linux-kernel@vger.kernel.org, Dave Hansen , Ashok Raj , Thomas Gleixner , Tim Chen , Andy Lutomirski , Linus Torvalds , Greg KH , Andrea Arcangeli , Andi Kleen , Arjan Van De Ven , Dan Williams , Paolo Bonzini , Jun Nakajima , Asit Mallick Subject: Re: [PATCH v2 03/10] x86: Add a type field to alt_instr Message-ID: <20180116230629.ods5wult5ejv7lwa@treble> References: <20180116142825.376986833@infradead.org> <20180116143241.063773295@infradead.org> <20180116224955.zwfqr74agcy6qymw@treble> <20180116225325.pnjepmfeml3yvvgw@pd.tnic> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20180116225325.pnjepmfeml3yvvgw@pd.tnic> User-Agent: Mutt/1.6.0.1 (2016-04-01) X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.27]); Tue, 16 Jan 2018 23:06:34 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Return-Path: On Tue, Jan 16, 2018 at 11:53:25PM +0100, Borislav Petkov wrote: > On Tue, Jan 16, 2018 at 04:49:55PM -0600, Josh Poimboeuf wrote: > > Unfortunately it's against policy to include kernel headers from the > > tools subdirectory, so we generally have to either hard code things like > > this, or have a duplicated header file which needs to be kept in sync. > > Yeah, and AFAIR, perf tool guys already have machinery that does check > whether stuff went out of sync and they resync the required definitions. > > You could reuse that... We could, but it can be a real PITA to keep those files in sync, as they often change for minor changes which don't even matter. So it might not be worth the trouble. Objtool seems to complain when the alternatives struct size changes anyway, so it's probably not a big deal. -- Josh