Received: by 2002:a05:7412:d1aa:b0:fc:a2b0:25d7 with SMTP id ba42csp1784550rdb; Wed, 31 Jan 2024 08:59:45 -0800 (PST) X-Google-Smtp-Source: AGHT+IGp7hGi2IsYMKGCyjo5syoyixCapiMTPviPEW6xdfhW23QeMLff0sLfPQqBxbmLNZ9uQE8T X-Received: by 2002:a17:906:d8d:b0:a36:6c98:a506 with SMTP id m13-20020a1709060d8d00b00a366c98a506mr2004734eji.18.1706720385120; Wed, 31 Jan 2024 08:59:45 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1706720385; cv=pass; d=google.com; s=arc-20160816; b=ZH0ZKkOX7GER6tC+oOYTAYi4CcgaAfpm92rIL5NO3dMPrALyKy0+ePz3oqP/lZwyJO hI5X0bqKOfssM8+qNbsuXBxOtoe7ZCVobJNE+7/UiUGGNCaYj1gTITI9RRtXizKBUD4o Ot8Orkj1GaSNioS9pU9Elxq+qWC1WB4bUJf6Dnp8NENjwxbKhlndHknSIPO57O2VPHoV 4wEPeSkjzqQWqwA8oW8NiLjjl4WqYpyh1E9ehfHkyryI9xnME8gSq7xHiqCmX6mIxyOa ly/bRJZmUDK3IdIwkDwKl2NlwnJZNB04Qk+WcNvC2launAnM3W8ThBMPB8Ng8WRABWbK uW6w== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:date:message-id; bh=JIjyF8EJ3yi1KvZBFlXFEA3IDdQ8f6IaWS8hCpKOfks=; fh=nZdz4SngU8SUYRqqwVRTaQPPSt8nHdYZewCpKAnwIwA=; b=yXznAMSOWEj5L1PzAdrWjZWaAd1b6uXyZYHioAXvreNf7ABXGVi2t+d5QGELB8B1wp 8IAkqGTDLfLGob9edIKirWWJwQV7HoYwPIlHicZDfCYqKKYH9s7pVFgGywJ1Ev5WqGKe tYXJIB2nn3oBDJWygdklpCKz8jxzfHMu8u8zTtTkMlNBJYxAALPQwrakYNxu7TqndEsE E8a5qzz+UfS3UPdJsDc0fqseV4kpEN0UW5kcv6bN3emon+woVeyzOnXcunWUoIvxRNik Wit5hnMSk+Bgvj1rSWabyRwRtNxJ0EwVhcjugMXkt4VyGPZC+aeSFZ0561+NzB4KNcJ6 ISpw==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; arc=pass (i=1 spf=pass spfdomain=arm.com dmarc=pass fromdomain=arm.com); spf=pass (google.com: domain of linux-kernel+bounces-46851-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) smtp.mailfrom="linux-kernel+bounces-46851-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com X-Forwarded-Encrypted: i=1; AJvYcCUK7muZP+ioC5HUnM0on4hXUOLrHwjZXRmEhNDKUOgL73f9dLP2HKtrDpbeNc2s7o/hF5o2HJ/5YaQc/wNuQhlSUhg9JFsty76IOfcS9g== Return-Path: Received: from am.mirrors.kernel.org (am.mirrors.kernel.org. [147.75.80.249]) by mx.google.com with ESMTPS id z27-20020a170906075b00b00a364af04fc2si1346895ejb.794.2024.01.31.08.59.45 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 31 Jan 2024 08:59:45 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-46851-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249; Authentication-Results: mx.google.com; arc=pass (i=1 spf=pass spfdomain=arm.com dmarc=pass fromdomain=arm.com); spf=pass (google.com: domain of linux-kernel+bounces-46851-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) smtp.mailfrom="linux-kernel+bounces-46851-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by am.mirrors.kernel.org (Postfix) with ESMTPS id D9D221F22081 for ; Wed, 31 Jan 2024 16:59:44 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id E98BF12BE9D; Wed, 31 Jan 2024 16:59:11 +0000 (UTC) Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 9449512AAD1 for ; Wed, 31 Jan 2024 16:59:09 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.140.110.172 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1706720351; cv=none; b=rLIzunYYnhh/QgvHmSSkK2yC3BR/nlBdhqVgA2OyHDknH0/WAAcy8IsGHOboCML7qD+/JTQE+klEz4V3wyPlZh0RrEv/zUO0MYmLTg4xbzngCXu51wXjattSzxzwqLlCZep2gv3L6YChrz0la/pwA2DjRzK0nDfCkHTkSkTsMCE= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1706720351; c=relaxed/simple; bh=GjVWMfDZ6tvNQI5J0g8k01qf+gu5mZITKfSXLDmab8I=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=ndr4Mj5DsFaGTLJunxJbNjaCa/dEDehtu4paa64hf+YQyNI10dgkLTiHRF1zdrC30fUBuaXa0ZNfItDzN6wJua5vbh7q//cWkKr7mVZ92JNmjaVrVx5J+E5jFzw21SVQ6OSIY79x4G2HK3gFMiZz/U3JtN9BS0TxIfUC6v8of94= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com; spf=pass smtp.mailfrom=arm.com; arc=none smtp.client-ip=217.140.110.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=arm.com Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 098CFDA7; Wed, 31 Jan 2024 08:59:52 -0800 (PST) Received: from [10.1.196.40] (e121345-lin.cambridge.arm.com [10.1.196.40]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 886B13F762; Wed, 31 Jan 2024 08:59:07 -0800 (PST) Message-ID: Date: Wed, 31 Jan 2024 16:59:05 +0000 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2] dt-bingdings: perf: Support uncore ARM NI-700 PMU Content-Language: en-GB To: =?UTF-8?B?WWFuZyBKaWFsb25nIOadqOS9s+m+mQ==?= , Krzysztof Kozlowski , will , "mark.rutland" Cc: shenghui.qu@shingroup.cn, =?UTF-8?B?6LW15Y+v?= , zhijie.ren@shingroup.cn, linux-kernel@vger.kernel.org, linux-arm-kernel , Jonathan Cameron References: <20240131065953.9634-1-jialong.yang@shingroup.cn> <02B995F774AB7A9D+adfd0934-00cb-4dc3-8bf8-058b83dc2fbb@shingroup.cn> <873A6CC450C1D0E6+75a39d6c-4fbf-4e30-8630-3226bd725901@shingroup.cn> From: Robin Murphy In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit On 31/01/2024 10:18 am, Yang Jialong 杨佳龙 wrote: > > > 在 2024/1/31 18:16, Krzysztof Kozlowski 写道: >> On 31/01/2024 11:13, Yang Jialong 杨佳龙 wrote: >>> >>> >>> 在 2024/1/31 17:30, Krzysztof Kozlowski 写道: >>>> On 31/01/2024 10:26, Yang Jialong 杨佳龙 wrote: >>>>> >>>>> >>>>> 在 2024/1/31 15:49, Krzysztof Kozlowski 写道: >>>>>> On 31/01/2024 07:59, JiaLong.Yang wrote: >>>>>>> Add file corresponding to hx_arm_ni.c introducing ARM NI-700 PMU >>>>>>> driver >>>>>>> for HX. >>>>>>> >>>>>>> Signed-off-by: JiaLong.Yang >>>>>>> --- >>>>>>> v1 --> v2: >>>>>>> 1. Submit dt-bindings file Seperately. >>>>>> >>>>>> Where is the driver? >>>>>> >>>>>> Please read: >>>>>> https://elixir.bootlin.com/linux/v6.8-rc2/source/Documentation/process/submitting-patches.rst >>>>>> before posting. >>>>>> >>>> >>>> Keep all discussions public. >>> >>> Get. >>> >>>> >>>> >>>>>>> +  pccs-id: >>>>>>> +    $ref: /schemas/types.yaml#/definitions/uint32 >>>>>>> +    description: Used to identify NIs in system which has more than >>>>>>> +      one NI. >>>>>> >>>>>> No, reg does it. Drop the property. Anyway you miss here vendor >>>>>> prefix >>>>>> and proper explanation. >>>>>> >>>>> >>>>> reg will tell phy address. Phy address is too long. I just want a id. >>>>> example: perf stat -a -e ni_pmu_${pccs-id}/cycles/ >>>>> I will use it in user space. Not only in driver. >>>> >>>> Custom vendor property is not for that purpose. Use for example IDR, DT >>>> aliases or something else. >>>> >>>> >>> >>> I have considered TD aliases. It's not very easy. Two places... >>> IDR.. I have tested. But it chouldn't correspond to the HWs. >>> I need it to identify NIs. >>> DT aliases is reachable. But no very easy. >> >> Except that "you want" I did not see any rationale, any argument >> explaining why this is needed and why this has to be present. > > OK. DT aliases it good. The real way to address that particular issue is to fix perf to properly associate the PMU device with the underlying hardware device. Jonathan had a series doing that[1], but I'm not sure what its status is now. Thanks, Robin. [1] https://lore.kernel.org/linux-arm-kernel/20230404134225.13408-1-Jonathan.Cameron@huawei.com/