From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id D905AA0547; Sun, 16 May 2021 19:34:08 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 52FFE40041; Sun, 16 May 2021 19:34:08 +0200 (CEST) Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [216.205.24.124]) by mails.dpdk.org (Postfix) with ESMTP id 4C1974003C for ; Sun, 16 May 2021 19:34:07 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1621186446; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=MaY3ArneS6wWyB6Yu7KZnBpcY1++aGl0zxeIkuTV30M=; b=W7nmIjbXZvPzLOuzRfUXeumgD/FY/jc0V40+hD5Ac+THEtmE1TUnPWFQ3x2BHNjH23+J4R lySrUvAXVHL4ow0MkOQcPiXII4auZuMHfjyxJ4PSIIQojDOFua7Mfc0JeipSAC/VG3urW+ 2weTnw+Tb3V3aDAY2CZcKLE633BifuI= Received: from mail-ua1-f70.google.com (mail-ua1-f70.google.com [209.85.222.70]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-173-uJ7Y8EYfPB6bXuqtd3MSDg-1; Sun, 16 May 2021 13:34:04 -0400 X-MC-Unique: uJ7Y8EYfPB6bXuqtd3MSDg-1 Received: by mail-ua1-f70.google.com with SMTP id v7-20020a9f31870000b029020b2d4ea5d9so2522729uad.8 for ; Sun, 16 May 2021 10:34:04 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=MaY3ArneS6wWyB6Yu7KZnBpcY1++aGl0zxeIkuTV30M=; b=MlXzXv/nHnoiIUmFfc7GzXGr85SehsKUVpfFgiZ9hP4RXd+Q2aVgr1HheG5DIa0gUz oAA43WfidgDiHlPQ+T1gQVK+ZOajhlDWn0QzcnBJxdHqHNY7AokX+LxT2QrihVZkamYZ twXv4xUGW0Iryld7yWg0sKhV39O/lbT3+oeRUtzGzW9txHDlzaAQ77MTdob5UrhqnVe2 LvDwo11Mb1mnuIyAtXgNPlZoNp3CWJiSY64qNafh04U5t8fc1t85o2zOYUW5j5gg5A2Y LaSKNjfpUZ2sL6A9NfSOlSruFG+wPsItBZjZK/aDk98aC07PQpMX0RirmxXwPPMB+ulF vQbg== X-Gm-Message-State: AOAM533dZYIIMY9Ys6JPMYBrqJtetDpqBkisgnRk4a6/cDA9zSUIHyJM I76DfmR4e1BuLUr19M9esAQYX2M+YZ3ml3ThSBChX2j2VBfIacwqEF5ZzsCVgceNXf4XweQj0qo cduKaLwj2f60rkY+tgCk= X-Received: by 2002:ab0:132a:: with SMTP id g39mr19307961uae.53.1621186444150; Sun, 16 May 2021 10:34:04 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwi1R98Zuy5qp9V1DADcOWI18bm+OiXsYSo+BHmzF3tVhpeOIVeQT9HGOygnOnzj7eQ6ezjM0aSgj3GR8+gp8E= X-Received: by 2002:ab0:132a:: with SMTP id g39mr19307950uae.53.1621186443730; Sun, 16 May 2021 10:34:03 -0700 (PDT) MIME-Version: 1.0 References: <1621099654-25535-1-git-send-email-timothy.mcdaniel@intel.com> In-Reply-To: <1621099654-25535-1-git-send-email-timothy.mcdaniel@intel.com> From: David Marchand Date: Sun, 16 May 2021 19:33:52 +0200 Message-ID: To: Timothy McDaniel Cc: dev , Erik Gabriel Carrillo , Van Haaren Harry , Jerin Jacob Kollanukkaran , Thomas Monjalon Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=dmarchan@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="UTF-8" Subject: Re: [dpdk-dev] [PATCH] doc: fix guide for DLB v2.5 X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 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" On Sat, May 15, 2021 at 7:29 PM Timothy McDaniel wrote: > > - Remove references to deferred scheduling. That feature applies > to DLB v1.0 only. > - Replace vdev references with the pci devargs equivalent > - Add section for new "vector_opts_enabled" devarg > > Fixes: 7c6cc633fc7d ("doc: update guide for DLB v2.5") > Cc: timothy.mcdaniel@intel.com > > Signed-off-by: Timothy McDaniel > --- > doc/guides/eventdevs/dlb2.rst | 54 +++++++++++++++-------------------- > 1 file changed, 23 insertions(+), 31 deletions(-) > > diff --git a/doc/guides/eventdevs/dlb2.rst b/doc/guides/eventdevs/dlb2.rst > index 31de6bc47..bce984ca0 100644 > --- a/doc/guides/eventdevs/dlb2.rst > +++ b/doc/guides/eventdevs/dlb2.rst > @@ -152,19 +152,19 @@ These pools' sizes are controlled by the nb_events_limit field in struct > rte_event_dev_config. The load-balanced pool is sized to contain > nb_events_limit credits, and the directed pool is sized to contain > nb_events_limit/4 credits. The directed pool size can be overridden with the > -num_dir_credits vdev argument, like so: > +num_dir_credits devargs argument, like so: > > .. code-block:: console > > - --vdev=dlb2_event,num_dir_credits= > + --allow ea:00.0,num_dir_credits= > > This can be used if the default allocation is too low or too high for the > -specific application needs. The PMD also supports a vdev arg that limits the > +specific application needs. The PMD also supports a devarg that limits the > max_num_events reported by rte_event_dev_info_get(): > > .. code-block:: console > > - --vdev=dlb2_event,max_num_events= > + --allow ea:00.0,max_num_events= > > By default, max_num_events is reported as the total available load-balanced > credits. If multiple DLB-based applications are being used, it may be desirable > @@ -293,27 +293,6 @@ The PMD does not support the following configuration sequences: > This sequence is not supported because the event device must be reconfigured > before its ports or queues can be. > > -Deferred Scheduling > -~~~~~~~~~~~~~~~~~~~ > - > -The DLB PMD's default behavior for managing a CQ is to "pop" the CQ once per > -dequeued event before returning from rte_event_dequeue_burst(). This frees the > -corresponding entries in the CQ, which enables the DLB to schedule more events > -to it. > - > -To support applications seeking finer-grained scheduling control -- for example > -deferring scheduling to get the best possible priority scheduling and > -load-balancing -- the PMD supports a deferred scheduling mode. In this mode, > -the CQ entry is not popped until the *subsequent* rte_event_dequeue_burst() > -call. This mode only applies to load-balanced event ports with dequeue depth of > -1. > - > -To enable deferred scheduling, use the defer_sched vdev argument like so: > - > - .. code-block:: console > - > - --vdev=dlb2_event,defer_sched=on > - > Atomic Inflights Allocation > ~~~~~~~~~~~~~~~~~~~~~~~~~~~ > > @@ -336,11 +315,11 @@ buffer space (e.g. if not all queues are used, or aren't used for atomic > scheduling). > > The PMD provides a dev arg to override the default per-queue allocation. To > -increase a vdev's per-queue atomic-inflight allocation to (for example) 64: > +increase per-queue atomic-inflight allocation to (for example) 64: > > .. code-block:: console > > - --vdev=dlb2_event,atm_inflights=64 > + --allow ea:00.0,atm_inflights=64 > > QID Depth Threshold > ~~~~~~~~~~~~~~~~~~~ > @@ -363,9 +342,9 @@ shown below. > > .. code-block:: console > > - --vdev=dlb2_event,qid_depth_thresh=all: > - --vdev=dlb2_event,qid_depth_thresh=qidA-qidB: > - --vdev=dlb2_event,qid_depth_thresh=qid: > + --allow ea:00.0,qid_depth_thresh=all: > + --allow ea:00.0,qid_depth_thresh=qidA-qidB: > + --allow ea:00.0,qid_depth_thresh=qid: Did you try this syntax? The previous syntax with vdev did not work, and the new one probably won't either. Only the first devargs will be passed to the driver. Example with the null pmd: $ ./build/app/dpdk-testpmd -m 512 --no-huge --log-level=*:debug --vdev net_null,copy=1 --vdev net_null,size=1024 -- -ia --total-num-mbufs=2048 ... vdev_probe_all_drivers(): Search driver to probe device net_null rte_pmd_null_probe(): Initializing pmd_null for net_null rte_pmd_null_probe(): Configure pmd_null: packet size is 64, packet copy is enabled ^^ ^^^^^^^ eth_dev_null_create(): Creating null ethdev on numa socket 0 ... If you want to pass multiple options for a single device, pass them all at once. Like: --allow ea:00.0,qid_depth_thresh=all:,qid_depth_thresh=qidA-qidB:,qid_depth_thresh=qid: > > Class of service > ~~~~~~~~~~~~~~~~ > @@ -387,4 +366,17 @@ Class of service can be specified in the devargs, as follows > > .. code-block:: console > > - --vdev=dlb2_event,cos=<0..4> > + --allow ea:00.0,cos=<0..4> > + > +Use X86 Vector Instructions > +~~~~~~~~~~~~~~~~~~~~~~~~~~~ > + > +DLB supports using x86 vector instructions to optimize the data path. > + > +The default mode of operation is to use scalar instructions, but > +the use of vector instructions can be enabled in the devargs, as > +follows > + > + .. code-block:: console > + > + --allow ea:00.0,vector_opts_enabled= This option does not exist. All I see is: drivers/event/dlb2/dlb2_priv.h:#define DLB2_VECTOR_OPTS_DISAB_ARG "vector_opts_disable" What of --force-max-simd-bitwidth ? -- David Marchand