From patchwork Fri Jun 26 17:11:28 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ezequiel Garcia X-Patchwork-Id: 11628097 Return-Path: Received: from mail.kernel.org (pdx-korg-mail-1.web.codeaurora.org [172.30.200.123]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id AC14D13B4 for ; Fri, 26 Jun 2020 17:12:34 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 86535207FC for ; Fri, 26 Jun 2020 17:12:34 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="RFWzfcu9" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 86535207FC Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=collabora.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-rockchip-bounces+patchwork-linux-rockchip=patchwork.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:MIME-Version:Message-Id:Date:Subject:To:From: Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From:Resent-Sender :Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References:List-Owner; bh=h58nO8XWiTrqBm1bywi20p1XzpVX3UIZWXz1SEmcjco=; b=RFWzfcu9VyaGYCP3aVP4JcRd4R y8zdXmg/PmI2CeYmKBAfICJuV6DsOlDHMXuZ+2i23Wvy3K8rPCY30aE64nVD9vswgnF9BVut+PqZf Dxi8sM+rPXtEugD23Pxpj0Fcos9kVA8/LGce7Tfov/yqXQmFMa0F2wOItoMSo3ZCZE5tptaeHo6zx 3n5HHkV/zEMkCJGW+GTulMi7vIhKK1jzMQ+BD+cZaJTzpW3NjxYuNUhrjDOm5xCVCHO6+E+U4Y1Ky C2L6uzYlRZJBZZjQxZiIA9I29diwI2oQl1gYUNYAFk+ARvJJAVeIV09BWULWnpy6FRl1KCbhprWMC H8Raskaw==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1jortq-0007k2-5x; Fri, 26 Jun 2020 17:12:26 +0000 Received: from bhuna.collabora.co.uk ([2a00:1098:0:82:1000:25:2eeb:e3e3]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1jorto-0007jZ-FL for linux-rockchip@lists.infradead.org; Fri, 26 Jun 2020 17:12:25 +0000 Received: from [127.0.0.1] (localhost [127.0.0.1]) (Authenticated sender: ezequiel) with ESMTPSA id 9AC972A5D20 From: Ezequiel Garcia To: linux-media@vger.kernel.org, linux-kernel@vger.kernel.org, linux-rockchip@lists.infradead.org Subject: [PATCH 0/2] media: hantro/rkvdec handle unsupported H.264 bitstreams Date: Fri, 26 Jun 2020 14:11:28 -0300 Message-Id: <20200626171130.27346-1-ezequiel@collabora.com> X-Mailer: git-send-email 2.26.0.rc2 MIME-Version: 1.0 X-Spam-Note: CRM114 invocation failed X-Spam-Score: -0.0 (/) X-Spam-Report: SpamAssassin version 3.4.4 on merlin.infradead.org summary: Content analysis details: (-0.0 points) pts rule name description ---- ---------------------- -------------------------------------------------- -0.0 SPF_HELO_PASS SPF: HELO matches SPF record -0.0 SPF_PASS SPF: sender matches SPF record 0.0 UNPARSEABLE_RELAY Informational: message has unparseable relay lines X-BeenThere: linux-rockchip@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Upstream kernel work for Rockchip platforms List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: kernel@collabora.com, Jonas Karlman , Hans Verkuil , Philipp Zabel , Nicolas Dufresne , Ezequiel Garcia Sender: "Linux-rockchip" Errors-To: linux-rockchip-bounces+patchwork-linux-rockchip=patchwork.kernel.org@lists.infradead.org Hi all, Small patchset to add a check at TRY_EXT_CTRLS time, via the H264 SPS control and reject unsupported bitstreams. Properly refusing to decode unsupported bitstreams allows applications to cleanly fallback to software decoding. Note that Rockchip VDEC hardware is capable of decoding High-10 and High-422 bitstreams. This needs more work, so for now they are refused. The same approach can be use on Cedrus, but since I'm not very familiar there, I'll leave that to others. Applies on top of media master. Ezequiel Garcia (2): rkvdec: h264: Refuse to decode unsupported bitstream hantro: h264: Refuse to decode unsupported bitstream drivers/staging/media/hantro/hantro_drv.c | 29 ++++++++++++++++++++--- drivers/staging/media/rkvdec/rkvdec.c | 27 +++++++++++++++++++++ 2 files changed, 53 insertions(+), 3 deletions(-)