linux-kernel - [PATCH] Allow S/G_PARM for stateful decoder

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-Id: <1661759670-19902-1-git-send-email-quic_dikshita@quicinc.com>
Date:   Mon, 29 Aug 2022 13:24:30 +0530
From:   Dikshita Agarwal <quic_dikshita@...cinc.com>
To:     linux-media@...r.kernel.org, linux-kernel@...r.kernel.org,
        hverkuil-cisco@...all.nl
Cc:     stanimir.varbanov@...aro.org, quic_vgarodia@...cinc.com,
        quic_majja@...cinc.com, quic_jdas@...cinc.com,
        Dikshita Agarwal <quic_dikshita@...cinc.com>
Subject: [PATCH] Allow S/G_PARM for stateful decoder

Some stateful decoder supports S/G_PARM similar to a
stateful encoder. S_PARM(OUTPUT) reserves hardware decoder
resources, and G_PARM(CAPTURE) returns the embedded
frame interval. Allow the same with this change.

Signed-off-by: Dikshita Agarwal <quic_dikshita@...cinc.com>
---
 utils/v4l2-compliance/v4l2-test-formats.cpp | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/utils/v4l2-compliance/v4l2-test-formats.cpp b/utils/v4l2-compliance/v4l2-test-formats.cpp
index 269a383..e996558 100644
--- a/utils/v4l2-compliance/v4l2-test-formats.cpp
+++ b/utils/v4l2-compliance/v4l2-test-formats.cpp
@@ -64,8 +64,6 @@ static int testEnumFrameIntervals(struct node *node, __u32 pixfmt,
 		ret = doioctl(node, VIDIOC_ENUM_FRAMEINTERVALS, &frmival);
 		if (ret == ENOTTY)
 			return ret;
-		// M2M devices don't support this, except for stateful encoders
-		fail_on_test(node->is_m2m && !(node->codec_mask & STATEFUL_ENCODER));
 		if (f == 0 && ret == EINVAL) {
 			if (type == V4L2_FRMSIZE_TYPE_DISCRETE)
 				warn("found framesize %dx%d, but no frame intervals\n", w, h);
@@ -1367,8 +1365,6 @@ static int testParmType(struct node *node, unsigned type)
 	}
 	if (ret == ENOTTY)
 		return ret;
-	// M2M devices don't support this, except for stateful encoders
-	fail_on_test(node->is_m2m && !is_stateful_enc);
 	if (ret == EINVAL)
 		return ENOTTY;
 	if (ret)
-- 
2.7.4