From: Dave Airlie <airlied@gmail.com>
To: Chris Wilson <chris@chris-wilson.co.uk>,
Dave Airlie <airlied@gmail.com>, Daniel Vetter <daniel@ffwll.ch>,
Aaron Plattner <aplattner@nvidia.com>,
dri-devel <dri-devel@lists.freedesktop.org>
Subject: Re: [RFC] drm + i915 DP MST code preview
Date: Thu, 8 May 2014 17:32:53 +1000 [thread overview]
Message-ID: <CAPM=9tzP5ZDV6w82Bd6OVTTAc2HFuCaaMGYc0RjSRF0PpLwUBw@mail.gmail.com> (raw)
In-Reply-To: <20140508072402.GB20315@nuc-i3427.alporthouse.com>
On 8 May 2014 17:24, Chris Wilson <chris@chris-wilson.co.uk> wrote:
> On Thu, May 08, 2014 at 11:56:02AM +1000, Dave Airlie wrote:
>> I've pushed a branch of -modesetting to
>>
>> http://cgit.freedesktop.org/~airlied/xf86-video-modesetting/log/?h=dp-mst
>
> In parse_path_blob():
> char conn_id[5];
>
> len = conn - (blob_data + 4);
> if (len + 1> 5) return -1;
>
> memcpy(conn_id, blob_data + 4, len);
> conn_id[len + 1] = '\0';
> ^^^^^^^ should just be len.
Nice catch, thanks, I'll fix that,
The kernel side for this along with even more locking and refcounting
fixes is in
http://cgit.freedesktop.org/~airlied/linux/log/?h=drm-i915-mst-support
Dave.
next prev parent reply other threads:[~2014-05-08 7:32 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-05-02 4:39 [RFC] drm + i915 DP MST code preview Dave Airlie
2014-05-02 4:39 ` [PATCH 1/8] drm/dp_helper: add defines for DP 1.2 and MST support Dave Airlie
2014-05-02 4:39 ` [PATCH 2/8] drm: add DP MST encoder type Dave Airlie
2014-05-02 4:39 ` [PATCH 3/8] drm/i915: add some registers need for displayport MST support Dave Airlie
2014-05-02 4:39 ` [PATCH 4/8] drm/crtc: add interface to reinitialise the legacy mode group Dave Airlie
2014-05-02 4:39 ` [PATCH 5/8] drm/helper: add Displayport multi-stream helper Dave Airlie
2014-05-02 4:39 ` [PATCH 6/8] i915: split some DP modesetting code into a separate function Dave Airlie
2014-05-02 4:39 ` [PATCH 7/8] drm/i915: check connector->encoder before using it Dave Airlie
2014-05-02 4:39 ` [PATCH 8/8] i915: add DP 1.2 MST support (v0.1) Dave Airlie
2014-05-02 4:51 ` [RFC] drm + i915 DP MST code preview Dave Airlie
2014-05-02 8:52 ` Chris Wilson
2014-05-02 21:08 ` Dave Airlie
2014-05-03 9:00 ` Chris Wilson
2014-05-07 7:16 ` Aaron Plattner
2014-05-07 8:07 ` Daniel Vetter
2014-05-07 16:46 ` Aaron Plattner
2014-05-07 22:24 ` Dave Airlie
2014-05-08 1:56 ` Dave Airlie
2014-05-08 7:24 ` Chris Wilson
2014-05-08 7:32 ` Dave Airlie [this message]
2014-05-07 9:00 ` Dave Airlie
2014-05-07 16:42 ` Aaron Plattner
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='CAPM=9tzP5ZDV6w82Bd6OVTTAc2HFuCaaMGYc0RjSRF0PpLwUBw@mail.gmail.com' \
--to=airlied@gmail.com \
--cc=aplattner@nvidia.com \
--cc=chris@chris-wilson.co.uk \
--cc=daniel@ffwll.ch \
--cc=dri-devel@lists.freedesktop.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.