Received: by 2002:a05:6602:2086:0:0:0:0 with SMTP id a6csp4670956ioa; Wed, 27 Apr 2022 08:37:13 -0700 (PDT) X-Google-Smtp-Source: ABdhPJw3J3pz85MyS2lG5qwwL24ykyqjw1A+xOeKbOM4NT9OPhdOKkIEGUllgND5rCj0kzA62pkI X-Received: by 2002:a17:90b:1b4f:b0:1d2:a09e:1fc6 with SMTP id nv15-20020a17090b1b4f00b001d2a09e1fc6mr44072006pjb.147.1651073833121; Wed, 27 Apr 2022 08:37:13 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1651073833; cv=none; d=google.com; s=arc-20160816; b=YPQHijRuAPgzIHCht7wlbxdPM0CHHcw7kvLMHXyDIYAiuJxzE/0pKE731jxMuQSpJ1 PNEAxjo0XLq5CUyLSMPY3mzTE/fTDCnwEhesoVw8inu33kc+d72cXfjtyp0tHQYYCOp7 Q39YJ8+eSVLHpFR33A5ZCcDCIsZrzGArV3gUxeY6X0OsCn74+XF/igAwlnSVCC/KykdV uFPnoz832TAYip/XP5VRkmgqGCTvOO/AFTmlTq9u3OiXHjMWj1MR5uHUcJDfjlbCd+wL T02ZhhseTJPjdSCHa0r60FduDNmq0SdLlz/FDI1q/btQmSweCtHG6pZPi7iIypnenvaL DyBA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:mime-version:user-agent :content-transfer-encoding:references:in-reply-to:date:cc:to:from :subject:message-id:dkim-signature; bh=BGXgRkm7KfAuxs/lpObycbS7QSUjU1DegZpjJv5xMRo=; b=leWyedrYgduGOEK24bF1P7QoEfLpaCjyHl2H3Um8QHqNpdPzpPvfYfvrJVWCfV5OeP uCMmVhIqIXien52GEI+iT6HjeB2Fo4wFx9rCUUrcsIuMZZuzrsj9weZLrwkTqeqOf+rB fINeEZ6J4RQb8Okv0b5IDdoztLzaG/mPa6qwbCWrnWPXPC9bOMrx+uSX7kWEob7hR9DV AZfuJYxF8TN0QlNjK5if/SkwUIwHPjCOKCmfgnFoaBEhR8d+noz28WI4rh3KGIzrfifh +hwH80ge7Ht6d/WoyrJuqbIwXyf0vLnGbWk7ar69WHNmuLs8uP8DLskAsjjrp5V+N7GN eUCQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@ndufresne-ca.20210112.gappssmtp.com header.s=20210112 header.b="zQCZg/1g"; spf=softfail (google.com: domain of transitioning linux-kernel-owner@vger.kernel.org does not designate 23.128.96.19 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from lindbergh.monkeyblade.net (lindbergh.monkeyblade.net. [23.128.96.19]) by mx.google.com with ESMTPS id m7-20020a17090a730700b001d950ab2cfdsi1910665pjk.75.2022.04.27.08.37.12 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 27 Apr 2022 08:37:13 -0700 (PDT) Received-SPF: softfail (google.com: domain of transitioning linux-kernel-owner@vger.kernel.org does not designate 23.128.96.19 as permitted sender) client-ip=23.128.96.19; Authentication-Results: mx.google.com; dkim=pass header.i=@ndufresne-ca.20210112.gappssmtp.com header.s=20210112 header.b="zQCZg/1g"; spf=softfail (google.com: domain of transitioning linux-kernel-owner@vger.kernel.org does not designate 23.128.96.19 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id D906A2D028E; Wed, 27 Apr 2022 08:09:40 -0700 (PDT) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S239480AbiD0PMg (ORCPT + 99 others); Wed, 27 Apr 2022 11:12:36 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:44698 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S239216AbiD0PLo (ORCPT ); Wed, 27 Apr 2022 11:11:44 -0400 Received: from mail-qk1-x736.google.com (mail-qk1-x736.google.com [IPv6:2607:f8b0:4864:20::736]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 38D0A2010D4 for ; Wed, 27 Apr 2022 08:08:29 -0700 (PDT) Received: by mail-qk1-x736.google.com with SMTP id f186so1474625qke.8 for ; Wed, 27 Apr 2022 08:08:29 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ndufresne-ca.20210112.gappssmtp.com; s=20210112; h=message-id:subject:from:to:cc:date:in-reply-to:references :content-transfer-encoding:user-agent:mime-version; bh=BGXgRkm7KfAuxs/lpObycbS7QSUjU1DegZpjJv5xMRo=; b=zQCZg/1gEuydchdqj6KgGuunQzzEztTpMwRrPhBiadpx9coH4AxqBO1HpODzdIHeVh ZO7Svx7GUbRvl93GxQFyUkRwfSjo3I7TZ0RKbbUhR2DlLz5iEm9FkTkeOxnyG7+czP9x vfrl9y0VK8vlKPZZi3FNtbDlGkwqinateCU8uXhRG0XNkl8wd6IH+vk4ZnehGkOWCctK gpnG2qvACm3Tz5lSWnpg7gmhlIxxMQ1Ebc4FS/xxc0kRi/vVUGugtDxGL7eGZVkznZJI WElUfnl3w5FOkimxquM6ZIQj8xIKQKcxxlSCJu0QL5CDLDxt+xZvfqGivWjfS/5v9rNJ gYXg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:message-id:subject:from:to:cc:date:in-reply-to :references:content-transfer-encoding:user-agent:mime-version; bh=BGXgRkm7KfAuxs/lpObycbS7QSUjU1DegZpjJv5xMRo=; b=11IhNNceNFimQDp2pNQpSgg8h6G5LDSCC0hQRhjHB3ttuSQVNQwC56wyOUYt4BP34P n6oVY898KGVZihvgAb5IiWhDS2HJ6fuVxXgBtPPd5sLemO0aUlrutKbmNvF1mWNeFJ8h n5AlUPVxEP5GvKqKbaSKS0R5fpOZRbpYcbn1oAOfg1sXDj4l27YDl1tOnKqqHs9T536J p0MDkVYP1v5F2rJg73K0TrgosNcjjnsZwdu2pAPMf8laJDg/mKPnb6+THeKKg3+LVj7P 9QqCUjZajZ3ky7n7KcKh+1QizF2Be8FmSnF1dxIpsv6rL3NGz6C87+GbBSceBuP8qsQU ZkGQ== X-Gm-Message-State: AOAM530l+thCueFYFTRQGydkHN1qOXJZZVxVmgjV0aTRUmDfVKvntxDz 3oio8Ig74lpwnGl+7IObxQ7coq7xCD0xxA== X-Received: by 2002:a05:620a:4450:b0:69e:a2ca:e7d7 with SMTP id w16-20020a05620a445000b0069ea2cae7d7mr16860296qkp.178.1651072107898; Wed, 27 Apr 2022 08:08:27 -0700 (PDT) Received: from nicolas-tpx395.localdomain (173-246-12-168.qc.cable.ebox.net. [173.246.12.168]) by smtp.gmail.com with ESMTPSA id k20-20020a05622a03d400b002ec16d2694fsm10227421qtx.39.2022.04.27.08.08.26 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 27 Apr 2022 08:08:27 -0700 (PDT) Message-ID: Subject: Re: [PATCH v4 03/24] media: videobuf2-v4l2: Warn on holding buffers without support From: Nicolas Dufresne To: Tomasz Figa , Marek Szyprowski , Mauro Carvalho Chehab , Hans Verkuil Cc: Sebastian Fricke , linux-media@vger.kernel.org, Ezequiel Garcia , linux-kernel@vger.kernel.org Date: Wed, 27 Apr 2022 11:08:24 -0400 In-Reply-To: References: <20220426125751.108293-1-nicolas.dufresne@collabora.com> <20220426125751.108293-4-nicolas.dufresne@collabora.com> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable User-Agent: Evolution 3.44.0 (3.44.0-1.fc36) MIME-Version: 1.0 X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,RDNS_NONE, SPF_HELO_NONE autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Le mercredi 27 avril 2022 =C3=A0 13:31 +0900, Tomasz Figa a =C3=A9crit=C2= =A0: > Hi Nicolas, Sebastian, >=20 > On Tue, Apr 26, 2022 at 9:58 PM Nicolas Dufresne > wrote: > >=20 > > From: Sebastian Fricke > >=20 > > Using V4L2_BUF_FLAG_M2M_HOLD_CAPTURE_BUF flag without specifying the > > subsystem flag VB2_V4L2_FL_SUPPORTS_M2M_HOLD_CAPTURE_BUF, results in > > silently ignoring it. > > Warn the user via a debug print when the flag is requested but ignored > > by the videobuf2 framework. > >=20 > > Signed-off-by: Sebastian Fricke > > Signed-off-by: Nicolas Dufresne > > Reviewed-by: Ezequiel Garcia > > --- > > drivers/media/common/videobuf2/videobuf2-v4l2.c | 7 ++++++- > > 1 file changed, 6 insertions(+), 1 deletion(-) > >=20 >=20 > Thanks for the patch. Please see my comments inline. >=20 > > diff --git a/drivers/media/common/videobuf2/videobuf2-v4l2.c b/drivers/= media/common/videobuf2/videobuf2-v4l2.c > > index 6edf4508c636..812c8d1962e0 100644 > > --- a/drivers/media/common/videobuf2/videobuf2-v4l2.c > > +++ b/drivers/media/common/videobuf2/videobuf2-v4l2.c > > @@ -329,8 +329,13 @@ static int vb2_fill_vb2_v4l2_buffer(struct vb2_buf= fer *vb, struct v4l2_buffer *b > > */ > > vbuf->flags &=3D ~V4L2_BUF_FLAG_TIMECODE; > > vbuf->field =3D b->field; > > - if (!(q->subsystem_flags & VB2_V4L2_FL_SUPPORTS_M2M_HOL= D_CAPTURE_BUF)) > > + if (!(q->subsystem_flags & VB2_V4L2_FL_SUPPORTS_M2M_HOL= D_CAPTURE_BUF)) { > > + if (vbuf->flags & V4L2_BUF_FLAG_M2M_HOLD_CAPTUR= E_BUF) > > + dprintk(q, 1, > > + "Request holding buffer (%d), u= nsupported on output queue\n", > > + b->index); >=20 > I wonder if we shouldn't just fail such a QBUF operation. Otherwise > the application would get unexpected behavior from the kernel. > Although it might be too late to do it now if there are applications > that rely on this implicit ignore... In the context of this patchset, the statu quo seems to be the logical thin= g to do. We can raise this up in a separate thread. The side effect is of course confusing for developers, but it is hard for me to tell if a hard failure m= ay break an existing software. regards, Nicolas >=20 > Best regards, > Tomasz