Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753530Ab3EaHCE (ORCPT ); Fri, 31 May 2013 03:02:04 -0400 Received: from hqemgate14.nvidia.com ([216.228.121.143]:5199 "EHLO hqemgate14.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751028Ab3EaHB5 (ORCPT ); Fri, 31 May 2013 03:01:57 -0400 X-PGP-Universal: processed; by hqnvupgp07.nvidia.com on Fri, 31 May 2013 00:01:38 -0700 Message-ID: <51A84B33.70109@nvidia.com> Date: Fri, 31 May 2013 10:03:15 +0300 From: =?ISO-8859-1?Q?Terje_Bergstr=F6m?= User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130329 Thunderbird/17.0.5 MIME-Version: 1.0 To: Arto Merilainen CC: "thierry.reding@gmail.com" , "airlied@linux.ie" , "linux-tegra@vger.kernel.org" , "dri-devel@lists.freedesktop.org" , "linux-kernel@vger.kernel.org" Subject: Re: [PATCHv2 0/7] Miscellaneous fixes to host1x References: <1369823168-5396-1-git-send-email-amerilainen@nvidia.com> In-Reply-To: <1369823168-5396-1-git-send-email-amerilainen@nvidia.com> Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2188 Lines: 54 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 Terje -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/