From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 603BFECAA27 for ; Thu, 25 Aug 2022 19:54:13 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=qT1X0LReBuc4dGg0BLxNXDYpzl91NCZSOzJQHw8f4Ak=; b=OrlNr3o+hLNHUe zS71gxWn0pUqLam7/zE83x33I4N0PaZn8wAPslz+v87QOLjW+Fh0ibF5zmjegeWECqKqN0AzGSPsA W059W2Z2s6idO8RH5/qcRi+BOB72khnAdhsztrHdPPdvxlg6YZS3UaqC6W85FgTxCxEBLH0c9wbGq uIzH3VzoURgpWg4JQN2IXFoGO6jZjHJb0UDou06U/6qGULX7I5K4zVqpqqZ5eBLAYqXNNowDH7S6a wUK5xt5/BDxUTIRMiIkLva/3LZgz43MHAF8LUDlYj52uu+mEBV4j4ISbVWmTFS24YyynUqRXw6kSs XJqQKHoavmAFPoBAmD5g==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oRIul-002svM-52; Thu, 25 Aug 2022 19:53:19 +0000 Received: from desiato.infradead.org ([2001:8b0:10b:1:d65d:64ff:fe57:4e05]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1oRIui-002su2-7J for linux-arm-kernel@bombadil.infradead.org; Thu, 25 Aug 2022 19:53:16 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=desiato.20200630; h=In-Reply-To:Content-Type:MIME-Version: References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description; bh=FplKOFIaq0IDqpTmW3uF0ML683YxEl+mwa+e6onT/GA=; b=hcWu91k3r6ioB5Idrxyfd+srLp AwbRVrMeoXAsaxAqdsjn+8HQN02yVxai4WQucwlbk57xEzjDgZm+de5FSs0IryNeVMT0zGqHIgKD3 jIxdHWJ9JY9rctLTXWQsYn797ZrQfhvUhjboUk0QlnIDPTar4f/EjlKFTEExAfcfUzsPHRor4i74+ tNKvQEurNr//MWV1GmiiT2chdUX9k4/UKRbIlfTUBZOY4/OSpfNSqYPJBF57OmeK0rG1u/iqX0akv axeNQ/xYst3mCF+wR4F3HzZosjqIhVLzzEnWLTqcwgYS7seWwAPkRKgxAUUc94fXmSNQ6Y/LJ48j1 Te0LgBWg==; Received: from [187.19.239.32] (helo=quaco.ghostprotocols.net) by desiato.infradead.org with esmtpsa (Exim 4.94.2 #2 (Red Hat Linux)) id 1oRIug-0065vz-6M; Thu, 25 Aug 2022 19:53:14 +0000 Received: by quaco.ghostprotocols.net (Postfix, from userid 1000) id 6BA87404A1; Thu, 25 Aug 2022 16:53:11 -0300 (-03) Date: Thu, 25 Aug 2022 16:53:11 -0300 From: Arnaldo Carvalho de Melo To: Rob Herring Cc: Alexander Shishkin , Ingo Molnar , Catalin Marinas , Peter Zijlstra , Mark Rutland , Will Deacon , Jiri Olsa , Namhyung Kim , "linux-kernel@vger.kernel.org" , linux-arm-kernel , linux-perf-users Subject: Re: [PATCH RFC v1 2/3] perf: Add perf_event_attr::config3 Message-ID: References: <20220825-arm-spe-v8-7-v1-0-c75b8d92e692@kernel.org> <20220825-arm-spe-v8-7-v1-2-c75b8d92e692@kernel.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-Url: http://acmel.wordpress.com X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Em Thu, Aug 25, 2022 at 02:43:42PM -0500, Rob Herring escreveu: > On Thu, Aug 25, 2022 at 2:31 PM Arnaldo Carvalho de Melo wrote: > > Em Thu, Aug 25, 2022 at 01:08:01PM -0500, Rob Herring escreveu: > > > Arm SPEv1.2 adds another 64-bits of event filtering control. As the > > > existing perf_event_attr::configN fields are all used up for SPE PMU, an > > > additional field is needed. Add a new 'config3' field. > > Try not to have tools/ and kernel code in the same patch, else you'll > > burden kernel developers into testing tools/, which so far has been > > refrained. > I knew that, but assumed the header was special... So, we have tools/perf/check-headers.sh for that purpose, so that tools developers can have the opportunity to see that the kernel ABI changed, think about it, check what is needed to best support the new kernel ABI, etc. I really love when kernel developers take the time and help with supporting new features by working on the tools/ bits, that would be perfect! But, as we saw just a few days ago, when changes were made in the kernel and in tools/ in tandem, tools/ got broken just before -rc2: "perf tools: Fix compile error for x86" https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=cfd2b5c1106fa20254d9f24970232cdf24860005> So it seems better to first do the kernel work, then do the user part, to avoid burdening kernel developers with tools/ work. - Arnaldo _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel