Messages in this thread | | | Date | Tue, 19 Jun 2012 08:45:29 -0300 | From | Eugeni Dodonov <> | Subject | Re: [ 04/69] drm/i915: fix up ivb plane 3 pageflips |
| |
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA256
On 06/19/2012 12:59 AM, Ben Hutchings wrote: > On Mon, 2012-06-18 at 19:10 -0300, Eugeni Dodonov wrote: >> On 06/18/2012 06:55 PM, Herton Ronaldo Krzesinski wrote: >>>> + default: + WARN_ONCE(1, "unknown plane in flip >>>> command\n"); + ret = -ENODEV; + goto err; >>> >>> Shouldn't this have been "goto err_unpin;"? >>> >>> It's not a problem in the backport for 3.2, same code is in the >>> original patch, just noticed it seems a problem while looking >>> at it; but by what changelog says the default case is said to >>> not trigger in "practice" (catch programming error), so I think >>> it doesn't pose a real issue in practice. >> >> You are right, it should be err_unpin. I sent a patch rectifying >> this. >> >> In practice, this should not happen, but still, better be safe >> than sorry. Thanks for catching that! > > So should I drop this until the second patch is upstream?
I don't think there is a need. It is nearly impossible to hit this situation, and if it happens the machine will be quite broken in a way this patch won't help recovering it anyway.
So you could merge the original patch as is - it fixes some real issues on Ivy Bridge which are much more likely to happen.
Eugeni -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.19 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/
iQEcBAEBCAAGBQJP4GZZAAoJEPE+raS4SlCBBncH/08HYA+d304lNiEKSq7i/0i7 V399aS9KtimsXNKsf7wal017hmxevujc5fElY5lFYM0RTw3nEUGmnOlWZ0FdRt/S BuIz6BQ5RcZBexdYxwss/J/grLo5Wv/v6oBf+0zb2nJVSt6fmbU8vWixnbmsiQF/ uMZJ5o1wD/Dj4gIF8g91altkLMDE5HBrUC6HqF/pPSd+h+Xp6+VHXlre5+YZ85ZK HefD9kjWSMLzWZfvbySua7hqoEb2Z5/bSzyvtOFtf3teg/uLICvX6wiNhUjahGCQ 6gHTZz+FtzQ7iqqtO3+6loQrcx/u7cbNi8HoJkwyIKO8bxth1Rv1DWUPsHoCb/8= =UozC -----END PGP SIGNATURE-----
| |