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 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 smtp.lore.kernel.org (Postfix) with ESMTPS id 487E5C83F12 for ; Tue, 29 Aug 2023 08:44:38 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:From:References:Cc:To:Subject: MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=Z70JsVSCVMTqn+h8pnWSXcX/gcFtl86ab2z7lelViP8=; b=GF0UjPZlZJk4b6 +32Ujj08rm3pMmUCySxfjrqClil1jAZNSd8wfIKS0QYBpU7ZyTB16SepaEOR9c2/afgnNLSdAWbnh I4nWLCjlWNPf6zfGMzKIb50JWel6tpM1pO/S8wi2ObnyT04mQT0Z+fOwLhVccvXO5N3wt0+pUrhES FibNsnf4W4CJzJ6+0Zpa0712zl857Q91BOociHFixFhqyEeY6xXof/zDi/JcuX3kMNtexzgmPDyOV xE8pb4vc3w/PkM1dhaOzW2lQjFR7rBJoUFiErhIRCuMrsAKCEQXGbzIRcNJbCArZhAFWMMBdx2+Ff iXAdPQgtNBXTipldAIQw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qauKa-00B487-0I; Tue, 29 Aug 2023 08:44:12 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qauKW-00B45J-1E for linux-arm-kernel@lists.infradead.org; Tue, 29 Aug 2023 08:44:10 +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 305C32F4; Tue, 29 Aug 2023 01:44:39 -0700 (PDT) Received: from [10.57.3.66] (unknown [10.57.3.66]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 093D43F64C; Tue, 29 Aug 2023 01:43:57 -0700 (PDT) Message-ID: <96d41d86-bfa2-f4d4-49ea-449c3471d036@arm.com> Date: Tue, 29 Aug 2023 09:43:56 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:102.0) Gecko/20100101 Thunderbird/102.14.0 Subject: Re: [PATCH V5 0/4] coresight: trbe: Enable ACPI based devices To: Will Deacon Cc: Anshuman Khandual , linux-arm-kernel@lists.infradead.org, catalin.marinas@arm.com, kernel-team@android.com, Alexander Shishkin , coresight@lists.linaro.org, linux-kernel@vger.kernel.org, James Clark , Mike Leach , yangyicong@huawei.com, Mark Rutland , Sami Mujawar , Leo Yan References: <20230817055405.249630-1-anshuman.khandual@arm.com> <169237557557.1402587.14721093106016169238.b4-ty@kernel.org> <20230821112851.GC19469@willie-the-truck> <311d0ecf-dddf-dcf5-9545-3f8a51634a9a@arm.com> <20230828213537.GA27629@willie-the-truck> From: Suzuki K Poulose In-Reply-To: <20230828213537.GA27629@willie-the-truck> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230829_014408_481773_47DA99B4 X-CRM114-Status: GOOD ( 17.15 ) 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-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 28/08/2023 22:35, Will Deacon wrote: > On Sun, Aug 27, 2023 at 11:11:16PM +0100, Suzuki K Poulose wrote: >> On 21/08/2023 12:28, Will Deacon wrote: >>> Hmm, the rationale behind your change to make the pdata allocation >>> per-device in ("coresight: trbe: Allocate platform data per device") >>> confuses me: with Anshuman's change to allocate the pdata using >>> devm_kzalloc(), there shouldn't be any connections for the coresight >>> core to trip over, should there? >> >> Anshuman's patch is working around the problem of "TRBE platform >> device with ACPI doesn't have a valid companion device" - this is a problem >> for the acpi_get_coresight_platform_data(). The work >> around is to move the "allocation" from coresight_get_platform_data() >> to the driver (given we don't need anything else from the ACPI except >> the IRQ). That doesn't change *how* it is allocated. >> Also please note that, the TRBE driver creates a TRBE coresight_device >> per-CPU and the platform data is shared by all of these devices, which >> the coresight core driver doesn't cope with. The other option is to >> move the releasing of these platform-data to the individual drivers, >> which is quite an invasive change. Or, make the core driver tolerate >> a NULL platform data, which is also again invasive. So the merged fix >> is correct and is still valid after this patch. > > Ah, ok, so the problem with TRBE isn't anything to do with its connections, > but simply because the pdata is shared? Correct, thats an issue outside of the ACPI support. With ACPI, the coresight_get_platform_data() can't cope with the TRBE with missing companion device, that is fixed by Anshuman's patch in this series. Also, please could you confirm the plan forward for merging this (next version of course) ? Cheers Suzuki > > Will _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel