From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 7DFA6A0350; Wed, 1 Jul 2020 12:34:46 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id CD74F1C22A; Wed, 1 Jul 2020 12:34:45 +0200 (CEST) Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by dpdk.org (Postfix) with ESMTP id CA6041C21E for ; Wed, 1 Jul 2020 12:34:44 +0200 (CEST) IronPort-SDR: vy/2eEoCZnlY7HPOdKq3h8kko9muKAL+RWwSXDYPYK0Dz5YIKa9TKzhq/VKWeZy2+SQJfLaSQ5 IwN98esRAtIw== X-IronPort-AV: E=McAfee;i="6000,8403,9668"; a="144705744" X-IronPort-AV: E=Sophos;i="5.75,299,1589266800"; d="scan'208";a="144705744" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga003.jf.intel.com ([10.7.209.27]) by fmsmga102.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 01 Jul 2020 03:34:43 -0700 IronPort-SDR: PrsSC1L7wIe/v/BlordBIgc9XJ6BfSxXejVuK1E/GM2A8qYg0AcT8aUsBDgUw1LoRsRNBoUh0w 48QapoIVmJLA== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.75,299,1589266800"; d="scan'208";a="277735853" Received: from orsmsx109.amr.corp.intel.com ([10.22.240.7]) by orsmga003.jf.intel.com with ESMTP; 01 Jul 2020 03:34:43 -0700 Received: from ORSEDG002.ED.cps.intel.com (10.7.248.5) by ORSMSX109.amr.corp.intel.com (10.22.240.7) with Microsoft SMTP Server (TLS) id 14.3.439.0; Wed, 1 Jul 2020 03:34:43 -0700 Received: from NAM02-CY1-obe.outbound.protection.outlook.com (104.47.37.56) by edgegateway.intel.com (134.134.137.101) with Microsoft SMTP Server (TLS) id 14.3.439.0; Wed, 1 Jul 2020 03:34:43 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ODnJOv9MOX5A4WwzKDK3E5CrUY7hYzvG40xgPGSkSdDUlr8xe2rNoDoJ8dEImcU2mPsspiKXjJh4xfirQY8XPoMqArx8XaVLBUqZGLm7/Pm8TVeQcQ33FLISaWyLezSNCFzN6eM6RssRkz4N9gJkpLnXss2mYE4pe9uUmr+ewceef/rz8ua9oiB2QoPS+dOrMGTHu7Wn9uoFwTvq7gEYvr6GGYDArV7IScYPVukqOUT3njSl4ydcAW/Oii4QdpD0Df9KKQfrb6xtwjATUz89j26uqR7rV2/BTA6+yhXSuUDbcfS4TF6/6+LcDM0YfJEK9qhXf6vTiKCrBueewlo6BA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=16GPifNDGC3OzuY9Bk1tErfUBkeMlKZikiabeUEwYYo=; b=mOteAJ6pGysLIeH5Mo4EcIlY2jYIAnl6JIMrx4TX0yoyxBSVLQ/cEETM4wWZ9iWtmnXonL/oBg905on1V4jQwXbs/hvpr9eMsriNu5Q1W0yW66rXLfrfsuhIK2Gkl3mWrvkdQ9KOa8IHnuPCa8rIOSRjCGc5Bmux7yJZ0bm2uKqnahll/x7AhohbUjMoaEp/Gol76lyUoSP3SMsT40QqEWrhI4dhu665JfY0exb3rXGq3qCU6k3MptKkb7fHtLgfIcMStEV8lXqPLJcoo5qNFrCmr0PFYK9A3+sIvoGcLahG+mFDNoD1Og+iOeHACQQ2g90NpAY4kOZld7MzgCwDCg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com; dkim=pass header.d=intel.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=intel.onmicrosoft.com; s=selector2-intel-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=16GPifNDGC3OzuY9Bk1tErfUBkeMlKZikiabeUEwYYo=; b=LOd9d41AQ13H/o0HPoeqOimF8OU8WSwGvyAI/k7LFVydB3R19bHssOtF7KT0odiS30KSci0ywEVlD5yq5ujGhSjI2K84ajecQOSRQtxupHkoTIzKwIv6mKsmQy09Jw9262a97Rcdy6/cjzKcmq20aYKm3//fp9/cTkTAVp97gu4= Received: from BYAPR11MB3301.namprd11.prod.outlook.com (2603:10b6:a03:7f::26) by BY5PR11MB4307.namprd11.prod.outlook.com (2603:10b6:a03:1bd::27) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3131.23; Wed, 1 Jul 2020 10:34:42 +0000 Received: from BYAPR11MB3301.namprd11.prod.outlook.com ([fe80::f160:29ab:b8f9:4189]) by BYAPR11MB3301.namprd11.prod.outlook.com ([fe80::f160:29ab:b8f9:4189%6]) with mapi id 15.20.3131.027; Wed, 1 Jul 2020 10:34:41 +0000 From: "Ananyev, Konstantin" To: Honnappa Nagarahalli , "dev@dpdk.org" CC: Feifei Wang , nd , nd Thread-Topic: [PATCH] test/ring: add stress test for ST peek API Thread-Index: AQHWTvqjPUKKDGLQpUOJXRrsAoUBD6jxxsIAgADBJVA= Date: Wed, 1 Jul 2020 10:34:41 +0000 Message-ID: References: <20200630162139.19512-1-konstantin.ananyev@intel.com> In-Reply-To: Accept-Language: en-GB, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-reaction: no-action dlp-version: 11.2.0.6 authentication-results: arm.com; dkim=none (message not signed) header.d=none;arm.com; dmarc=none action=none header.from=intel.com; x-originating-ip: [192.198.151.183] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 2e615cd8-ba87-41b5-c039-08d81daa5d40 x-ms-traffictypediagnostic: BY5PR11MB4307: x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:6790; x-forefront-prvs: 04519BA941 x-ms-exchange-senderadcheck: 1 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: Ca71eDISkqWRI20Yh0qhv/CxrhalbEAhjzwiPd1gu+4jVgG3Y5trDsompk+e5qGhfCJ1nNzzxd7QCRhcXAwGGSKqvbUrm2qq/N6rAG2SZnFwMFyoDicZTK5X4XTU1Q8tp9/Ju7FvriYlv43s1a8bzKs9x/uLk/uS16XoqLYT6gKxQJ1W+GaoSlCDa+/BHrLDtv5gV1SrK/i+E8kJEdeGYlb+UD5qk0ovptQ+9CVvKra/Y7TEEymY/DR7sjylTs5HxALr4G2pyRKuk2rzpeBNznmg6DIXc/5FMYCprEAZh7FsSchqpgZqFJ/mPRBIPBk8q/TFw4IGz/VIMQ3bX5lkoMDDlyqY4bYhST4d/kVxKS0yU+Acjf2fMumnhwr+jDg9X/n+rR11If/+XGrMxYubSA== x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:BYAPR11MB3301.namprd11.prod.outlook.com; PTR:; CAT:NONE; SFTY:; SFS:(4636009)(39860400002)(376002)(136003)(346002)(396003)(366004)(8676002)(478600001)(33656002)(7696005)(186003)(83380400001)(71200400001)(26005)(8936002)(66946007)(2906002)(86362001)(6506007)(54906003)(76116006)(66446008)(66556008)(66476007)(52536014)(64756008)(110136005)(4326008)(5660300002)(9686003)(55016002)(316002); DIR:OUT; SFP:1102; x-ms-exchange-antispam-messagedata: lqG3nqXyWQz5jP6yTBao4sxkeyVyKhcDmZ0p72WuLEBg3A59Wxm32yQgWB/6qWb4lFK5lrhEPPiOHtmwUR/R5IimTylXuqTHnU1NodUyq9nOPMbyG6obfcqp6QRV6l9/L4Q1P/rie8VTNQpRdylKpVMfR5Dplor4Xr5ZrZzbSwRTvNv8wtHoveOcF7wN250CqWrA8A9co01IUkREYbXJqBdjL8DaeyWqIIMKSr7UEPljJvKzxaGJHPCz0yplBPLblLJ/iurMbeFPOAxaCb3gfZpg7+ddOlw6odTo5F6oTDGkA99l3Pd4St5PAHRsw9+SFe7XysCmAAxWBcOPaBoQiJVv1jMxiS4Z7562GaMRCbhpWnhOujIRJ0Rxlw/jksb8Ayk1we+urQDs1hbQdVEUZ8+8fc7T5wiMb08eBdICjIv5JPAtk6p/h91bx+G5scgdoG8WU7ZVB1sZfIqBbewumtXbMJFXpT9Q9S3CLVDU26XXKyobaRtK8+o11IQZNadO x-ms-exchange-transport-forked: True Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: BYAPR11MB3301.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 2e615cd8-ba87-41b5-c039-08d81daa5d40 X-MS-Exchange-CrossTenant-originalarrivaltime: 01 Jul 2020 10:34:41.8046 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: VuB9B8MXQY4K7ikBcJRDdB/W1bcin+7WivbpIHntDVoi0I14TQhrQC1rmNp8tnYgHw4y+VaDsAQxxrYdr0rxs/x6cbArSGd3inEu66U1uz4= X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY5PR11MB4307 X-OriginatorOrg: intel.com Subject: Re: [dpdk-dev] [PATCH] test/ring: add stress test for ST peek API X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Hi Honnappa, >=20 > Hi Konstantin, > It looks fine overall, few comments inline. >=20 > >=20 > > Subject: [PATCH] test/ring: add stress test for ST peek API > > > > Introduce new test case to test ST peek API. > > > > Signed-off-by: Konstantin Ananyev > > --- > > > > This patch depends on the following patch: > > "ring: fix error vlaue of tail in the peek API for ST mode" > > (http://patches.dpdk.org/patch/72374/) > > to run successfully. > > > > app/test/Makefile | 1 + > > app/test/meson.build | 1 + > > app/test/test_ring_st_peek_stress.c | 54 +++++++++++++++++++++++++++++ > > app/test/test_ring_stress.c | 3 ++ > > app/test/test_ring_stress.h | 1 + > > 5 files changed, 60 insertions(+) > > create mode 100644 app/test/test_ring_st_peek_stress.c > > > > diff --git a/app/test/Makefile b/app/test/Makefile index > > 7b96a03a6..37bdaf891 100644 > > --- a/app/test/Makefile > > +++ b/app/test/Makefile > > @@ -83,6 +83,7 @@ SRCS-y +=3D test_ring_hts_stress.c SRCS-y +=3D > > test_ring_perf.c SRCS-y +=3D test_ring_peek_stress.c SRCS-y +=3D > > test_ring_rts_stress.c > > +SRCS-y +=3D test_ring_st_peek_stress.c > > SRCS-y +=3D test_ring_stress.c > > SRCS-y +=3D test_pmd_perf.c > > > > diff --git a/app/test/meson.build b/app/test/meson.build index > > 5233ead46..4ec7da6b2 100644 > > --- a/app/test/meson.build > > +++ b/app/test/meson.build > > @@ -108,6 +108,7 @@ test_sources =3D files('commands.c', > > 'test_ring_peek_stress.c', > > 'test_ring_perf.c', > > 'test_ring_rts_stress.c', > > + 'test_ring_st_peek_stress.c', > I think we should rename test_ring_peek_stress.c to test_ring_mpmc_hts_pe= ek_stress.c to be consistent with this? Ok, maybe then: 'test_ring_st_peek_stress.c' and 'test_ring_mt_peek_stress.c' to keep naming convention the same? >=20 > > 'test_ring_stress.c', > > 'test_rwlock.c', > > 'test_sched.c',