From:   Bharat Bhushan <bbhushan2@marvell.com>
To:     Will Deacon <will@kernel.org>
CC:     "mark.rutland@arm.com" <mark.rutland@arm.com>,
        "robh+dt@kernel.org" <robh+dt@kernel.org>,
        Bhaskara Budiredla <bbudiredla@marvell.com>,
        Sunil Kovvuri Goutham <sgoutham@marvell.com>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [EXT] Re: [PATCH v6 4/4] perf/marvell: cn10k DDR perf event core
 ownership
Thread-Topic: [EXT] Re: [PATCH v6 4/4] perf/marvell: cn10k DDR perf event core
 ownership
Thread-Index: AQHXzLwcGXQkFkVJY0GLNfcVFE9U4awyMcoAgAAAYDA=
Date:   Tue, 14 Dec 2021 12:27:08 +0000
Message-ID: <CO6PR18MB4465F1A9FFB5C36E75A7749EE3759@CO6PR18MB4465.namprd18.prod.outlook.com>
References: <20211029115643.32351-1-bbhushan2@marvell.com>
 <20211029115643.32351-5-bbhushan2@marvell.com>
 <20211214122507.GC14247@willie-the-truck>
In-Reply-To: <20211214122507.GC14247@willie-the-truck>
Accept-Language: en-IN, en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?z3KB5PuYizjlvwlMR7uF8eFnrx6wmIeL351QIUeNRE8zHWOaaMY5gvlnLBAN?=
 =?us-ascii?Q?cMVhg5y/I5GLdXCbLUA40+ARuG601Ikhxg9y4ml6gpxe2b04z3Gg2au0xb2U?=
 =?us-ascii?Q?52u7ESEpdt5dpDuVw/kFUcxd2FFabZ4D5pQUwYLi4SO8jFXXx5ubEbtwbjRA?=
 =?us-ascii?Q?sWNNGskiKGxFlAiGvmRz9GaNTCuKC9lBKtOFmqVbgkv6qWuiJGpoLG6Semfn?=
 =?us-ascii?Q?XJqgoK2A2k/SHEqb84JDXXyCYb7CkO32VsLEtbS6PRyPNCAiC6HKJ6+23Rqp?=
 =?us-ascii?Q?wDKmP9VD35d4YGGJey6aNte0kt1qXAE2l4YRa+vhHVy3Jcx607oI/PMNMZ2k?=
 =?us-ascii?Q?lwOw3sJJASx29fZc1XxaYZYxaqy8Swqp8mh/v+5VdNYeMVeaYVWw7MHXe0Cr?=
 =?us-ascii?Q?PRE4v5oVtdxWYQGtYsCymBIwTnTM4419iRk+dAWMcj8fW9o+T+/kIk+a16vh?=
 =?us-ascii?Q?pGOI9w6bUMx7gGLbNeCQsMpWPkMbEL9qka9CWdKjw4+HW3jpj+06E+0OOC/w?=
 =?us-ascii?Q?fOgHSP5zwiroO1POx6mLIPfRyXHs0r/sKidHcY/CZ6IGccmKmeXAZamWbvtD?=
 =?us-ascii?Q?Ld61MxnBFEJ+ISiXbWQNl9tgYMa9bGINsEVaheUU7bWEpcD+Ers1KMM6U5qs?=
 =?us-ascii?Q?VdDjXlrkpVz/1yfTOg74/X06lnXA8Jkwf1AD3LI1ONFoaH8y4BOsO2tZJqx8?=
 =?us-ascii?Q?Vxc4J3EH7Cfx8cfbPxCd7ednI6ip8fgAslnf7AjkwtgOcCxoc9fkk/Vkk+d2?=
 =?us-ascii?Q?nzzZ61l1wWohhnJ8I4FzoVTG1N70lSpQz/f1lLCsO2GT+8xHvt6UMvWWgOhH?=
 =?us-ascii?Q?MKDfEI7Ne3TDAfJEKM4fgLy4i7dUbW/BaDVIA4YzvtMlZ/bFLGNKpuKzYBMb?=
 =?us-ascii?Q?iANlkeVS3uwUXNcvKJ2l8HOTj5siF9IkOwmfAhJDwHOYrijsPaOvY7ly7LSd?=
 =?us-ascii?Q?dX1zpxW2s/XZtCD+xn6G0Aim6qm+0/mc0QcwO19Qz/+OfiYGtQSAsYHkDnNa?=
 =?us-ascii?Q?GUelfLsUbM28MVpJStQ/XRNVgrL1LilniUtVj9m8k7YryePtIVBk2j3WSTZp?=
 =?us-ascii?Q?nmzui1F2icL1SetgrzrWFHmh3PdxtCxuXUpwKiTB/oqLnMpiupJQDbADTzNa?=
 =?us-ascii?Q?AI3ISYhV/m3rYj1Qt7Z+GjqDNYIzgkrVJVBDLsz6v/l2jRufvW/Fdt/JfOBY?=
 =?us-ascii?Q?tJ5vI4wJIgwQEpLGA/7aWDZDs54rghkobXOn+7XVZuttKYPqjXapzSC6BAOG?=
 =?us-ascii?Q?MwLI1gge4GuaaJ9J1SjgFkasqpy9C5b7NCGm+Zr0Cb0bKJZ/JnRdQMN3uTrz?=
 =?us-ascii?Q?GiNGH/vdhwcIPps9HGOfpPVJruMVcRKLzOV3gMtK2b2pOuUBUur0qNnpxRZ/?=
 =?us-ascii?Q?ep8wQLF18sqKn7AFQ7Zomqr/QYF6JMUJDgRigf6ggcwDneUKbvuQb9Z2RvLl?=
 =?us-ascii?Q?nogQf7SC3xkwZgjC3L6xtA38mehBKAxuT86j8ecJ18dtnia0DG/ep5UduPbK?=
 =?us-ascii?Q?H4i0N6zKAu9ZHUWoh1LRgUm1Ewi72WIG06glQMUDIkc3q28yvYAu+gKz5sKy?=
 =?us-ascii?Q?EvkQ2/8EhXANjfx7rvYgcrRLm6yTLht2aII4wrTrtlMEkL3OhhD/9IaiYFSM?=
 =?us-ascii?Q?eaLIsNPxd1IzmM7/XWvFH78=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: marvell.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: CO6PR18MB4465.namprd18.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 64f44c5c-314b-414d-61b3-08d9befd0c08
