[PATCH v1] eventdev/crypto: fix enqueueing invalid ops

[Jerin Jacob]

On Wed, Feb 28, 2024 at 10:42 PM Gujjar, Abhinandan S
<abhinandan.gujjar at intel.com> wrote:
>
>
>
> > -----Original Message-----
> > From: Kundapura, Ganapati <ganapati.kundapura at intel.com>
> > Sent: Wednesday, February 28, 2024 4:09 PM
> > To: dev at dpdk.org; jerinj at marvell.com; Jayatheerthan, Jay
> > <jay.jayatheerthan at intel.com>
> > Cc: Naga Harish K, S V <s.v.naga.harish.k at intel.com>; Gujjar, Abhinandan S
> > <abhinandan.gujjar at intel.com>
> > Subject: [PATCH v1] eventdev/crypto: fix enqueueing invalid ops
> >
> > When tail pointer of Circ buffer rollsover as the Circ buffer becomes full, crypto
> > adapter is enqueueing ops beyond the size of the Circ buffer leading to segfault
> > due to invalid ops access.
> >
> > Fixed by enqueueing ops from head pointer to (size-head) number of ops when
> > Circ buffer becomes full and the remaining ops will be flushed in next iteration.
> >
> > Fixes: 6c3c888656fc ("eventdev/crypto: fix circular buffer full case")
> >
> > Signed-off-by: Ganapati Kundapura <ganapati.kundapura at intel.com>
> >
> Acked-by: Abhinandan Gujjar <abhinandan.gujjar at intel.com>


Updated the git commit as follows and applied to
dpdk-next-net-eventdev/for-main. Thanks

    eventdev/crypto: fix enqueueing invalid ops

    When tail pointer of circular buffer rolls over as the circular buffer
    becomes full, crypto adapter is enqueueing ops beyond the size of the
    circular buffer leading to segfault due to invalid ops access.

    Fixed by enqueueing ops from head pointer to (size-head) number of ops
    when circular buffer becomes full and the remaining ops will be flushed
    in next iteration.

    Fixes: 6c3c888656fc ("eventdev/crypto: fix circular buffer full case")
    Cc: stable at dpdk.org

    Signed-off-by: Ganapati Kundapura <ganapati.kundapura at intel.com>
    Acked-by: Abhinandan Gujjar <abhinandan.gujjar at intel.com>