From: Arto Merilainen <amerilainen@nvidia.com>
To: <thierry.reding@gmail.com>, <airlied@linux.ie>,
<linux-tegra@vger.kernel.org>, <dri-devel@lists.freedesktop.org>
Cc: <tbergstrom@nvidia.com>, <linux-kernel@vger.kernel.org>,
Arto Merilainen <amerilainen@nvidia.com>
Subject: [PATCHv2 0/7] Miscellaneous fixes to host1x
Date: Wed, 29 May 2013 13:26:01 +0300 [thread overview]
Message-ID: <1369823168-5396-1-git-send-email-amerilainen@nvidia.com> (raw)
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(-)
--
1.8.1.5
next reply other threads:[~2013-05-29 10:27 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-05-29 10:26 Arto Merilainen [this message]
2013-05-29 10:26 ` [PATCHv2 1/7] gpu: host1x: Check INCR opcode correctly Arto Merilainen
2013-05-29 10:26 ` [PATCHv2 2/7] gpu: host1x: Check reloc table before usage Arto Merilainen
2013-05-29 10:26 ` [PATCHv2 3/7] gpu: host1x: Don't reset firewall between gathers Arto Merilainen
2013-05-29 11:21 ` Thierry Reding
2013-05-29 11:29 ` Arto Merilainen
2013-05-29 10:26 ` [PATCHv2 4/7] gpu: host1x: Copy gathers before verification Arto Merilainen
2013-05-29 10:26 ` [PATCHv2 5/7] gpu: host1x: Fix memory access in syncpt request Arto Merilainen
2013-05-29 10:26 ` [PATCHv2 6/7] gpu: host1x: Fix client_managed type Arto Merilainen
2013-05-29 10:26 ` [PATCHv2 7/7] gpu: host1x: Rework CPU syncpoint increment Arto Merilainen
2013-05-29 11:24 ` [PATCHv2 0/7] Miscellaneous fixes to host1x Thierry Reding
2013-05-31 7:03 ` Terje Bergström
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1369823168-5396-1-git-send-email-amerilainen@nvidia.com \
--to=amerilainen@nvidia.com \
--cc=airlied@linux.ie \
--cc=dri-devel@lists.freedesktop.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-tegra@vger.kernel.org \
--cc=tbergstrom@nvidia.com \
--cc=thierry.reding@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Powered by JetHome