Message ID | cover.1627967359.git.gmuthukrishn@marvell.com (mailing list archive) |
---|---|
Headers |
Return-Path: <dev-bounces@dpdk.org> X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 07C89A0C41; Tue, 3 Aug 2021 10:05:55 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 45527411A7; Tue, 3 Aug 2021 10:05:55 +0200 (CEST) Received: from mx0b-0016f401.pphosted.com (mx0b-0016f401.pphosted.com [67.231.156.173]) by mails.dpdk.org (Postfix) with ESMTP id 38F6B40E32 for <dev@dpdk.org>; Tue, 3 Aug 2021 10:05:54 +0200 (CEST) Received: from pps.filterd (m0045851.ppops.net [127.0.0.1]) by mx0b-0016f401.pphosted.com (8.16.0.43/8.16.0.43) with SMTP id 17382EHD009546; Tue, 3 Aug 2021 01:05:53 -0700 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=marvell.com; h=from : to : cc : subject : date : message-id : in-reply-to : references : mime-version : content-transfer-encoding : content-type; s=pfpt0220; bh=+L4yGYJJcU8jRanAPxOqzj9u0JfOAA3JtxE07iujqkk=; b=RZhWXLFk6OYODhcdZwCK3pXXhzEW6Wgbyhdg2ph6p6+84hKssHBSymEdKSXlD54r8HkO VX1tGPcnAiAA0T0ezQHxEzwK4mgLiBA1661LldocpemK4DR0wzuJA791h4ykw8SKlBrl SBnOqCL9oGT/yfaVj74sl0vkpyKWrGbX4XwMVrQ0duICqlcwZM5RE5GsQRHF30aMXtln q/N0sgY0mLGMTDgKKguYQU0SUp22EppELc8NNJ7m7e/jswd6P02myAjfZtXws5vtGumd 95BS0QzPrzQ7V5/bxT0G1e3jyze2zLkm/R6bY9BpzZwhJlH0kp1NGdBTqwfwJaHgm7WK HQ== Received: from dc5-exch02.marvell.com ([199.233.59.182]) by mx0b-0016f401.pphosted.com with ESMTP id 3a6b46v9wr-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-SHA384 bits=256 verify=NOT); Tue, 03 Aug 2021 01:05:53 -0700 Received: from DC5-EXCH01.marvell.com (10.69.176.38) by DC5-EXCH02.marvell.com (10.69.176.39) with Microsoft SMTP Server (TLS) id 15.0.1497.18; Tue, 3 Aug 2021 01:05:51 -0700 Received: from maili.marvell.com (10.69.176.80) by DC5-EXCH01.marvell.com (10.69.176.38) with Microsoft SMTP Server id 15.0.1497.18 via Frontend Transport; Tue, 3 Aug 2021 01:05:51 -0700 Received: from localhost.localdomain (unknown [10.28.34.38]) by maili.marvell.com (Postfix) with ESMTP id 0B5315B694B; Tue, 3 Aug 2021 01:05:48 -0700 (PDT) From: Gowrishankar Muthukrishnan <gmuthukrishn@marvell.com> To: <dev@dpdk.org> CC: <bruce.richardson@intel.com>, <ciara.power@intel.com>, <jerinj@marvell.com>, <kirankumark@marvell.com>, <ndabilpuram@marvell.com>, <skori@marvell.com>, <skoteshwar@marvell.com>, Gowrishankar Muthukrishnan <gmuthukrishn@marvell.com> Date: Tue, 3 Aug 2021 13:35:43 +0530 Message-ID: <cover.1627967359.git.gmuthukrishn@marvell.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <cover.1627839051.git.gmuthukrishn@marvell.com> References: <cover.1627839051.git.gmuthukrishn@marvell.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain X-Proofpoint-GUID: 3C30XVJ3B4HHyIDLXr2Ir5zaEgpNZBOp X-Proofpoint-ORIG-GUID: 3C30XVJ3B4HHyIDLXr2Ir5zaEgpNZBOp X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.391, 18.0.790 definitions=2021-08-03_02:2021-08-03, 2021-08-03 signatures=0 Subject: [dpdk-dev] [v3, 0/3] common/cnxk: enable npa telemetry X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions <dev.dpdk.org> List-Unsubscribe: <https://mails.dpdk.org/options/dev>, <mailto:dev-request@dpdk.org?subject=unsubscribe> List-Archive: <http://mails.dpdk.org/archives/dev/> List-Post: <mailto:dev@dpdk.org> List-Help: <mailto:dev-request@dpdk.org?subject=help> List-Subscribe: <https://mails.dpdk.org/listinfo/dev>, <mailto:dev-request@dpdk.org?subject=subscribe> Errors-To: dev-bounces@dpdk.org Sender: "dev" <dev-bounces@dpdk.org> |
Series |
common/cnxk: enable npa telemetry
|
|
Message
Gowrishankar Muthukrishnan
Aug. 3, 2021, 8:05 a.m. UTC
This patch series enables telemetry in NPA LF of cnxk. v3: - fixed format specifier for uintptr_t Gowrishankar Muthukrishnan (3): telemetry: enable storing pointer value test/telemetry: add unit tests for pointer value common/cnxk: add telemetry endpoints to npa app/test/test_telemetry_data.c | 125 +++++++++++++ app/test/test_telemetry_json.c | 29 ++- drivers/common/cnxk/cnxk_telemetry.h | 26 +++ drivers/common/cnxk/cnxk_telemetry_npa.c | 227 +++++++++++++++++++++++ drivers/common/cnxk/meson.build | 4 + drivers/common/cnxk/roc_platform.h | 8 + lib/telemetry/rte_telemetry.h | 37 +++- lib/telemetry/telemetry.c | 21 ++- lib/telemetry/telemetry_data.c | 40 +++- lib/telemetry/telemetry_data.h | 2 + lib/telemetry/telemetry_json.h | 32 ++++ lib/telemetry/version.map | 2 + 12 files changed, 539 insertions(+), 14 deletions(-) create mode 100644 drivers/common/cnxk/cnxk_telemetry.h create mode 100644 drivers/common/cnxk/cnxk_telemetry_npa.c
Comments
Hi Gowrishankar, >-----Original Message----- >From: Gowrishankar Muthukrishnan <gmuthukrishn@marvell.com> >Sent: Tuesday 3 August 2021 09:06 >To: dev@dpdk.org >Cc: Richardson, Bruce <bruce.richardson@intel.com>; Power, Ciara ><ciara.power@intel.com>; jerinj@marvell.com; kirankumark@marvell.com; >ndabilpuram@marvell.com; skori@marvell.com; skoteshwar@marvell.com; >Gowrishankar Muthukrishnan <gmuthukrishn@marvell.com> >Subject: [v3, 0/3] common/cnxk: enable npa telemetry > >This patch series enables telemetry in NPA LF of cnxk. > >v3: > - fixed format specifier for uintptr_t > >Gowrishankar Muthukrishnan (3): > telemetry: enable storing pointer value > test/telemetry: add unit tests for pointer value > common/cnxk: add telemetry endpoints to npa > > app/test/test_telemetry_data.c | 125 +++++++++++++ > app/test/test_telemetry_json.c | 29 ++- > drivers/common/cnxk/cnxk_telemetry.h | 26 +++ > drivers/common/cnxk/cnxk_telemetry_npa.c | 227 >+++++++++++++++++++++++ > drivers/common/cnxk/meson.build | 4 + > drivers/common/cnxk/roc_platform.h | 8 + > lib/telemetry/rte_telemetry.h | 37 +++- > lib/telemetry/telemetry.c | 21 ++- > lib/telemetry/telemetry_data.c | 40 +++- > lib/telemetry/telemetry_data.h | 2 + > lib/telemetry/telemetry_json.h | 32 ++++ > lib/telemetry/version.map | 2 + > 12 files changed, 539 insertions(+), 14 deletions(-) create mode 100644 >drivers/common/cnxk/cnxk_telemetry.h > create mode 100644 drivers/common/cnxk/cnxk_telemetry_npa.c > >-- >2.25.1 I am still unsure exactly what the use case is here - why are we choosing to publish the pointer values through telemetry rather than using a debug log for example? Maybe I am missing something here. Thanks, Ciara
Hi Ciara, > -----Original Message----- > From: Power, Ciara <ciara.power@intel.com> > Sent: Wednesday, August 11, 2021 9:30 PM > To: Gowrishankar Muthukrishnan <gmuthukrishn@marvell.com>; dev@dpdk.org > Cc: Richardson, Bruce <bruce.richardson@intel.com>; Jerin Jacob Kollanukkaran > <jerinj@marvell.com>; Kiran Kumar Kokkilagadda <kirankumark@marvell.com>; > Nithin Kumar Dabilpuram <ndabilpuram@marvell.com>; Sunil Kumar Kori > <skori@marvell.com>; Satha Koteswara Rao Kottidi > <skoteshwar@marvell.com> > Subject: [EXT] RE: [v3, 0/3] common/cnxk: enable npa telemetry > > External Email > > ---------------------------------------------------------------------- > Hi Gowrishankar, > > >-----Original Message----- > >From: Gowrishankar Muthukrishnan <gmuthukrishn@marvell.com> > >Sent: Tuesday 3 August 2021 09:06 > >To: dev@dpdk.org > >Cc: Richardson, Bruce <bruce.richardson@intel.com>; Power, Ciara > ><ciara.power@intel.com>; jerinj@marvell.com; kirankumark@marvell.com; > >ndabilpuram@marvell.com; skori@marvell.com; skoteshwar@marvell.com; > >Gowrishankar Muthukrishnan <gmuthukrishn@marvell.com> > >Subject: [v3, 0/3] common/cnxk: enable npa telemetry > > > >This patch series enables telemetry in NPA LF of cnxk. > > > >v3: > > - fixed format specifier for uintptr_t > > > >Gowrishankar Muthukrishnan (3): > > telemetry: enable storing pointer value > > test/telemetry: add unit tests for pointer value > > common/cnxk: add telemetry endpoints to npa > > > > app/test/test_telemetry_data.c | 125 +++++++++++++ > > app/test/test_telemetry_json.c | 29 ++- > > drivers/common/cnxk/cnxk_telemetry.h | 26 +++ > > drivers/common/cnxk/cnxk_telemetry_npa.c | 227 > >+++++++++++++++++++++++ > > drivers/common/cnxk/meson.build | 4 + > > drivers/common/cnxk/roc_platform.h | 8 + > > lib/telemetry/rte_telemetry.h | 37 +++- > > lib/telemetry/telemetry.c | 21 ++- > > lib/telemetry/telemetry_data.c | 40 +++- > > lib/telemetry/telemetry_data.h | 2 + > > lib/telemetry/telemetry_json.h | 32 ++++ > > lib/telemetry/version.map | 2 + > > 12 files changed, 539 insertions(+), 14 deletions(-) create mode > >100644 drivers/common/cnxk/cnxk_telemetry.h > > create mode 100644 drivers/common/cnxk/cnxk_telemetry_npa.c > > > >-- > >2.25.1 > > I am still unsure exactly what the use case is here - why are we choosing to > publish the pointer values through telemetry rather than using a debug log for > example? Pointer values are useful sometimes for more debugging through telemetry, hence this proposal. As I mentioned in v1 thread, this is architecture compliant approach rather than assuming pointer value is always 64 bit, when there is need to use pointer value in current telemetry path. Thanks, Gowrishankar > Maybe I am missing something here. > > Thanks, > Ciara
Hi, Please let me know if any other info is required. Thanks, Gowrishankar > -----Original Message----- > From: dev <dev-bounces@dpdk.org> On Behalf Of Gowrishankar Muthukrishnan > Sent: Wednesday, August 11, 2021 9:49 PM > To: Power, Ciara <ciara.power@intel.com>; dev@dpdk.org > Cc: Richardson, Bruce <bruce.richardson@intel.com>; Jerin Jacob Kollanukkaran > <jerinj@marvell.com>; Kiran Kumar Kokkilagadda <kirankumark@marvell.com>; > Nithin Kumar Dabilpuram <ndabilpuram@marvell.com>; Sunil Kumar Kori > <skori@marvell.com>; Satha Koteswara Rao Kottidi > <skoteshwar@marvell.com> > Subject: [EXT] Re: [dpdk-dev] [v3, 0/3] common/cnxk: enable npa telemetry > > External Email > > ---------------------------------------------------------------------- > Hi Ciara, > > > -----Original Message----- > > From: Power, Ciara <ciara.power@intel.com> > > Sent: Wednesday, August 11, 2021 9:30 PM > > To: Gowrishankar Muthukrishnan <gmuthukrishn@marvell.com>; > > dev@dpdk.org > > Cc: Richardson, Bruce <bruce.richardson@intel.com>; Jerin Jacob > > Kollanukkaran <jerinj@marvell.com>; Kiran Kumar Kokkilagadda > > <kirankumark@marvell.com>; Nithin Kumar Dabilpuram > > <ndabilpuram@marvell.com>; Sunil Kumar Kori <skori@marvell.com>; Satha > > Koteswara Rao Kottidi <skoteshwar@marvell.com> > > Subject: [EXT] RE: [v3, 0/3] common/cnxk: enable npa telemetry > > > > External Email > > > > ---------------------------------------------------------------------- > > Hi Gowrishankar, > > > > >-----Original Message----- > > >From: Gowrishankar Muthukrishnan <gmuthukrishn@marvell.com> > > >Sent: Tuesday 3 August 2021 09:06 > > >To: dev@dpdk.org > > >Cc: Richardson, Bruce <bruce.richardson@intel.com>; Power, Ciara > > ><ciara.power@intel.com>; jerinj@marvell.com; kirankumark@marvell.com; > > >ndabilpuram@marvell.com; skori@marvell.com; skoteshwar@marvell.com; > > >Gowrishankar Muthukrishnan <gmuthukrishn@marvell.com> > > >Subject: [v3, 0/3] common/cnxk: enable npa telemetry > > > > > >This patch series enables telemetry in NPA LF of cnxk. > > > > > >v3: > > > - fixed format specifier for uintptr_t > > > > > >Gowrishankar Muthukrishnan (3): > > > telemetry: enable storing pointer value > > > test/telemetry: add unit tests for pointer value > > > common/cnxk: add telemetry endpoints to npa > > > > > > app/test/test_telemetry_data.c | 125 +++++++++++++ > > > app/test/test_telemetry_json.c | 29 ++- > > > drivers/common/cnxk/cnxk_telemetry.h | 26 +++ > > > drivers/common/cnxk/cnxk_telemetry_npa.c | 227 > > >+++++++++++++++++++++++ > > > drivers/common/cnxk/meson.build | 4 + > > > drivers/common/cnxk/roc_platform.h | 8 + > > > lib/telemetry/rte_telemetry.h | 37 +++- > > > lib/telemetry/telemetry.c | 21 ++- > > > lib/telemetry/telemetry_data.c | 40 +++- > > > lib/telemetry/telemetry_data.h | 2 + > > > lib/telemetry/telemetry_json.h | 32 ++++ > > > lib/telemetry/version.map | 2 + > > > 12 files changed, 539 insertions(+), 14 deletions(-) create mode > > >100644 drivers/common/cnxk/cnxk_telemetry.h > > > create mode 100644 drivers/common/cnxk/cnxk_telemetry_npa.c > > > > > >-- > > >2.25.1 > > > > I am still unsure exactly what the use case is here - why are we > > choosing to publish the pointer values through telemetry rather than > > using a debug log for example? > > Pointer values are useful sometimes for more debugging through telemetry, > hence this proposal. > As I mentioned in v1 thread, this is architecture compliant approach rather than > assuming pointer value is always 64 bit, when there is need to use pointer value > in current telemetry path. > > Thanks, > Gowrishankar > > > Maybe I am missing something here. > > > > Thanks, > > Ciara
11/08/2021 18:18, Gowrishankar Muthukrishnan: > From: Power, Ciara <ciara.power@intel.com> > > From: Gowrishankar Muthukrishnan <gmuthukrishn@marvell.com> > > > telemetry: enable storing pointer value [...] > > I am still unsure exactly what the use case is here - why are we choosing to > > publish the pointer values through telemetry rather than using a debug log for > > example? > > Pointer values are useful sometimes for more debugging through telemetry, hence this proposal. > As I mentioned in v1 thread, this is architecture compliant approach rather than assuming pointer > value is always 64 bit, when there is need to use pointer value in current telemetry path. Why using telemetry for debugging? I agree with Ciara, debug log is more appropriate. You can also use the tracing framework.
Hi Thomas, > [...] > > > I am still unsure exactly what the use case is here - why are we > > > choosing to publish the pointer values through telemetry rather than > > > using a debug log for example? > > > > Pointer values are useful sometimes for more debugging through telemetry, > hence this proposal. > > As I mentioned in v1 thread, this is architecture compliant approach > > rather than assuming pointer value is always 64 bit, when there is need to use > pointer value in current telemetry path. > > Why using telemetry for debugging? > I agree with Ciara, debug log is more appropriate. > You can also use the tracing framework. > In addition to getting status/stats, we are using this library for remote debugging as this is the only mechanism available in DPDK now. For an instance, accessing data from a pointer address through a root privileged tool. We can internally type cast them to u64 as well but thought of standardizing the conversion through library API instead in case it is useful in other platforms too. If it does not seem more useful for now, I will take another approach on type casting the values internally in our driver (and avoiding library patch). Please share your thoughts. Thanks, Gowrishankar