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 mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 93F76C433FE for ; Tue, 12 Oct 2021 13:30:57 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 689DB60EB6 for ; Tue, 12 Oct 2021 13:30:57 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236734AbhJLNc5 (ORCPT ); Tue, 12 Oct 2021 09:32:57 -0400 Received: from foss.arm.com ([217.140.110.172]:42520 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233296AbhJLNc4 (ORCPT ); Tue, 12 Oct 2021 09:32:56 -0400 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 F2EB91063; Tue, 12 Oct 2021 06:30:54 -0700 (PDT) Received: from [10.57.21.6] (unknown [10.57.21.6]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id D0A593F66F; Tue, 12 Oct 2021 06:30:52 -0700 (PDT) Subject: Re: [PATCH 0/3] perf tools: Enable strict JSON parsing To: Arnaldo Carvalho de Melo , kajoljain Cc: john.garry@huawei.com, ak@linux.intel.com, linux-perf-users@vger.kernel.org, Nick.Forrington@arm.com, Andrew.Kilroy@arm.com, Will Deacon , Mathieu Poirier , Leo Yan , Mark Rutland , Alexander Shishkin , Jiri Olsa , Namhyung Kim , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org References: <20211007110543.564963-1-james.clark@arm.com> <5947c093-cff9-f70e-af20-75bc053edf5f@linux.ibm.com> From: James Clark Message-ID: <0d73d04f-925c-4c97-9a07-18cc64a9c68b@arm.com> Date: Tue, 12 Oct 2021 14:30:51 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.13.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 08/10/2021 20:00, Arnaldo Carvalho de Melo wrote: > Em Fri, Oct 08, 2021 at 04:56:55PM +0530, kajoljain escreveu: >> >> >> On 10/8/21 3:32 PM, James Clark wrote: >>> >>> >>> On 08/10/2021 08:43, kajoljain wrote: >>>> >>>> >>>> On 10/7/21 4:35 PM, James Clark wrote: >>>>> After a discussion on "[PATCH 1/4] perf vendor events: Syntax corrections in Neoverse N1 json", >>>>> John Garry suggested that we can just modify the parser to make it more strict. Hopefully this will >>>>> remove the need to apply any future JSON comma fixup commits. >>>>> >>>>> Applies on top of "[PATCH v2 1/3] perf vendor events: Syntax corrections in Neoverse N1 json" on >>>>> perf/core. >>>>> >>>>> Also available at: >>>>> git clone --branch james-json-parse-fix git@git.gitlab.arm.com:linux-arm/linux-jc.git >>>> >>>> Hi James, >>>> Do we have any dependency patches on top of this patch series. I am >>>> reviewing and testing it, but in both powerpc and x86 system I am >>>> getting build issue. Not sure if I am missing something> >>>> I am trying your changes on top of upstream perf. >>>> >>>> pmu-events/arch/test/test_soc/sys/uncore.json: json error Invalid >>>> character inside JSON string >>> >>> Hi Kajol, >>> >>> A trailing comma was fixed in this file 3 weeks ago at b8b350a. Can you >>> confirm if you have updated to get this commit on perf core? >>> >>> Alternately you could pull from my branch above which is up to date enough >>> to include it. >> >> Hi James, >> Thanks for pointing it. Not getting build issue now. >>> >>> The file is in pmu-events/arch/test/ so I would expect it to fail on all platforms. >>> >>>> make[3]: *** [pmu-events/Build:18: pmu-events/pmu-events.c] Error 1 >>>> make[3]: *** Deleting file 'pmu-events/pmu-events.c' >>>> make[2]: *** [Makefile.perf:667: pmu-events/pmu-events-in.o] Error 2 >>>> make[2]: *** Waiting for unfinished jobs.... >>>> make[1]: *** [Makefile.perf:238: sub-make] Error 2 >>>> make: *** [Makefile:70: all] Error 2 >>>> >>>> Also, Is it possible to add line number along with file name while >>>> showing this error `json error Invalid character inside JSON string`. >>>> It might make it easy to fix. >>> >>> I can add a character number with the following fix if you think that would >>> be good enough? A line number might be a bigger change and involve keeping >>> track of newline characters. >> >> Sure. I think then we can skip this change. Not sure if character >> number will be helpful. >> >> Patch-set looks good to me. >> >> Reviewed-by Kajol Jain > > Applied ok as-is to my perf/core branch, applied and added your > Reviewed-by, thanks. > Thanks Arnaldo. This does mean that the arm64 build will fail until "[PATCH v2 1/3] perf vendor events: Syntax corrections in Neoverse N1 json" is applied. I think there is also an arm64 build issue with "[PATCH 02/21] perf pmu: Add const to pmu_events_map." which Andrew Kilroy has replied to. James 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 mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id DBA11C433EF for ; Tue, 12 Oct 2021 13:36:57 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id 9BFD360F3A for ; Tue, 12 Oct 2021 13:36:57 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 9BFD360F3A Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=arm.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.infradead.org 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:Date: Message-ID:From:References:Cc:To:Subject:Reply-To:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Owner; bh=qk18+rjDcFUV3bsLKSYj6K/s1FDlUKlK/oX1PdDIUNs=; b=uzL6YJ0gVTLR9XXc6o2w7y7kRt EFlwzdEU0r1OpFwwNfoPbi8pMQza2JFYRtTeySiuP7WCCWfJCEb4n6dujParYh3oCOYkLEgZz0oE+ BQB+5VPru8RskuFbgOO6LLpIBYEojMWR629QaukR+X8YgG1TgZjC903l5cJEubQpRykn6gWvF2Dmi q7FrkYeT4VXAtwFGdPHTawvaoCn1rFv/U3IDXQvEZ8Clp7hrgH0XjsJdpNbsoyHCUMR4kz+Cjaz1O 2Ui6d5/kVr/GbCdspUS4cWp5UHJoZFCmL45nZqY8m3RvjoPf3aUwpgI9zF9k73nQu2pdqoDeTSmjM MUu9oVOQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1maHvW-00Czka-7G; Tue, 12 Oct 2021 13:34:43 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1maHrr-00CyCu-VC for linux-arm-kernel@lists.infradead.org; Tue, 12 Oct 2021 13:30:57 +0000 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 F2EB91063; Tue, 12 Oct 2021 06:30:54 -0700 (PDT) Received: from [10.57.21.6] (unknown [10.57.21.6]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id D0A593F66F; Tue, 12 Oct 2021 06:30:52 -0700 (PDT) Subject: Re: [PATCH 0/3] perf tools: Enable strict JSON parsing To: Arnaldo Carvalho de Melo , kajoljain Cc: john.garry@huawei.com, ak@linux.intel.com, linux-perf-users@vger.kernel.org, Nick.Forrington@arm.com, Andrew.Kilroy@arm.com, Will Deacon , Mathieu Poirier , Leo Yan , Mark Rutland , Alexander Shishkin , Jiri Olsa , Namhyung Kim , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org References: <20211007110543.564963-1-james.clark@arm.com> <5947c093-cff9-f70e-af20-75bc053edf5f@linux.ibm.com> From: James Clark Message-ID: <0d73d04f-925c-4c97-9a07-18cc64a9c68b@arm.com> Date: Tue, 12 Oct 2021 14:30:51 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.13.0 MIME-Version: 1.0 In-Reply-To: Content-Language: en-US X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20211012_063056_160741_FE0AE1B6 X-CRM114-Status: GOOD ( 20.76 ) 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 On 08/10/2021 20:00, Arnaldo Carvalho de Melo wrote: > Em Fri, Oct 08, 2021 at 04:56:55PM +0530, kajoljain escreveu: >> >> >> On 10/8/21 3:32 PM, James Clark wrote: >>> >>> >>> On 08/10/2021 08:43, kajoljain wrote: >>>> >>>> >>>> On 10/7/21 4:35 PM, James Clark wrote: >>>>> After a discussion on "[PATCH 1/4] perf vendor events: Syntax corrections in Neoverse N1 json", >>>>> John Garry suggested that we can just modify the parser to make it more strict. Hopefully this will >>>>> remove the need to apply any future JSON comma fixup commits. >>>>> >>>>> Applies on top of "[PATCH v2 1/3] perf vendor events: Syntax corrections in Neoverse N1 json" on >>>>> perf/core. >>>>> >>>>> Also available at: >>>>> git clone --branch james-json-parse-fix git@git.gitlab.arm.com:linux-arm/linux-jc.git >>>> >>>> Hi James, >>>> Do we have any dependency patches on top of this patch series. I am >>>> reviewing and testing it, but in both powerpc and x86 system I am >>>> getting build issue. Not sure if I am missing something> >>>> I am trying your changes on top of upstream perf. >>>> >>>> pmu-events/arch/test/test_soc/sys/uncore.json: json error Invalid >>>> character inside JSON string >>> >>> Hi Kajol, >>> >>> A trailing comma was fixed in this file 3 weeks ago at b8b350a. Can you >>> confirm if you have updated to get this commit on perf core? >>> >>> Alternately you could pull from my branch above which is up to date enough >>> to include it. >> >> Hi James, >> Thanks for pointing it. Not getting build issue now. >>> >>> The file is in pmu-events/arch/test/ so I would expect it to fail on all platforms. >>> >>>> make[3]: *** [pmu-events/Build:18: pmu-events/pmu-events.c] Error 1 >>>> make[3]: *** Deleting file 'pmu-events/pmu-events.c' >>>> make[2]: *** [Makefile.perf:667: pmu-events/pmu-events-in.o] Error 2 >>>> make[2]: *** Waiting for unfinished jobs.... >>>> make[1]: *** [Makefile.perf:238: sub-make] Error 2 >>>> make: *** [Makefile:70: all] Error 2 >>>> >>>> Also, Is it possible to add line number along with file name while >>>> showing this error `json error Invalid character inside JSON string`. >>>> It might make it easy to fix. >>> >>> I can add a character number with the following fix if you think that would >>> be good enough? A line number might be a bigger change and involve keeping >>> track of newline characters. >> >> Sure. I think then we can skip this change. Not sure if character >> number will be helpful. >> >> Patch-set looks good to me. >> >> Reviewed-by Kajol Jain > > Applied ok as-is to my perf/core branch, applied and added your > Reviewed-by, thanks. > Thanks Arnaldo. This does mean that the arm64 build will fail until "[PATCH v2 1/3] perf vendor events: Syntax corrections in Neoverse N1 json" is applied. I think there is also an arm64 build issue with "[PATCH 02/21] perf pmu: Add const to pmu_events_map." which Andrew Kilroy has replied to. James _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel