[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <51A84B33.70109@nvidia.com>
Date: Fri, 31 May 2013 10:03:15 +0300
From: Terje Bergström <tbergstrom@...dia.com>
To: Arto Merilainen <amerilainen@...dia.com>
CC: "thierry.reding@...il.com" <thierry.reding@...il.com>,
"airlied@...ux.ie" <airlied@...ux.ie>,
"linux-tegra@...r.kernel.org" <linux-tegra@...r.kernel.org>,
"dri-devel@...ts.freedesktop.org" <dri-devel@...ts.freedesktop.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCHv2 0/7] Miscellaneous fixes to host1x
On 29.05.2013 13:26, Arto Merilainen wrote:
> This patch series fixes two issues in the host1x driver: First, the
> command buffer validation routine had vulnerabilities that were not
> detected in earlier testing. Second, the return codes of some
> functions were misleading or completely missing. This caused the
> driver to give wrong return codes also to the userspace.
>
> The series is based on top of 3.10rc3. I have tested the patch series
> on cardhu by running host1x and gr2d test cases (available at [0]).
> I would appreciate any help in testing/reviewing these patches.
>
> Changes from v1:
> * Rebased on top of 3.10rc3
> * Split firewall fixes to smaller patches
> * Reworked no-reloc case in firewall code. Fix in v1 was not
> sufficient in all cases
> * Dropped patch "Fix syncpoint wait return value" as it is not
> critical and discussion on it has not yet settled.
> * Fixed style and whitespace issues
>
> [0] https://gitorious.org/linux-host1x/libdrm-host1x
>
> Arto Merilainen (5):
> gpu: host1x: Check reloc table before usage
> gpu: host1x: Copy gathers before verification
> gpu: host1x: Fix memory access in syncpt request
> gpu: host1x: Fix client_managed type
> gpu: host1x: Rework CPU syncpoint increment
>
> Terje Bergstrom (2):
> gpu: host1x: Check INCR opcode correctly
> gpu: host1x: Don't reset firewall between gathers
>
> drivers/gpu/host1x/dev.h | 8 +--
> drivers/gpu/host1x/drm/drm.c | 3 +-
> drivers/gpu/host1x/drm/gr2d.c | 2 +-
> drivers/gpu/host1x/hw/cdma_hw.c | 2 +-
> drivers/gpu/host1x/hw/syncpt_hw.c | 12 ++--
> drivers/gpu/host1x/job.c | 135 +++++++++++++++++---------------------
> drivers/gpu/host1x/syncpt.c | 26 +++-----
> drivers/gpu/host1x/syncpt.h | 13 ++--
> 8 files changed, 85 insertions(+), 116 deletions(-)
>
Arto's patches above,
Acked-By: Terje Bergstrom <tbergstrom@...dia.com>
Terje
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists