Message ID | 20221004171656.17967-1-nicolas.chautru@intel.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 67DDBA0542; Tue, 4 Oct 2022 19:17:09 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 110F240E2D; Tue, 4 Oct 2022 19:17:09 +0200 (CEST) Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by mails.dpdk.org (Postfix) with ESMTP id A8D8D40A79 for <dev@dpdk.org>; Tue, 4 Oct 2022 19:17:07 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1664903827; x=1696439827; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=McPt39efSQ2EwARnvTnjrW/9CIPDsFfzZoateIWFljU=; b=URb6wpQ6TGfTrPC+NZ5pDGznm9VF+lK/YI1ikc3FhgebDeYpofjuxS62 BiC1SeCpy9haxCIko0L5G07s0RQUjqYYBxzrCaZuc41p+023GTks3Nmwj qEEWIJoMSzg12SKR04gsaQJ77z2efLc3DFUVQ8lwOCHTXhMUvRXBwndff B7E1A1JnbcXWmP8+1wbYRYJEectwzEB6u24NMY4JT7JYaYa+n1ibNmgP/ T0lbRP6BEjG4wg926i1Hy7ra5samSzDR7jSVfA8KXES8Lw+DNaIWoBfs1 uHL/gc+emYkQlsORNvH5qAQ0uvhDeHaAczRn6rKTRaC/65TDlzVW6lHev A==; X-IronPort-AV: E=McAfee;i="6500,9779,10490"; a="290192242" X-IronPort-AV: E=Sophos;i="5.95,158,1661842800"; d="scan'208";a="290192242" Received: from fmsmga007.fm.intel.com ([10.253.24.52]) by orsmga101.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 04 Oct 2022 10:17:05 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6500,9779,10490"; a="626256634" X-IronPort-AV: E=Sophos;i="5.95,158,1661842800"; d="scan'208";a="626256634" Received: from unknown (HELO icx-npg-scs1-cp1.localdomain) ([10.233.180.245]) by fmsmga007.fm.intel.com with ESMTP; 04 Oct 2022 10:17:04 -0700 From: Nicolas Chautru <nicolas.chautru@intel.com> To: dev@dpdk.org, thomas@monjalon.net, gakhil@marvell.com Cc: maxime.coquelin@redhat.com, trix@redhat.com, mdr@ashroe.eu, bruce.richardson@intel.com, david.marchand@redhat.com, stephen@networkplumber.org, mingshan.zhang@intel.com, hemant.agrawal@nxp.com, Nicolas Chautru <nicolas.chautru@intel.com> Subject: [PATCH v12 0/7] bbdev changes for 22.11 Date: Tue, 4 Oct 2022 10:16:49 -0700 Message-Id: <20221004171656.17967-1-nicolas.chautru@intel.com> X-Mailer: git-send-email 2.37.1 In-Reply-To: <1655491040-183649-6-git-send-email-nicolas.chautru@intel.com> References: <1655491040-183649-6-git-send-email-nicolas.chautru@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit 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 |
Series |
bbdev changes for 22.11
|
|
Message
Chautru, Nicolas
Oct. 4, 2022, 5:16 p.m. UTC
Hi Akhil, Thomas, v12: minor change to fix misaligned comment on patch 6 raised by Thomas. Thanks. v11: updated based on Thomas review notably on comments through the serie and ordering. Thanks. I have also updated rel_notes and deprecation through the serie this time. v10: replacing the _PADDED_MAX enum to _SIZE_MAX macro based on suggestion from Ferruh/Maxime/Akhil. Thanks v9: removing code snippet from documentation in 5/7 requested by Akhil. Thanks. v8: edit based on review by Akhil : typos, coding guidelines. No functional change. Thanks v7: couple of typos in documentation spotted by Maxime. Thanks. v6: added one comment in commit 2/7 suggested by Maxime. v5: update base on review from Tom Rix. Number of typos reported and resolved, removed the commit related to rw_lock for now, added a commit for code clean up from review, resolved one rebase issue between 2 commits, used size of array for some bound check implementation. Thanks. v4: update to the last 2 commits to include function to print the queue status and a fix to the rte_lock within the wrong structure v3: update to device status info to also use padded size for the related array. Adding also 2 additionals commits to allow the API struc to expose more information related to queues corner cases/warning as well as an optional rw lock. Hemant, Maxime, this is planned for DPDK 21.11 but would like review/ack early is possible to get this applied earlier and due to time off this summer. Thanks Nic Nicolas Chautru (7): bbdev: allow operation type enum for growth bbdev: add device status info bbdev: add device info on queue topology drivers/baseband: update PMDs to expose queue per operation bbdev: add new operation for FFT processing bbdev: add queue related warning and status information bbdev: remove unnecessary if-check app/test-bbdev/test_bbdev.c | 2 +- app/test-bbdev/test_bbdev_perf.c | 6 +- doc/guides/prog_guide/bbdev.rst | 103 +++++++++++ doc/guides/rel_notes/deprecation.rst | 13 -- doc/guides/rel_notes/release_22_11.rst | 14 ++ drivers/baseband/acc100/rte_acc100_pmd.c | 30 ++-- .../fpga_5gnr_fec/rte_fpga_5gnr_fec.c | 9 + drivers/baseband/fpga_lte_fec/fpga_lte_fec.c | 9 + drivers/baseband/la12xx/bbdev_la12xx.c | 10 +- drivers/baseband/null/bbdev_null.c | 1 + .../baseband/turbo_sw/bbdev_turbo_software.c | 13 ++ examples/bbdev_app/main.c | 2 +- lib/bbdev/rte_bbdev.c | 57 +++++- lib/bbdev/rte_bbdev.h | 158 +++++++++++++++- lib/bbdev/rte_bbdev_op.h | 169 ++++++++++++++++-- lib/bbdev/version.map | 12 ++ 16 files changed, 560 insertions(+), 48 deletions(-)
Comments
> Hi Akhil, Thomas, > > v12: minor change to fix misaligned comment on patch 6 raised by Thomas. > Thanks. > v11: updated based on Thomas review notably on comments through the serie > and ordering. Thanks. I have also updated rel_notes and deprecation through > the serie this time. > v10: replacing the _PADDED_MAX enum to _SIZE_MAX macro based on > suggestion from Ferruh/Maxime/Akhil. Thanks > v9: removing code snippet from documentation in 5/7 requested by Akhil. > Thanks. > v8: edit based on review by Akhil : typos, coding guidelines. No functional > change. Thanks > v7: couple of typos in documentation spotted by Maxime. Thanks. > v6: added one comment in commit 2/7 suggested by Maxime. > v5: update base on review from Tom Rix. Number of typos reported and > resolved, > removed the commit related to rw_lock for now, added a commit for > code clean up from review, resolved one rebase issue between 2 commits, used > size of array for some bound check implementation. Thanks. > v4: update to the last 2 commits to include function to print the queue status > and a fix to the rte_lock within the wrong structure > v3: update to device status info to also use padded size for the related array. > Adding also 2 additionals commits to allow the API struc to expose more > information related to queues corner cases/warning as well as an optional rw > lock. > Hemant, Maxime, this is planned for DPDK 21.11 but would like review/ack early > is possible to get this applied earlier and due to time off this summer. > Thanks > Nic > > Nicolas Chautru (7): > bbdev: allow operation type enum for growth > bbdev: add device status info > bbdev: add device info on queue topology > drivers/baseband: update PMDs to expose queue per operation > bbdev: add new operation for FFT processing > bbdev: add queue related warning and status information > bbdev: remove unnecessary if-check > > app/test-bbdev/test_bbdev.c | 2 +- > app/test-bbdev/test_bbdev_perf.c | 6 +- > doc/guides/prog_guide/bbdev.rst | 103 +++++++++++ > doc/guides/rel_notes/deprecation.rst | 13 -- > doc/guides/rel_notes/release_22_11.rst | 14 ++ > drivers/baseband/acc100/rte_acc100_pmd.c | 30 ++-- > .../fpga_5gnr_fec/rte_fpga_5gnr_fec.c | 9 + > drivers/baseband/fpga_lte_fec/fpga_lte_fec.c | 9 + > drivers/baseband/la12xx/bbdev_la12xx.c | 10 +- > drivers/baseband/null/bbdev_null.c | 1 + > .../baseband/turbo_sw/bbdev_turbo_software.c | 13 ++ > examples/bbdev_app/main.c | 2 +- > lib/bbdev/rte_bbdev.c | 57 +++++- > lib/bbdev/rte_bbdev.h | 158 +++++++++++++++- > lib/bbdev/rte_bbdev_op.h | 169 ++++++++++++++++-- > lib/bbdev/version.map | 12 ++ > 16 files changed, 560 insertions(+), 48 deletions(-) > Hi Nicolas, There were many formatting issues in this patch. This has been a practice in whole of bbdev. Please take a diff of what is applied on the tree and what was submitted to take care of the formatting in future patches and please plan to fix the rest of bbdev documentation (API + prog_guide). Series applied to dpdk-next-crypto Thanks.
Thanks Akhil, A couple of things I miss in term of guidelines for my benefit and that I don't see in documentation: - May I ask what rule we should use for documentation line breaking? I am unclear of the reason for some of the changes you made whose origin version looked legit to me. Were you trying to fit into 80 chars (not always the case) or was this to split the sentence by phrase as opposed to breaking simply just before 100 chars? Can you clarify the recommendation? I see in the doc it suggests 80 chars but unsure whether this is because this was not updated when moving to 100 chars. - Do we really need a dot at the end of every text in a comment even when not an actual sentence? I don't personally see the point nor do I believe this is followed in much of DPDK existing code (it depends at best), nor was this case in the file where new structure was added (keeping consistency). Is that really a recommendation for DPDK guideline for all new code to add a dot at the end of all text? Thanks for kindly clarifying, I can then update the other serie based on such guidelines. Thanks for the other formatting issues and commit message changes. Nic > -----Original Message----- > From: Akhil Goyal <gakhil@marvell.com> > Sent: Thursday, October 6, 2022 10:32 AM > To: Chautru, Nicolas <nicolas.chautru@intel.com>; dev@dpdk.org; > thomas@monjalon.net > Cc: maxime.coquelin@redhat.com; trix@redhat.com; mdr@ashroe.eu; > Richardson, Bruce <bruce.richardson@intel.com>; > david.marchand@redhat.com; stephen@networkplumber.org; Zhang, > Mingshan <mingshan.zhang@intel.com>; hemant.agrawal@nxp.com > Subject: RE: [EXT] [PATCH v12 0/7] bbdev changes for 22.11 > > > Hi Akhil, Thomas, > > > > v12: minor change to fix misaligned comment on patch 6 raised by Thomas. > > Thanks. > > v11: updated based on Thomas review notably on comments through the > > serie and ordering. Thanks. I have also updated rel_notes and > > deprecation through the serie this time. > > v10: replacing the _PADDED_MAX enum to _SIZE_MAX macro based on > > suggestion from Ferruh/Maxime/Akhil. Thanks > > v9: removing code snippet from documentation in 5/7 requested by Akhil. > > Thanks. > > v8: edit based on review by Akhil : typos, coding guidelines. No > > functional change. Thanks > > v7: couple of typos in documentation spotted by Maxime. Thanks. > > v6: added one comment in commit 2/7 suggested by Maxime. > > v5: update base on review from Tom Rix. Number of typos reported and > > resolved, removed the commit related to rw_lock for now, added a > > commit for code clean up from review, resolved one rebase issue > > between 2 commits, used size of array for some bound check > > implementation. Thanks. > > v4: update to the last 2 commits to include function to print the > > queue status and a fix to the rte_lock within the wrong structure > > v3: update to device status info to also use padded size for the related > array. > > Adding also 2 additionals commits to allow the API struc to expose > > more information related to queues corner cases/warning as well as an > > optional rw lock. > > Hemant, Maxime, this is planned for DPDK 21.11 but would like > > review/ack early is possible to get this applied earlier and due to time off > this summer. > > Thanks > > Nic > > > > Nicolas Chautru (7): > > bbdev: allow operation type enum for growth > > bbdev: add device status info > > bbdev: add device info on queue topology > > drivers/baseband: update PMDs to expose queue per operation > > bbdev: add new operation for FFT processing > > bbdev: add queue related warning and status information > > bbdev: remove unnecessary if-check > > > > app/test-bbdev/test_bbdev.c | 2 +- > > app/test-bbdev/test_bbdev_perf.c | 6 +- > > doc/guides/prog_guide/bbdev.rst | 103 +++++++++++ > > doc/guides/rel_notes/deprecation.rst | 13 -- > > doc/guides/rel_notes/release_22_11.rst | 14 ++ > > drivers/baseband/acc100/rte_acc100_pmd.c | 30 ++-- > > .../fpga_5gnr_fec/rte_fpga_5gnr_fec.c | 9 + > > drivers/baseband/fpga_lte_fec/fpga_lte_fec.c | 9 + > > drivers/baseband/la12xx/bbdev_la12xx.c | 10 +- > > drivers/baseband/null/bbdev_null.c | 1 + > > .../baseband/turbo_sw/bbdev_turbo_software.c | 13 ++ > > examples/bbdev_app/main.c | 2 +- > > lib/bbdev/rte_bbdev.c | 57 +++++- > > lib/bbdev/rte_bbdev.h | 158 +++++++++++++++- > > lib/bbdev/rte_bbdev_op.h | 169 ++++++++++++++++-- > > lib/bbdev/version.map | 12 ++ > > 16 files changed, 560 insertions(+), 48 deletions(-) > > > Hi Nicolas, > > There were many formatting issues in this patch. This has been a practice in > whole of bbdev. > Please take a diff of what is applied on the tree and what was submitted to > take care of the formatting in future patches and please plan to fix the rest > of bbdev documentation (API + prog_guide). > > Series applied to dpdk-next-crypto > > Thanks.
> Thanks Akhil, > > A couple of things I miss in term of guidelines for my benefit and that I don't see > in documentation: > - May I ask what rule we should use for documentation line breaking? I am > unclear of the reason for some of the changes you made whose origin version > looked legit to me. Were you trying to fit into 80 chars (not always the case) or > was this to split the sentence by phrase as opposed to breaking simply just > before 100 chars? Can you clarify the recommendation? I see in the doc it > suggests 80 chars but unsure whether this is because this was not updated when > moving to 100 chars. The guideline for is for 80 characters, but if the sentence is going to be split across two Lines, we should break at logical points. This improves readability. > - Do we really need a dot at the end of every text in a comment even when not > an actual sentence? I don't personally see the point nor do I believe this is > followed in much of DPDK existing code (it depends at best), nor was this case in > the file where new structure was added (keeping consistency). Is that really a > recommendation for DPDK guideline for all new code to add a dot at the end of > all text? Dots should be added for each of the comment wherever we have a sentence. They may be skipped for a couple of words. > > Thanks for kindly clarifying, I can then update the other serie based on such > guidelines. > > Thanks for the other formatting issues and commit message changes. > > Nic > > > -----Original Message----- > > From: Akhil Goyal <gakhil@marvell.com> > > Sent: Thursday, October 6, 2022 10:32 AM > > To: Chautru, Nicolas <nicolas.chautru@intel.com>; dev@dpdk.org; > > thomas@monjalon.net > > Cc: maxime.coquelin@redhat.com; trix@redhat.com; mdr@ashroe.eu; > > Richardson, Bruce <bruce.richardson@intel.com>; > > david.marchand@redhat.com; stephen@networkplumber.org; Zhang, > > Mingshan <mingshan.zhang@intel.com>; hemant.agrawal@nxp.com > > Subject: RE: [EXT] [PATCH v12 0/7] bbdev changes for 22.11 > > > > > Hi Akhil, Thomas, > > > > > > v12: minor change to fix misaligned comment on patch 6 raised by Thomas. > > > Thanks. > > > v11: updated based on Thomas review notably on comments through the > > > serie and ordering. Thanks. I have also updated rel_notes and > > > deprecation through the serie this time. > > > v10: replacing the _PADDED_MAX enum to _SIZE_MAX macro based on > > > suggestion from Ferruh/Maxime/Akhil. Thanks > > > v9: removing code snippet from documentation in 5/7 requested by Akhil. > > > Thanks. > > > v8: edit based on review by Akhil : typos, coding guidelines. No > > > functional change. Thanks > > > v7: couple of typos in documentation spotted by Maxime. Thanks. > > > v6: added one comment in commit 2/7 suggested by Maxime. > > > v5: update base on review from Tom Rix. Number of typos reported and > > > resolved, removed the commit related to rw_lock for now, added a > > > commit for code clean up from review, resolved one rebase issue > > > between 2 commits, used size of array for some bound check > > > implementation. Thanks. > > > v4: update to the last 2 commits to include function to print the > > > queue status and a fix to the rte_lock within the wrong structure > > > v3: update to device status info to also use padded size for the related > > array. > > > Adding also 2 additionals commits to allow the API struc to expose > > > more information related to queues corner cases/warning as well as an > > > optional rw lock. > > > Hemant, Maxime, this is planned for DPDK 21.11 but would like > > > review/ack early is possible to get this applied earlier and due to time off > > this summer. > > > Thanks > > > Nic > > > > > > Nicolas Chautru (7): > > > bbdev: allow operation type enum for growth > > > bbdev: add device status info > > > bbdev: add device info on queue topology > > > drivers/baseband: update PMDs to expose queue per operation > > > bbdev: add new operation for FFT processing > > > bbdev: add queue related warning and status information > > > bbdev: remove unnecessary if-check > > > > > > app/test-bbdev/test_bbdev.c | 2 +- > > > app/test-bbdev/test_bbdev_perf.c | 6 +- > > > doc/guides/prog_guide/bbdev.rst | 103 +++++++++++ > > > doc/guides/rel_notes/deprecation.rst | 13 -- > > > doc/guides/rel_notes/release_22_11.rst | 14 ++ > > > drivers/baseband/acc100/rte_acc100_pmd.c | 30 ++-- > > > .../fpga_5gnr_fec/rte_fpga_5gnr_fec.c | 9 + > > > drivers/baseband/fpga_lte_fec/fpga_lte_fec.c | 9 + > > > drivers/baseband/la12xx/bbdev_la12xx.c | 10 +- > > > drivers/baseband/null/bbdev_null.c | 1 + > > > .../baseband/turbo_sw/bbdev_turbo_software.c | 13 ++ > > > examples/bbdev_app/main.c | 2 +- > > > lib/bbdev/rte_bbdev.c | 57 +++++- > > > lib/bbdev/rte_bbdev.h | 158 +++++++++++++++- > > > lib/bbdev/rte_bbdev_op.h | 169 ++++++++++++++++-- > > > lib/bbdev/version.map | 12 ++ > > > 16 files changed, 560 insertions(+), 48 deletions(-) > > > > > Hi Nicolas, > > > > There were many formatting issues in this patch. This has been a practice in > > whole of bbdev. > > Please take a diff of what is applied on the tree and what was submitted to > > take care of the formatting in future patches and please plan to fix the rest > > of bbdev documentation (API + prog_guide). > > > > Series applied to dpdk-next-crypto > > > > Thanks.
06/10/2022 19:31, Akhil Goyal: > There were many formatting issues in this patch. This has been a practice in whole of bbdev. > Please take a diff of what is applied on the tree and what was submitted to take care of the > formatting in future patches and please plan to fix the rest of bbdev documentation (API + prog_guide). > > Series applied to dpdk-next-crypto Please could you check this build issue? https://bugs.dpdk.org/show_bug.cgi?id=1095 Ideally I'd like to see it fixed today, thanks.
Hi Thomas, Akhil, I put a fix here : https://patches.dpdk.org/project/dpdk/patch/20221010170403.21201-1-nicolas.chautru@intel.com/ I did not reproduce that issue on gcc or recent clang, only with clang 3.4.2 on Centos7. Thanks Nic > -----Original Message----- > From: Thomas Monjalon <thomas@monjalon.net> > Sent: Monday, October 10, 2022 12:36 AM > To: Chautru, Nicolas <nicolas.chautru@intel.com> > Cc: dev@dpdk.org; maxime.coquelin@redhat.com; trix@redhat.com; > mdr@ashroe.eu; Richardson, Bruce <bruce.richardson@intel.com>; > david.marchand@redhat.com; stephen@networkplumber.org; Zhang, > Mingshan <mingshan.zhang@intel.com>; hemant.agrawal@nxp.com; Akhil > Goyal <gakhil@marvell.com>; alialnu@nvidia.com > Subject: Re: [EXT] [PATCH v12 0/7] bbdev changes for 22.11 > > 06/10/2022 19:31, Akhil Goyal: > > There were many formatting issues in this patch. This has been a practice in > whole of bbdev. > > Please take a diff of what is applied on the tree and what was > > submitted to take care of the formatting in future patches and please plan > to fix the rest of bbdev documentation (API + prog_guide). > > > > Series applied to dpdk-next-crypto > > Please could you check this build issue? > https://bugs.dpdk.org/show_bug.cgi?id=1095 > > Ideally I'd like to see it fixed today, thanks. >