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 gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (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 99798C4167B for ; Mon, 11 Dec 2023 06:35:51 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 209FC10E085; Mon, 11 Dec 2023 06:35:51 +0000 (UTC) Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.11]) by gabe.freedesktop.org (Postfix) with ESMTPS id 0254910E085 for ; Mon, 11 Dec 2023 06:35:49 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1702276550; x=1733812550; h=date:message-id:from:to:cc:subject:in-reply-to: references:mime-version; bh=JAy0LhAfmumsWeFA2tSCOU/AVT1/2DjcpmYsZijZOKk=; b=FWt1t9wumn27TeDh6viXN5uxDt9TkYr0nWQHDBRJZ+5jLgCRpVMV0sqW KoztO603MWg37wyUS5yw59TvprEza9pyKs7pmh3LyFWS2op89ytqYgrn3 gKEUl8CfcOxh5RX0hnqDRlHlVVsq7W5vXXnnnQHc2P2uba67HPpp6nHZY eTE4RVJQcyYKfqDvk+/iqY1eKrcLefyJG6yUOC4j/HCsJAFP+7ry5dKQp DBS3bG8EanLJ+lMYL/UiOluW5iexNTYlcFOCit3dl5w0arLXmz8NU9eq5 4FeNm5RujvWFXuSuhlWpNIgoMu0p5IRpS0iEMqlPPG1clE/MlIW8E5MNa A==; X-IronPort-AV: E=McAfee;i="6600,9927,10920"; a="1438787" X-IronPort-AV: E=Sophos;i="6.04,267,1695711600"; d="scan'208";a="1438787" Received: from orsmga002.jf.intel.com ([10.7.209.21]) by orvoesa103.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 10 Dec 2023 22:35:49 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10920"; a="772928145" X-IronPort-AV: E=Sophos;i="6.04,267,1695711600"; d="scan'208";a="772928145" Received: from adixit-mobl.amr.corp.intel.com (HELO adixit-arch.intel.com) ([10.209.47.126]) by orsmga002-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 10 Dec 2023 22:35:49 -0800 Date: Sun, 10 Dec 2023 22:35:48 -0800 Message-ID: <87il55i6uz.wl-ashutosh.dixit@intel.com> From: "Dixit, Ashutosh" To: Aravind Iddamsetty Subject: Re: [PATCH] drm/xe/pmu: Remove PMU from Xe till uapi is finalized In-Reply-To: References: <20231209224352.2720403-1-ashutosh.dixit@intel.com> <87msujhtu3.wl-ashutosh.dixit@intel.com> User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI-EPG/1.14.7 (Harue) FLIM-LB/1.14.9 (=?ISO-8859-4?Q?Goj=F2?=) APEL-LB/10.8 EasyPG/1.0.0 Emacs/29.1 (x86_64-pc-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") Content-Type: text/plain; charset=US-ASCII X-BeenThere: intel-xe@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Intel Xe graphics driver List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: francois.dugast@intel.com, Lucas De Marchi , Rodrigo Vivi , intel-xe@lists.freedesktop.org Errors-To: intel-xe-bounces@lists.freedesktop.org Sender: "Intel-xe" On Sun, 10 Dec 2023 22:04:25 -0800, Aravind Iddamsetty wrote: > Hi Aravind, > On 12/10/23 04:22, Dixit, Ashutosh wrote: > > On Sat, 09 Dec 2023 14:43:52 -0800, Ashutosh Dixit wrote: > >> PMU uapi is likely to change in the future. Till the uapi is finalized, > >> remove PMU from Xe. PMU can be re-added after uapi is finalized. > > Hi Ashutosh, > > Is it possible to have the infra as it is and just remove the events > part. If you want to send a patch which can do this cleanly please go ahead. I was trying it but it was looking odd that the infra is there but no events were exposed. > As there is follow on series from Riana which uses the infra > https://patchwork.freedesktop.org/series/126919/. or the other > alternative would be for Riana to bring in the infra as part of her > series. So that is why I thought it would be easier if deleted the whole thing now and later Riana or Umesh could add an initial patch, authored by you, which adds the PMU infrastructure and then add their part in separate patches. Thanks. -- Ashutosh > > Complete removal of PMU for now is probably preferable than what was done > > in: > > > > https://patchwork.freedesktop.org/series/127590/ > > > > Also, it is probably preferable to remove PMU code via a rebase, but if > > that is time consuming, adding this patch will remove PMU code. > > > > I was trying to send a series of 'fixup' patches, but that seemed difficult > > because a large number of patches have touched non-PMU files, so all those > > patches will need to be fixed up. > > > > Thanks. > > -- > > Ashutosh