lkml.org 
[lkml]   [2022]   [Jun]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.18 103/879] media: venus: do not queue internal buffers from previous sequence
    Date
    From: Vikash Garodia <quic_vgarodia@quicinc.com>

    [ Upstream commit 73664f107c0fafb59cd91e576b81c986adb74610 ]

    During reconfig (DRC) event from firmware, it is not guaranteed that
    all the DPB(internal) buffers would be released by the firmware. Some
    buffers might be released gradually while processing frames from the
    new sequence. These buffers now stay idle in the dpblist.
    In subsequent call to queue the DPBs to firmware, these idle buffers
    should not be queued. The fix identifies those buffers and free them.

    Signed-off-by: Vikash Garodia <quic_vgarodia@quicinc.com>
    Tested-by: Fritz Koenig <frkoenig@chromium.org>
    Signed-off-by: Stanimir Varbanov <stanimir.varbanov@linaro.org>
    Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/media/platform/qcom/venus/helpers.c | 34 +++++++++++++++------
    1 file changed, 25 insertions(+), 9 deletions(-)

    diff --git a/drivers/media/platform/qcom/venus/helpers.c b/drivers/media/platform/qcom/venus/helpers.c
    index 0bca95d01650..fa01edd54c03 100644
    --- a/drivers/media/platform/qcom/venus/helpers.c
    +++ b/drivers/media/platform/qcom/venus/helpers.c
    @@ -90,12 +90,28 @@ bool venus_helper_check_codec(struct venus_inst *inst, u32 v4l2_pixfmt)
    }
    EXPORT_SYMBOL_GPL(venus_helper_check_codec);

    +static void free_dpb_buf(struct venus_inst *inst, struct intbuf *buf)
    +{
    + ida_free(&inst->dpb_ids, buf->dpb_out_tag);
    +
    + list_del_init(&buf->list);
    + dma_free_attrs(inst->core->dev, buf->size, buf->va, buf->da,
    + buf->attrs);
    + kfree(buf);
    +}
    +
    int venus_helper_queue_dpb_bufs(struct venus_inst *inst)
    {
    - struct intbuf *buf;
    + struct intbuf *buf, *next;
    + unsigned int dpb_size = 0;
    int ret = 0;

    - list_for_each_entry(buf, &inst->dpbbufs, list) {
    + if (inst->dpb_buftype == HFI_BUFFER_OUTPUT)
    + dpb_size = inst->output_buf_size;
    + else if (inst->dpb_buftype == HFI_BUFFER_OUTPUT2)
    + dpb_size = inst->output2_buf_size;
    +
    + list_for_each_entry_safe(buf, next, &inst->dpbbufs, list) {
    struct hfi_frame_data fdata;

    memset(&fdata, 0, sizeof(fdata));
    @@ -106,6 +122,12 @@ int venus_helper_queue_dpb_bufs(struct venus_inst *inst)
    if (buf->owned_by == FIRMWARE)
    continue;

    + /* free buffer from previous sequence which was released later */
    + if (dpb_size > buf->size) {
    + free_dpb_buf(inst, buf);
    + continue;
    + }
    +
    fdata.clnt_data = buf->dpb_out_tag;

    ret = hfi_session_process_buf(inst, &fdata);
    @@ -127,13 +149,7 @@ int venus_helper_free_dpb_bufs(struct venus_inst *inst)
    list_for_each_entry_safe(buf, n, &inst->dpbbufs, list) {
    if (buf->owned_by == FIRMWARE)
    continue;
    -
    - ida_free(&inst->dpb_ids, buf->dpb_out_tag);
    -
    - list_del_init(&buf->list);
    - dma_free_attrs(inst->core->dev, buf->size, buf->va, buf->da,
    - buf->attrs);
    - kfree(buf);
    + free_dpb_buf(inst, buf);
    }

    if (list_empty(&inst->dpbbufs))
    --
    2.35.1


    \
     
     \ /
      Last update: 2022-06-08 00:08    [W:4.095 / U:2.288 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site