From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id F29FEC433EF for ; Mon, 4 Oct 2021 16:09:28 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (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 C2D0461206 for ; Mon, 4 Oct 2021 16:09:28 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org C2D0461206 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-ID:Date:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=ZXDd4PHlontHBSFqFpZTeJj10kgw64QOSHCqi2rkIps=; b=YpiRvtSEMQkVEt JLkvl6IYFX0/Ihq9kDd4l2Zv8EJGjDM2asZdiDy+XFBQ2QXuIjzJoLJ3D2twMwgsKu15oXxAMFol3 okAqAwF0sH6oMapmKYAWecJL44kCRxbWD8HBhK/o195+WOrCdrYubocvoq+CZOyqJDV4Hd31a/XQF 99bK6CspoM1o2Y1OSUkf7Q4DqpFykeJAmq+tk5j6HMP6xqkHnfl/YQ/HCdBbPsoE6cLp9/v2souVl xpxtqAU+p5OcQXcAhFQdjP382DND2Tv5DBTEDzUzkNRluQNgr3XoX6HcCUHXPpR6avAzyDRci3hbw iytJfmOD7Uehx5Ve0Ovg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mXQUY-0071GK-77; Mon, 04 Oct 2021 16:07:02 +0000 Received: from mail-ed1-x52e.google.com ([2a00:1450:4864:20::52e]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1mXQUE-00719j-Kf for linux-arm-kernel@lists.infradead.org; Mon, 04 Oct 2021 16:06:46 +0000 Received: by mail-ed1-x52e.google.com with SMTP id dj4so67359261edb.5 for ; Mon, 04 Oct 2021 09:06:41 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=GqpwaHcYLWxKjnZgog6w5RyFafP586fBMegQAr62NIY=; b=QHqHkuI29B7ysAsKw3DIQtMpkDe2IVUsf2mPVI1tuSQczP83Om194FDjMbL035HJEp +97UN/SmVU9e4K5PhrDyR2lP+pKHeZUsiQTY/WZNayt46flF8OW7m1+WNRB7kq1scb3C cZBteAEGktBh+Lp9YjVp8KqmGSjKDLxykRAEPL/zr0kTaXfaWr3utY4re7T08wraYNmX mXH4xoYa49n7so495p8mkAEvW1BsxCIdmJEz6SvhneLxyiBt2fRvLhNR0mf2dEWJTgvK Ffjq2BQiN5EhrU6ImaLSxnrqCIUPAQcHEtSVecLdRPxDMLGGnNThFiOsaz7y9nwvTPGl 4RYA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=GqpwaHcYLWxKjnZgog6w5RyFafP586fBMegQAr62NIY=; b=BEcz2Wz1AmQYorLUK0cNSLoY6X/clCOrEHArmtjujVm2xGD3/RLdAUwaToq5y4aZMC fwy8SW62SbS78KaLMUCNmuTi3a5yWHY4/Vg8Znn+AX/S3BIJkRgXXMQQDch8lGlD6aco AEcJyJ9dEyT/LBAoOUeJbQKgkPkGhBkvzT6qSLnLqpKd0gT0PbXgcgn1y3M0O35MqaOj ixWCcq/QDdAkxg7jb628gxQO1pgXT086d8Xreq1QQAcxuKvl9UiMiQyG5gxoOijfso+T MqRiOhqMbp13CxDhDxN2argyzD4T9D5m+9ggPpunVZOku1eBq57+F1vluWAoUUNywohi vxCA== X-Gm-Message-State: AOAM533as903pklk4+VNS5P34Z0u3o2hkhmptVCkia3B5yw59riSJ1/D xO0kaGPGH0R5N42tEF/eTC4= X-Google-Smtp-Source: ABdhPJyBAp9owrsAfLgtmeoTzk6EGj3S9yhX24KyVsty+2JN9PWNljDsMFTcAhhxDmedLBudzk9ZHg== X-Received: by 2002:a50:d8cf:: with SMTP id y15mr19412186edj.66.1633363499052; Mon, 04 Oct 2021 09:04:59 -0700 (PDT) Received: from kista.localnet (cpe-86-58-29-253.static.triera.net. [86.58.29.253]) by smtp.gmail.com with ESMTPSA id p10sm2240068ejf.45.2021.10.04.09.04.57 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 04 Oct 2021 09:04:58 -0700 (PDT) From: Jernej =?utf-8?B?xaBrcmFiZWM=?= To: mripard@kernel.org, paul.kocialkowski@bootlin.com, Hans Verkuil Cc: wens@csie.org, mchehab@kernel.org, gregkh@linuxfoundation.org, linux-media@vger.kernel.org, linux-staging@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-sunxi@lists.linux.dev, linux-kernel@vger.kernel.org Subject: Re: Re: [PATCH] media: cedrus: Don't kernel map most buffers Date: Mon, 04 Oct 2021 18:04:56 +0200 Message-ID: <2933502.8iB9r8yIO8@kista> In-Reply-To: <0cb81145-16d9-fd28-832e-4010646e9512@xs4all.nl> References: <20210912060812.222996-1-jernej.skrabec@gmail.com> <0cb81145-16d9-fd28-832e-4010646e9512@xs4all.nl> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20211004_090642_725798_42BFE7B6 X-CRM114-Status: GOOD ( 36.92 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi! Dne ponedeljek, 04. oktober 2021 ob 10:10:55 CEST je Hans Verkuil napisal(a): > Hi Jernej, > > Some comments below: > > On 12/09/2021 08:08, Jernej Skrabec wrote: > > Except VP8 probability coefficients buffer, all other buffers are never > > Except -> Except for > Ok. > > accessed by CPU. That allows us to mark them with DMA_ATTR_NO_KERNEL_MAPPING > > flag. This helps with decoding big (like 4k) videos on 32-bit ARM > > platforms where default vmalloc size is relatively small - 240 MiB. > > Since auxiliary buffer are not yet efficiently allocated, this can be > > easily exceeded. Even if allocation is optimized, 4k videos will still > > often exceed this limit. > > > > Signed-off-by: Jernej Skrabec > > --- > > .../staging/media/sunxi/cedrus/cedrus_h264.c | 102 ++++++++++-------- > > .../staging/media/sunxi/cedrus/cedrus_h265.c | 28 ++--- > > .../staging/media/sunxi/cedrus/cedrus_video.c | 2 + > > 3 files changed, 73 insertions(+), 59 deletions(-) > > > > diff --git a/drivers/staging/media/sunxi/cedrus/cedrus_h264.c b/drivers/ staging/media/sunxi/cedrus/cedrus_h264.c > > index de7442d4834d..6e38b37d9fe1 100644 > > --- a/drivers/staging/media/sunxi/cedrus/cedrus_h264.c > > +++ b/drivers/staging/media/sunxi/cedrus/cedrus_h264.c > > @@ -538,23 +538,23 @@ static int cedrus_h264_start(struct cedrus_ctx *ctx) > > > > ctx->codec.h264.pic_info_buf_size = pic_info_size; > > ctx->codec.h264.pic_info_buf = > > - dma_alloc_coherent(dev->dev, ctx- >codec.h264.pic_info_buf_size, > > - &ctx- >codec.h264.pic_info_buf_dma, > > - GFP_KERNEL); > > + dma_alloc_attrs(dev->dev, ctx- >codec.h264.pic_info_buf_size, > > + &ctx- >codec.h264.pic_info_buf_dma, > > + GFP_KERNEL, DMA_ATTR_NO_KERNEL_MAPPING); > > if (!ctx->codec.h264.pic_info_buf) > > return -ENOMEM; > > > > /* > > * That buffer is supposed to be 16kiB in size, and be aligned > > - * on 16kiB as well. However, dma_alloc_coherent provides the > > + * on 16kiB as well. However, dma_alloc_attrs provides the > > * guarantee that we'll have a CPU and DMA address aligned on > > Does the 'CPU' part of this sentence still make sense since the CPU > won't access the buffer? > Good catch! I'll remove it. > > * the smallest page order that is greater to the requested > > * size, so we don't have to overallocate. > > */ > > ctx->codec.h264.neighbor_info_buf = > > - dma_alloc_coherent(dev->dev, CEDRUS_NEIGHBOR_INFO_BUF_SIZE, > > - &ctx- >codec.h264.neighbor_info_buf_dma, > > - GFP_KERNEL); > > + dma_alloc_attrs(dev->dev, CEDRUS_NEIGHBOR_INFO_BUF_SIZE, > > + &ctx- >codec.h264.neighbor_info_buf_dma, > > + GFP_KERNEL, DMA_ATTR_NO_KERNEL_MAPPING); > > I think it would be good to have a comment for all these dma_alloc_attrs > calls where you note that these buffers are used by the HW only, and > never by the CPU, hence the use of DMA_ATTR_NO_KERNEL_MAPPING. > Ok. > > if (!ctx->codec.h264.neighbor_info_buf) { > > ret = -ENOMEM; > > goto err_pic_buf; > > @@ -582,10 +582,11 @@ static int cedrus_h264_start(struct cedrus_ctx *ctx) > > > > mv_col_size = field_size * 2 * CEDRUS_H264_FRAME_NUM; > > ctx->codec.h264.mv_col_buf_size = mv_col_size; > > - ctx->codec.h264.mv_col_buf = dma_alloc_coherent(dev->dev, > > - ctx->codec.h264.mv_col_buf_size, > > - &ctx->codec.h264.mv_col_buf_dma, > > - GFP_KERNEL); > > + ctx->codec.h264.mv_col_buf = > > + dma_alloc_attrs(dev->dev, > > + ctx->codec.h264.mv_col_buf_size, > > + &ctx->codec.h264.mv_col_buf_dma, > > + GFP_KERNEL, DMA_ATTR_NO_KERNEL_MAPPING); > > if (!ctx->codec.h264.mv_col_buf) { > > ret = -ENOMEM; > > goto err_neighbor_buf; > > @@ -600,10 +601,10 @@ static int cedrus_h264_start(struct cedrus_ctx *ctx) > > ctx->codec.h264.deblk_buf_size = > > ALIGN(ctx->src_fmt.width, 32) * 12; > > ctx->codec.h264.deblk_buf = > > - dma_alloc_coherent(dev->dev, > > - ctx- >codec.h264.deblk_buf_size, > > - &ctx- >codec.h264.deblk_buf_dma, > > - GFP_KERNEL); > > + dma_alloc_attrs(dev->dev, > > + ctx- >codec.h264.deblk_buf_size, > > + &ctx- >codec.h264.deblk_buf_dma, > > + GFP_KERNEL, DMA_ATTR_NO_KERNEL_MAPPING); > > if (!ctx->codec.h264.deblk_buf) { > > ret = -ENOMEM; > > goto err_mv_col_buf; > > @@ -616,10 +617,10 @@ static int cedrus_h264_start(struct cedrus_ctx *ctx) > > ctx->codec.h264.intra_pred_buf_size = > > ALIGN(ctx->src_fmt.width, 64) * 5 * 2; > > ctx->codec.h264.intra_pred_buf = > > - dma_alloc_coherent(dev->dev, > > - ctx- >codec.h264.intra_pred_buf_size, > > - &ctx- >codec.h264.intra_pred_buf_dma, > > - GFP_KERNEL); > > + dma_alloc_attrs(dev->dev, > > + ctx- >codec.h264.intra_pred_buf_size, > > + &ctx- >codec.h264.intra_pred_buf_dma, > > + GFP_KERNEL, DMA_ATTR_NO_KERNEL_MAPPING); > > if (!ctx->codec.h264.intra_pred_buf) { > > ret = -ENOMEM; > > goto err_deblk_buf; > > @@ -629,24 +630,28 @@ static int cedrus_h264_start(struct cedrus_ctx *ctx) > > return 0; > > > > err_deblk_buf: > > - dma_free_coherent(dev->dev, ctx->codec.h264.deblk_buf_size, > > - ctx->codec.h264.deblk_buf, > > - ctx->codec.h264.deblk_buf_dma); > > + dma_free_attrs(dev->dev, ctx->codec.h264.deblk_buf_size, > > + ctx->codec.h264.deblk_buf, > > + ctx->codec.h264.deblk_buf_dma, > > + DMA_ATTR_NO_KERNEL_MAPPING); > > > > err_mv_col_buf: > > - dma_free_coherent(dev->dev, ctx->codec.h264.mv_col_buf_size, > > - ctx->codec.h264.mv_col_buf, > > - ctx->codec.h264.mv_col_buf_dma); > > + dma_free_attrs(dev->dev, ctx->codec.h264.mv_col_buf_size, > > + ctx->codec.h264.mv_col_buf, > > + ctx->codec.h264.mv_col_buf_dma, > > + DMA_ATTR_NO_KERNEL_MAPPING); > > > > err_neighbor_buf: > > - dma_free_coherent(dev->dev, CEDRUS_NEIGHBOR_INFO_BUF_SIZE, > > - ctx->codec.h264.neighbor_info_buf, > > - ctx->codec.h264.neighbor_info_buf_dma); > > + dma_free_attrs(dev->dev, CEDRUS_NEIGHBOR_INFO_BUF_SIZE, > > + ctx->codec.h264.neighbor_info_buf, > > + ctx->codec.h264.neighbor_info_buf_dma, > > + DMA_ATTR_NO_KERNEL_MAPPING); > > > > err_pic_buf: > > - dma_free_coherent(dev->dev, ctx->codec.h264.pic_info_buf_size, > > - ctx->codec.h264.pic_info_buf, > > - ctx->codec.h264.pic_info_buf_dma); > > + dma_free_attrs(dev->dev, ctx->codec.h264.pic_info_buf_size, > > + ctx->codec.h264.pic_info_buf, > > + ctx->codec.h264.pic_info_buf_dma, > > + DMA_ATTR_NO_KERNEL_MAPPING); > > return ret; > > } > > > > @@ -654,23 +659,28 @@ static void cedrus_h264_stop(struct cedrus_ctx *ctx) > > { > > struct cedrus_dev *dev = ctx->dev; > > > > - dma_free_coherent(dev->dev, ctx->codec.h264.mv_col_buf_size, > > - ctx->codec.h264.mv_col_buf, > > - ctx->codec.h264.mv_col_buf_dma); > > - dma_free_coherent(dev->dev, CEDRUS_NEIGHBOR_INFO_BUF_SIZE, > > - ctx->codec.h264.neighbor_info_buf, > > - ctx->codec.h264.neighbor_info_buf_dma); > > - dma_free_coherent(dev->dev, ctx->codec.h264.pic_info_buf_size, > > - ctx->codec.h264.pic_info_buf, > > - ctx->codec.h264.pic_info_buf_dma); > > + dma_free_attrs(dev->dev, ctx->codec.h264.mv_col_buf_size, > > + ctx->codec.h264.mv_col_buf, > > + ctx->codec.h264.mv_col_buf_dma, > > + DMA_ATTR_NO_KERNEL_MAPPING); > > + dma_free_attrs(dev->dev, CEDRUS_NEIGHBOR_INFO_BUF_SIZE, > > + ctx->codec.h264.neighbor_info_buf, > > + ctx->codec.h264.neighbor_info_buf_dma, > > + DMA_ATTR_NO_KERNEL_MAPPING); > > + dma_free_attrs(dev->dev, ctx->codec.h264.pic_info_buf_size, > > + ctx->codec.h264.pic_info_buf, > > + ctx->codec.h264.pic_info_buf_dma, > > + DMA_ATTR_NO_KERNEL_MAPPING); > > if (ctx->codec.h264.deblk_buf_size) > > - dma_free_coherent(dev->dev, ctx- >codec.h264.deblk_buf_size, > > - ctx->codec.h264.deblk_buf, > > - ctx->codec.h264.deblk_buf_dma); > > + dma_free_attrs(dev->dev, ctx- >codec.h264.deblk_buf_size, > > + ctx->codec.h264.deblk_buf, > > + ctx->codec.h264.deblk_buf_dma, > > + DMA_ATTR_NO_KERNEL_MAPPING); > > if (ctx->codec.h264.intra_pred_buf_size) > > - dma_free_coherent(dev->dev, ctx- >codec.h264.intra_pred_buf_size, > > - ctx->codec.h264.intra_pred_buf, > > - ctx- >codec.h264.intra_pred_buf_dma); > > + dma_free_attrs(dev->dev, ctx- >codec.h264.intra_pred_buf_size, > > + ctx->codec.h264.intra_pred_buf, > > + ctx->codec.h264.intra_pred_buf_dma, > > + DMA_ATTR_NO_KERNEL_MAPPING); > > } > > > > static void cedrus_h264_trigger(struct cedrus_ctx *ctx) > > diff --git a/drivers/staging/media/sunxi/cedrus/cedrus_h265.c b/drivers/ staging/media/sunxi/cedrus/cedrus_h265.c > > index 3d9561d4aadb..bb7eb56106c5 100644 > > --- a/drivers/staging/media/sunxi/cedrus/cedrus_h265.c > > +++ b/drivers/staging/media/sunxi/cedrus/cedrus_h265.c > > @@ -351,10 +351,10 @@ static void cedrus_h265_setup(struct cedrus_ctx *ctx, > > ctx->codec.h265.mv_col_buf_unit_size; > > > > ctx->codec.h265.mv_col_buf = > > - dma_alloc_coherent(dev->dev, > > - ctx- >codec.h265.mv_col_buf_size, > > - &ctx- >codec.h265.mv_col_buf_addr, > > - GFP_KERNEL); > > + dma_alloc_attrs(dev->dev, > > + ctx- >codec.h265.mv_col_buf_size, > > + &ctx- >codec.h265.mv_col_buf_addr, > > + GFP_KERNEL, DMA_ATTR_NO_KERNEL_MAPPING); > > if (!ctx->codec.h265.mv_col_buf) { > > ctx->codec.h265.mv_col_buf_size = 0; > > // TODO: Abort the process here. > > @@ -668,9 +668,9 @@ static int cedrus_h265_start(struct cedrus_ctx *ctx) > > ctx->codec.h265.mv_col_buf_size = 0; > > > > ctx->codec.h265.neighbor_info_buf = > > - dma_alloc_coherent(dev->dev, CEDRUS_H265_NEIGHBOR_INFO_BUF_SIZE, > > - &ctx- >codec.h265.neighbor_info_buf_addr, > > - GFP_KERNEL); > > + dma_alloc_attrs(dev->dev, CEDRUS_H265_NEIGHBOR_INFO_BUF_SIZE, > > + &ctx- >codec.h265.neighbor_info_buf_addr, > > + GFP_KERNEL, DMA_ATTR_NO_KERNEL_MAPPING); > > if (!ctx->codec.h265.neighbor_info_buf) > > return -ENOMEM; > > > > @@ -682,16 +682,18 @@ static void cedrus_h265_stop(struct cedrus_ctx *ctx) > > struct cedrus_dev *dev = ctx->dev; > > > > if (ctx->codec.h265.mv_col_buf_size > 0) { > > - dma_free_coherent(dev->dev, ctx- >codec.h265.mv_col_buf_size, > > - ctx->codec.h265.mv_col_buf, > > - ctx- >codec.h265.mv_col_buf_addr); > > + dma_free_attrs(dev->dev, ctx- >codec.h265.mv_col_buf_size, > > + ctx->codec.h265.mv_col_buf, > > + ctx->codec.h265.mv_col_buf_addr, > > + DMA_ATTR_NO_KERNEL_MAPPING); > > > > ctx->codec.h265.mv_col_buf_size = 0; > > } > > > > - dma_free_coherent(dev->dev, CEDRUS_H265_NEIGHBOR_INFO_BUF_SIZE, > > - ctx->codec.h265.neighbor_info_buf, > > - ctx->codec.h265.neighbor_info_buf_addr); > > + dma_free_attrs(dev->dev, CEDRUS_H265_NEIGHBOR_INFO_BUF_SIZE, > > + ctx->codec.h265.neighbor_info_buf, > > + ctx->codec.h265.neighbor_info_buf_addr, > > + DMA_ATTR_NO_KERNEL_MAPPING); > > } > > > > static void cedrus_h265_trigger(struct cedrus_ctx *ctx) > > diff --git a/drivers/staging/media/sunxi/cedrus/cedrus_video.c b/drivers/ staging/media/sunxi/cedrus/cedrus_video.c > > index 66714609b577..800ffa5382de 100644 > > --- a/drivers/staging/media/sunxi/cedrus/cedrus_video.c > > +++ b/drivers/staging/media/sunxi/cedrus/cedrus_video.c > > @@ -568,6 +568,7 @@ int cedrus_queue_init(void *priv, struct vb2_queue *src_vq, > > > > src_vq->type = V4L2_BUF_TYPE_VIDEO_OUTPUT; > > src_vq->io_modes = VB2_MMAP | VB2_DMABUF; > > + src_vq->dma_attrs = DMA_ATTR_NO_KERNEL_MAPPING; > > src_vq->drv_priv = ctx; > > src_vq->buf_struct_size = sizeof(struct cedrus_buffer); > > src_vq->ops = &cedrus_qops; > > @@ -584,6 +585,7 @@ int cedrus_queue_init(void *priv, struct vb2_queue *src_vq, > > > > dst_vq->type = V4L2_BUF_TYPE_VIDEO_CAPTURE; > > dst_vq->io_modes = VB2_MMAP | VB2_DMABUF; > > + src_vq->dma_attrs = DMA_ATTR_NO_KERNEL_MAPPING; > > This should be dst_vq! > > I'm also not quite sure that it is right to use this for the destination > buffer. Could this cause problems if this buffer is exported to a dmabuf > and another driver will be using it and requiring a kernel mapping. Hm, if I fix this, it actually breaks DRM mapping. For some reason it expects vmap to be valid. I'll drop this for now. Thanks for review! Best regards, Jernej > > Regards, > > Hans > > > dst_vq->drv_priv = ctx; > > dst_vq->buf_struct_size = sizeof(struct cedrus_buffer); > > dst_vq->ops = &cedrus_qops; > > > > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel