Received: by 2002:a05:7412:d1aa:b0:fc:a2b0:25d7 with SMTP id ba42csp2045517rdb; Wed, 31 Jan 2024 18:25:21 -0800 (PST) X-Google-Smtp-Source: AGHT+IEl5VQh8crIa8No0gB16Br3of7LT9FqRPkVZ6JtxHTGrIjnFi9qkNprzRIr0HqVcDi51ciS X-Received: by 2002:a05:6808:22a3:b0:3be:ca15:21e5 with SMTP id bo35-20020a05680822a300b003beca1521e5mr2916413oib.25.1706754321103; Wed, 31 Jan 2024 18:25:21 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1706754321; cv=pass; d=google.com; s=arc-20160816; b=a8XFrFmAYtMeIghg+agkft+FodGH2joPMK9+WEKC1ygpmFjqDEyMvJI4V/J3ioQuGh rCapojJgvAYKYvC1oAzOR6z4ErfXA7V1QtVFXrJ0C6QVrbja6mylV13+L6/9IgLaiPnV 9Opx9+171c5/j/shGhTjJPkBe4mbKy1PcqM+JWvrOU/otMA9SWwWLW/w5Bf+5WnNhRfg 7HVArLvF/0wW4RgqyrTQNTbDxA8NlLLCR9AL6x+qqRx/tqrGN6TzoPuOxkTwR48EnMav 2D/x0aONbkXA9uPKCV1yvDrBbl2lf+IwIHJ2oefzyj0L+M4nSLWrvX4AJX74f1JvtD+F ocBw== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=feedback-id: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=8Q/eysPx9K9u7wr8tt9PEiZqZSCqgJGhHiavi2Lsc7s=; fh=JuZu9QcNZeU1MOjo3MQ9/B4x/3cShtOm7IkD0selLxU=; b=JbC+m3T24yCDjF4ASDGFgGo7+D4le1ADDvhez3m2m+JXvJ+FswLxf3ruFJvsnrl7uE BZ7BCIpuSe3A47oMBRhBiLgDjv/8TUrB6Cq76mftT7/9hwNKwIK8y10gVIL6XM76dYZh oJLXciiJea8TMYIesNjEVevsrxYUA9iNEUsrWLAZ5Kg1EDZkIVPcV1RM3JpfeJvWi28B ThsupdvQjRGckvNY2EtT3Srmrss//kdFbafQTzjUaanY/4DMw9WUkBrwXzp9cFswcWZG s7KEa2uYmj8eY0snMAW2q7Hn2eu696qtV41ktugnQaRoIjl+WrJ3im0spsbtc2VIkejz j/FA==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; arc=pass (i=1 spf=pass spfdomain=shingroup.cn dmarc=pass fromdomain=shingroup.cn); spf=pass (google.com: domain of linux-kernel+bounces-47540-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-47540-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=shingroup.cn X-Forwarded-Encrypted: i=1; AJvYcCW9kV+br+LqujR1B9+Vezqw/xy/14kfGOPLHiGBIRRc+mXjUlxS1mhfIEdi6EDSZqHayNygPXP0/exKOzSe2+1fpqAzj5m0M5b0OkhgWQ== Return-Path: Received: from sv.mirrors.kernel.org (sv.mirrors.kernel.org. [2604:1380:45e3:2400::1]) by mx.google.com with ESMTPS id n24-20020a056a000d5800b006d9906df4ccsi10766936pfv.162.2024.01.31.18.25.20 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 31 Jan 2024 18:25:21 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-47540-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1; Authentication-Results: mx.google.com; arc=pass (i=1 spf=pass spfdomain=shingroup.cn dmarc=pass fromdomain=shingroup.cn); spf=pass (google.com: domain of linux-kernel+bounces-47540-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-47540-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=shingroup.cn 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 sv.mirrors.kernel.org (Postfix) with ESMTPS id B7B3528D40D for ; Thu, 1 Feb 2024 02:25:20 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id EB2C521112; Thu, 1 Feb 2024 02:25:16 +0000 (UTC) Received: from smtpbguseast3.qq.com (smtpbguseast3.qq.com [54.243.244.52]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id E6C90210E0 for ; Thu, 1 Feb 2024 02:25:10 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=54.243.244.52 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1706754316; cv=none; b=sV3MiTJ8j1O8T5eTPpQ7jVg38HPpQU40N3r6yzgKvMQk6E9xLBkIBY5JjCpfU/G/dTLSbbjAgmih9Slbf1auBqQZI56CJuQAPnGSKn/P8mPXC/wpqN4jKAYWu8Ja740hbJHeJ+TwXC+QgNF5w1H9UBdrJSxkNokziqAlkcZFG38= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1706754316; c=relaxed/simple; bh=+vXisyTED3oYjSB3fpGPEFDtkBLUsa77LAHwLPrG0y8=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=PbcMaHDf9nK93AoefkCKeHww79T+bcYHBUuPbTjNVDVCZ7oSMOcGAkuKQX2QmfhTm7Ynr16/HOWmceelvn6+5ZThSj8qjo9MDtYFMsBtOm+rMKaN7rlK75hj/2uLuyAbrDAoiXMlqmPoYOsBFPYmVP+nxqFCvq935byNxb6hEf8= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=shingroup.cn; spf=pass smtp.mailfrom=shingroup.cn; arc=none smtp.client-ip=54.243.244.52 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=shingroup.cn Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=shingroup.cn X-QQ-mid: bizesmtp75t1706754277tdnhvipn X-QQ-Originating-IP: kbaL+n5H+N5CjyyZxgbzU94/3JnTA9cqlLwUkW78w1g= Received: from [127.0.0.1] ( [223.112.234.130]) by bizesmtp.qq.com (ESMTP) with id ; Thu, 01 Feb 2024 10:24:35 +0800 (CST) X-QQ-SSF: 00400000000000B0B000000A0000000 X-QQ-FEAT: lm7sZZPcOdbLaBr9ejGiuJs25GZF8B5iemLB1NJcutsEeAisshX7xVdH2h/Lu /UpGw+iooGoYuuitGGU7W/Qqwam/v1MYsjqpzhOkiy20kDsw01DQGI8M2TVTsyROYvkIWMi /W/04hfBeofCwCI/bOS86BJkHE8QcSzW2aIbqlHovpSnv5yIjgnMGEpEW9LdXWDrW/p6Aal r06TPBG9sNtIwRvXUecpDPXh7SDwW0kQ3V1Ltcs9ug3OyQm/1HM4zUHVdr4YHYyXH+pw+Z5 2PSJIRslnAHuTvdKzMGCVr3Y2D9NjxAufDhMwxQ525qoO1sXB5Ncy4ox+rKrLUmco8FoIW3 oRJpmbHxq2kIr+Z3NrNuqMH0t1pL7VgV7mVUHiq7Xk6CEP+ckwZET5aRLBDCHbQvd3nTBTP o736OylHaHk= X-QQ-GoodBg: 2 X-BIZMAIL-ID: 14490681971206590225 Message-ID: Date: Thu, 1 Feb 2024 10:24:34 +0800 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-US To: Robin Murphy , 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: =?UTF-8?B?WWFuZyBKaWFsb25nIOadqOS9s+m+mQ==?= In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-QQ-SENDSIZE: 520 Feedback-ID: bizesmtp:shingroup.cn:qybglogicsvrgz:qybglogicsvrgz6a-1 在 2024/2/1 0:59, Robin Murphy 写道: > 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. > Good news. In NI, we have to consider NIs and PMUs in one NI. > Thanks, > Robin. > > [1] > https://lore.kernel.org/linux-arm-kernel/20230404134225.13408-1-Jonathan.Cameron@huawei.com/ >