<br><br><div class="gmail_quote">2012/11/8 Rob Clark <span dir="ltr"><<a href="mailto:rob.clark@linaro.org" target="_blank">rob.clark@linaro.org</a>></span><br><blockquote style="margin:0px 0px 0px 0.8ex;padding-left:1ex;border-left-color:rgb(204,204,204);border-left-width:1px;border-left-style:solid" class="gmail_quote">
<div class="HOEnZb"><div class="h5">On Wed, Nov 7, 2012 at 10:25 AM, Inki Dae <<a href="mailto:inki.dae@samsung.com">inki.dae@samsung.com</a>> wrote:<br>
><br>
><br>
> 2012/11/7 Imre Deak <<a href="mailto:imre.deak@intel.com">imre.deak@intel.com</a>><br>
>><br>
>> On Wed, 2012-11-07 at 18:31 +0900, Inki Dae wrote:<br>
>> > 2012/11/2 Imre Deak <<a href="mailto:imre.deak@intel.com">imre.deak@intel.com</a>><br>
>> >         The patchset adds the missing event_lock when accessing the<br>
>> >         vblank_event_list in drm_vblank_off() and as preparation for<br>
>> >         this<br>
>> >         also fixes a few other issues in the exynos driver.<br>
>> >         This is also a dependency for Rob Clark's<br>
>> >         drm_send_vblank_event()<br>
>> >         rework as that would trigger a warning for the unhold<br>
>> >         event_lock without<br>
>> >         this changeset.<br>
>> >         The exynos changes are only compile tested, the rest is tested<br>
>> >         on an<br>
>> >         Intel IVB machine on top of drm-intel-nightly +<br>
>> >         drm_send_vblank_event()<br>
>> >         rework, with i-g-t/flip_test.<br>
>> > Hi Imre,<br>
>> > Works fine. But we should wait for Rob's patch set to be merged to<br>
>> > -next, and this may be rebased on top of latest Rob's patch set again.<br>
>><br>
>> Ok, thanks for checking this. I assume then that this patchset will get<br>
>> merged through your tree.<br>
>><br>
>> I think Rob's patchset depends on this, so ideally this should go first.<br>
>> Otherwise the i915 driver would trigger the WARN in his patchset due to<br>
>> the unheld event_lock.<br>
><br>
><br>
> Ok, but I merge it first, shouldn't Rob's patch set be rebased? Anyway this<br>
> is minor issue so I could resolve it. And it seems like that your patch set<br>
> has no dependency of Rob's. I mean that your patch set worked fine without<br>
> Rob's.<br>
<br>
</div></div>I think there should be no hard dependency on my patch set.. the only<br>
connection is that my patchset without this patch will start showing<br>
the WARN_ON() traces<br>
<br></blockquote><div> </div><div>Right, My concern was just merge conflict.</div><div> </div><blockquote style="margin:0px 0px 0px 0.8ex;padding-left:1ex;border-left-color:rgb(204,204,204);border-left-width:1px;border-left-style:solid" class="gmail_quote">

BR,<br>
-R<br>
<div class="HOEnZb"><div class="h5"><br>
> Thanks,<br>
> Inki Dae<br>
><br>
>><br>
>><br>
>> --Imre<br>
>><br>
>><br>
>> _______________________________________________<br>
>> dri-devel mailing list<br>
>> <a href="mailto:dri-devel@lists.freedesktop.org">dri-devel@lists.freedesktop.org</a><br>
>> <a href="http://lists.freedesktop.org/mailman/listinfo/dri-devel" target="_blank">http://lists.freedesktop.org/mailman/listinfo/dri-devel</a><br>
><br>
><br>
><br>
> _______________________________________________<br>
> dri-devel mailing list<br>
> <a href="mailto:dri-devel@lists.freedesktop.org">dri-devel@lists.freedesktop.org</a><br>
> <a href="http://lists.freedesktop.org/mailman/listinfo/dri-devel" target="_blank">http://lists.freedesktop.org/mailman/listinfo/dri-devel</a><br>
><br>
_______________________________________________<br>
dri-devel mailing list<br>
<a href="mailto:dri-devel@lists.freedesktop.org">dri-devel@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/dri-devel" target="_blank">http://lists.freedesktop.org/mailman/listinfo/dri-devel</a><br>
</div></div></blockquote></div><br>