]> git.ipfire.org Git - thirdparty/kernel/stable-queue.git/commitdiff
4.4-stable patches
authorGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 19 Mar 2019 14:55:11 +0000 (15:55 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 19 Mar 2019 14:55:11 +0000 (15:55 +0100)
added patches:
media-videobuf2-v4l2-drop-warn_on-in-vb2_warn_zero_bytesused.patch

queue-4.4/media-videobuf2-v4l2-drop-warn_on-in-vb2_warn_zero_bytesused.patch [new file with mode: 0644]
queue-4.4/series

diff --git a/queue-4.4/media-videobuf2-v4l2-drop-warn_on-in-vb2_warn_zero_bytesused.patch b/queue-4.4/media-videobuf2-v4l2-drop-warn_on-in-vb2_warn_zero_bytesused.patch
new file mode 100644 (file)
index 0000000..4d08ec1
--- /dev/null
@@ -0,0 +1,36 @@
+From 5e99456c20f712dcc13d9f6ca4278937d5367355 Mon Sep 17 00:00:00 2001
+From: Hans Verkuil <hverkuil-cisco@xs4all.nl>
+Date: Mon, 19 Nov 2018 10:33:44 -0500
+Subject: media: videobuf2-v4l2: drop WARN_ON in vb2_warn_zero_bytesused()
+
+From: Hans Verkuil <hverkuil-cisco@xs4all.nl>
+
+commit 5e99456c20f712dcc13d9f6ca4278937d5367355 upstream.
+
+Userspace shouldn't set bytesused to 0 for output buffers.
+vb2_warn_zero_bytesused() warns about this (only once!), but it also
+calls WARN_ON(1), which is confusing since it is not immediately clear
+that it warns about a 0 value for bytesused.
+
+Just drop the WARN_ON as it serves no purpose.
+
+Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
+Acked-by: Ezequiel Garcia <ezequiel@collabora.com>
+Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
+Cc: Matthias Maennich <maennich@google.com>
+Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
+
+---
+ drivers/media/v4l2-core/videobuf2-v4l2.c |    1 -
+ 1 file changed, 1 deletion(-)
+
+--- a/drivers/media/v4l2-core/videobuf2-v4l2.c
++++ b/drivers/media/v4l2-core/videobuf2-v4l2.c
+@@ -141,7 +141,6 @@ static void vb2_warn_zero_bytesused(stru
+               return;
+       check_once = true;
+-      WARN_ON(1);
+       pr_warn("use of bytesused == 0 is deprecated and will be removed in the future,\n");
+       if (vb->vb2_queue->allow_zero_bytesused)
index 07ff509d6d9122ea52310319b24bdb5b205931e5..0870e9750c084e5ab864b0613e0d072168a20a0a 100644 (file)
@@ -146,3 +146,4 @@ tcp-dccp-remove-reqsk_put-from-inet_child_forget.patch
 alsa-bebob-use-more-identical-mod_alias-for-saffire-pro-10-i-o-against-liquid-saffire-56.patch
 fs-9p-use-fscache-mutex-rather-than-spinlock.patch
 it-s-wrong-to-add-len-to-sector_nr-in-raid10-reshape-twice.patch
+media-videobuf2-v4l2-drop-warn_on-in-vb2_warn_zero_bytesused.patch