mbox series

[v1,0/4] fbarray lookahead/lookbehind fixes

Message ID cover.1720454625.git.anatoly.burakov@intel.com (mailing list archive)
Headers
Series fbarray lookahead/lookbehind fixes |

Message

Burakov, Anatoly July 8, 2024, 4:07 p.m. UTC 
Once upon a time, a few patches were submitted by Vipin P R:

https://patches.dpdk.org/project/dpdk/patch/1673615669-21011-2-git-send-email-vipinp@vmware.com/
https://patches.dpdk.org/project/dpdk/patch/1673615669-21011-3-git-send-email-vipinp@vmware.com/
https://patches.dpdk.org/project/dpdk/patch/1673615669-21011-2-git-send-email-vipinp@vmware.com/
https://patches.dpdk.org/project/dpdk/patch/1673615669-21011-3-git-send-email-vipinp@vmware.com/

They were reviewed and changes were requested, but the author never followed up
and these patches kind of fell through the cracks. The patches fixed real bugs
in fbarray lookahead/lookbehind behavior, so now these bugs have resurfaced in
some customer reports.

This is a resubmit with improvements and added unit tests.

Anatoly Burakov (4):
  fbarray: fix incorrect lookahead behavior
  fbarray: fix incorrect lookbehind behavior
  fbarray: fix lookahead ignore mask handling
  fbarray: fix lookbehind ignore mask handling

 app/test/test_fbarray.c             | 102 ++++++++++++++++++++++++++++
 lib/eal/common/eal_common_fbarray.c |  28 ++++++--
 2 files changed, 123 insertions(+), 7 deletions(-)

Comments

David Marchand July 9, 2024, 4:57 a.m. UTC | #1 
On Mon, Jul 8, 2024 at 6:07 PM Anatoly Burakov
<anatoly.burakov@intel.com> wrote:
>
> Once upon a time, a few patches were submitted by Vipin P R:
>
> https://patches.dpdk.org/project/dpdk/patch/1673615669-21011-2-git-send-email-vipinp@vmware.com/
> https://patches.dpdk.org/project/dpdk/patch/1673615669-21011-3-git-send-email-vipinp@vmware.com/
> https://patches.dpdk.org/project/dpdk/patch/1673615669-21011-2-git-send-email-vipinp@vmware.com/
> https://patches.dpdk.org/project/dpdk/patch/1673615669-21011-3-git-send-email-vipinp@vmware.com/
>
> They were reviewed and changes were requested, but the author never followed up
> and these patches kind of fell through the cracks. The patches fixed real bugs
> in fbarray lookahead/lookbehind behavior, so now these bugs have resurfaced in
> some customer reports.
>
> This is a resubmit with improvements and added unit tests.
>
> Anatoly Burakov (4):
>   fbarray: fix incorrect lookahead behavior
>   fbarray: fix incorrect lookbehind behavior
>   fbarray: fix lookahead ignore mask handling
>   fbarray: fix lookbehind ignore mask handling
>
>  app/test/test_fbarray.c             | 102 ++++++++++++++++++++++++++++
>  lib/eal/common/eal_common_fbarray.c |  28 ++++++--
>  2 files changed, 123 insertions(+), 7 deletions(-)

Thank you, series applied.