Received: by 2002:a05:6a10:d5a5:0:0:0:0 with SMTP id gn37csp1848874pxb; Thu, 7 Oct 2021 16:56:22 -0700 (PDT) X-Google-Smtp-Source: ABdhPJz+SA6rb7D99dpO5/WZmBnSNdMFyhXn7dA8JL1aIqyoWpMYLocVr7nPaJCROL8+Y+mYMp9w X-Received: by 2002:a17:906:f8d0:: with SMTP id lh16mr9061718ejb.367.1633650981929; Thu, 07 Oct 2021 16:56:21 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1633650981; cv=none; d=google.com; s=arc-20160816; b=RbNiHXjDQXHmNKiyYfOFAX9SA2KfOt0iDyGQopAbZebCwJKq9EI+6zcGdbVTtlPkAv wJGQ+LuiBTEBvEkrdGfPJenXuIzLYZhE7H1JZ6N7pq643N217E6TLL1TPo/k3sn/GT8b jnrT3vetg92V29Wdz0cFiIRMRXgPjOjGyf3h347wiYayNW7HVg7hkRJ/heikZzPGD144 ien8NFk/mD0fy8YWdC/g7eO5S0XLiVeFvFaaIBfKW6r45xgLvprqg0e/SpEEDnY8Hzut Q89IWoNQOFNjffLbDYHULMQg2hPt+2yaOTmH9jvx/zkLV74mianzFe9mejIxkD81wwlE g78Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:in-reply-to:from :references:cc:to:content-language:subject:user-agent:mime-version :date:message-id; bh=yzVKQ2LwInavlNWURX5J1qxqjom+dFTk0AS3CU9ycQ0=; b=JBpj/PIIPjQnEo6cv8SzvmDPXkTBf+L9y7CfUfY/qJWA8ch9vMGtI1N1B7AXnkVM0v BpCefJkv8i/FhXjPvAtlEQyD9+laVXoHAk4XCFwBgW+kW0830u3dXtXBudfHGh/h8Vk0 p0BRyROZJXOfTJeM8FDay6bxLQDz87bwqyku0sqH8v41TVQyA2CVUtEa0dJeP3Xp5hv4 9+JLHVtUP50+90fGp4qdMYLaqzuV3HN+NL/gXPXlgyfOt8UCmsad2ZeclMqVDvb5dYyd fS0YgykomZy36n3GtM2pc1eXngtpR0ru0B8pZZkKpx0H2/qJeHREFy3p9t2thXf+mP4e 5B0w== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id lb12si1155241ejc.254.2021.10.07.16.55.58; Thu, 07 Oct 2021 16:56:21 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S237123AbhJGXx4 (ORCPT + 99 others); Thu, 7 Oct 2021 19:53:56 -0400 Received: from mga02.intel.com ([134.134.136.20]:3640 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S238068AbhJGXxy (ORCPT ); Thu, 7 Oct 2021 19:53:54 -0400 X-IronPort-AV: E=McAfee;i="6200,9189,10130"; a="213536473" X-IronPort-AV: E=Sophos;i="5.85,355,1624345200"; d="scan'208";a="213536473" Received: from orsmga003.jf.intel.com ([10.7.209.27]) by orsmga101.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 07 Oct 2021 16:51:59 -0700 X-IronPort-AV: E=Sophos;i="5.85,355,1624345200"; d="scan'208";a="440452771" Received: from akleen-mobl1.amr.corp.intel.com (HELO [10.209.19.60]) ([10.209.19.60]) by orsmga003-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 07 Oct 2021 16:51:58 -0700 Message-ID: Date: Thu, 7 Oct 2021 16:51:57 -0700 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101 Thunderbird/91.1.2 Subject: Re: [PATCH 0/3] perf tools: Enable strict JSON parsing Content-Language: en-US To: James Clark , acme@kernel.org, john.garry@huawei.com, linux-perf-users@vger.kernel.org Cc: 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> From: Andi Kleen In-Reply-To: <20211007110543.564963-1-james.clark@arm.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 10/7/2021 4:05 AM, 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. Looks good to me.  The Intel files are already generated by automated tools using the standard python JSON writer, I guess if it's out of sync someone must have edited it by hand. So it should be fine to fix it. Reviewed-by: Andi Kleen -Andi