From: Jani Nikula <jani.nikula@linux.intel.com>
To: Daniel Vetter <daniel@ffwll.ch>,
Maarten Lankhorst <maarten.lankhorst@ubuntu.com>
Cc: intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org
Subject: Re: [PATCH 5/5] drm/core: Fix old_fb handling in pan_display_atomic.
Date: Tue, 17 Nov 2015 13:04:36 +0200 [thread overview]
Message-ID: <87oaesevjv.fsf@intel.com> (raw)
In-Reply-To: <20151117095847.GM16848@phenom.ffwll.local>
On Tue, 17 Nov 2015, Daniel Vetter <daniel@ffwll.ch> wrote:
> On Wed, Nov 11, 2015 at 11:29:11AM +0100, Maarten Lankhorst wrote:
>> From: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
>>
>> Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
>
> Needs "Don't touch plane->old_fb/fb without having the right locks held."
> like the previous patch in the commit message. With that for patches 4&5:
>
> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Pushed the series to topic/drm-fixes, with the comments added, thanks
for the patches and review.
BR,
Jani.
--
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
prev parent reply other threads:[~2015-11-17 11:04 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-11-11 10:29 [PATCH 0/5] Atomic core fixes Maarten Lankhorst
2015-11-11 10:29 ` [PATCH 1/5] drm/core: Set legacy_cursor_update in drm_atomic_helper_disable_plane Maarten Lankhorst
2015-11-17 9:51 ` Daniel Vetter
2015-11-11 10:29 ` [PATCH 2/5] drm/core: Fix old_fb handling in drm_mode_atomic_ioctl Maarten Lankhorst
2015-11-17 9:54 ` Daniel Vetter
2015-11-17 9:54 ` Daniel Vetter
2015-11-11 10:29 ` [PATCH 3/5] drm/atomic: add a drm_atomic_clean_old_fb helper Maarten Lankhorst
2015-11-17 9:58 ` Daniel Vetter
2015-11-17 10:55 ` Maarten Lankhorst
2015-11-11 10:29 ` [PATCH 4/5] drm/core: Fix old_fb handling in restore_fbdev_mode_atomic Maarten Lankhorst
2015-11-11 10:29 ` [PATCH 5/5] drm/core: Fix old_fb handling in pan_display_atomic Maarten Lankhorst
2015-11-17 9:58 ` Daniel Vetter
2015-11-17 11:04 ` Jani Nikula [this message]
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=87oaesevjv.fsf@intel.com \
--to=jani.nikula@linux.intel.com \
--cc=daniel@ffwll.ch \
--cc=dri-devel@lists.freedesktop.org \
--cc=intel-gfx@lists.freedesktop.org \
--cc=maarten.lankhorst@ubuntu.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 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.