X-MS-Exchange-CrossTenant-originalarrivaltime: 14 Dec 2021 12:27:08.5334
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 70e1fb47-1155-421d-87fc-2e58f638b6e0
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: SSxMEB+n9oSF9KKJyXndqEPKHrU0R2MxtLPy2COWno7NoHl/sGp54rg2xm69Q6dZnILTHLX9FiRz39SSAFsW8Q==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CO1PR18MB4762
X-Proofpoint-GUID: To1R0IxsNKMkmfHX5ZbAkxiJ6s64aZDW
X-Proofpoint-ORIG-GUID: To1R0IxsNKMkmfHX5ZbAkxiJ6s64aZDW
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.205,Aquarius:18.0.790,Hydra:6.0.425,FMLib:17.11.62.513
 definitions=2021-12-14_06,2021-12-14_01,2021-12-02_01
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


> -----Original Message-----
> From: Will Deacon <will@kernel.org>
> Sent: Tuesday, December 14, 2021 5:55 PM
> To: Bharat Bhushan <bbhushan2@marvell.com>
> Cc: mark.rutland@arm.com; robh+dt@kernel.org; Bhaskara Budiredla
> <bbudiredla@marvell.com>; Sunil Kovvuri Goutham <sgoutham@marvell.com>;
> linux-arm-kernel@lists.infradead.org; devicetree@vger.kernel.org; linux-
> kernel@vger.kernel.org
> Subject: [EXT] Re: [PATCH v6 4/4] perf/marvell: cn10k DDR perf event core
> ownership
>=20
> External Email
>=20
> ----------------------------------------------------------------------
> On Fri, Oct 29, 2021 at 05:26:43PM +0530, Bharat Bhushan wrote:
> > As DDR perf event counters are not per core, so they should be
> > accessed only by one core at a time. Select new core when previously
> > owning core is going offline.
> >
> > Signed-off-by: Bharat Bhushan <bbhushan2@marvell.com>
> > ---
> > v1->v6
> >  - No Change
> >
> >  drivers/perf/marvell_cn10k_ddr_pmu.c | 50 ++++++++++++++++++++++++++--
> >  include/linux/cpuhotplug.h           |  1 +
> >  2 files changed, 49 insertions(+), 2 deletions(-)
>=20
> I don't think the driver is much use without this patch, so please can yo=
u move
> the Kconfig stuff to a patch at the end so that the driver can't be enabl=
ed in a
> broken state half way through the series?

Okay, will change.

Thanks
-Bharat

>=20
> Will