Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:4 as permitted sender) client-ip=2620:137:e000::3:4;
From:   Suzuki K Poulose <suzuki.poulose@arm.com>
To:     linux-kernel@vger.kernel.org, mathieu.poirier@linaro.org,
        Yicong Yang <yangyicong@huawei.com>,
        jonathan.cameron@huawei.com
Cc:     Suzuki K Poulose <suzuki.poulose@arm.com>, hejunhao3@huawei.com,
        yangyicong@hisilicon.com, helgaas@kernel.org, linuxarm@huawei.com,
        linux-pci@vger.kernel.org, alexander.shishkin@linux.intel.com,
        prime.zeng@hisilicon.com
Subject: Re: [PATCH v3 0/5] Several updates for PTT driver
Date:   Tue, 17 Oct 2023 18:01:32 +0100
Message-Id: <169756192530.746517.2466420521869567832.b4-ty@arm.com>
In-Reply-To: <20231010084731.30450-1-yangyicong@huawei.com>
References: <20231010084731.30450-1-yangyicong@huawei.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
Precedence: bulk

On Tue, 10 Oct 2023 16:47:26 +0800, Yicong Yang wrote:
> From: Yicong Yang <yangyicong@hisilicon.com>
> 
> This series contains several updates for PTT driver:
> - Disable interrupt when trace stops, reverse to what we do in trace start
> - Always handle the interrupt in hardirq context

I wrapped the commit description to 75 chars for this patch to suppress the
following checkpatch warning and queued it.

WARNING: Prefer a maximum 75 chars per line (possible unwrapped commit description?)
#10:
change of arm-ccn PMU commit 0811ef7e2f54 ("bus: arm-ccn: fix PMU interrupt flags").

total: 0 errors, 1 warnings, 12 lines checked

NOTE: For some of the reported defects, checkpatch may be able to
      mechanically convert to the typical style using --fix or --fix-inplace.

      "[PATCH] hwtracing: hisi_ptt: Handle the interrupt in hardirq context" has style problems, please review.


> - Optimize the AUX buffer handling to make consumer have more time to process
>   the data
> - Since we're a uncore PMU so block any task attach operation
> - Add a dummy pmu::read() callback since the perf core may use
> 
> [...]

Applied, thanks!

[1/5] hwtracing: hisi_ptt: Disable interrupt after trace end
      https://git.kernel.org/coresight/c/4669551e797a
[2/5] hwtracing: hisi_ptt: Handle the interrupt in hardirq context
      https://git.kernel.org/coresight/c/e8b7d8718c51
[3/5] hwtracing: hisi_ptt: Optimize the trace data committing
      https://git.kernel.org/coresight/c/7a527d4d9273
[4/5] hwtracing: hisi_ptt: Don't try to attach a task
      https://git.kernel.org/coresight/c/7d52e2cfef91
[5/5] hwtracing: hisi_ptt: Add dummy callback pmu::read()
      https://git.kernel.org/coresight/c/4708eada8bd6

Best regards,
-- 
Suzuki K Poulose <suzuki.poulose@arm.com>