From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paul Walmsley Subject: Re: "drm: Add reference counting to blob properties" breaks Tegra124 Jetson TK1 in linux-next Date: Tue, 12 May 2015 09:25:48 -0600 Message-ID: <55521B7C.3030708@nvidia.com> References: <55517AE5.30700@nvidia.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Daniel Stone Cc: Daniel Stone , Daniel Vetter , "dri-devel@lists.freedesktop.org" , "linux-tegra@vger.kernel.org" , Thierry Reding , "linux-arm-kernel@lists.infradead.org" List-Id: linux-tegra@vger.kernel.org SGkgRGFuaWVsLAoKT24gMDUvMTIvMjAxNSAwMzowNiBBTSwgRGFuaWVsIFN0b25lIHdyb3RlOgo+ IEhpIFBhdWwsCj4KPiBPbiAxMiBNYXkgMjAxNSBhdCAwNTowMCwgUGF1bCBXYWxtc2xleSA8cHdh bG1zbGV5QG52aWRpYS5jb20+IHdyb3RlOgo+PiBUZWdyYTEyNCBKZXRzb24gVEsxIGhhbmdzIGR1 cmluZyBib290IGluIG5leHQtMjAxNTA1MDggYW5kIGJleW9uZDoKPj4KPj4gaHR0cDovL252dC5w d3Nhbi5jb20vZXhwZXJpbWVudGFsL2xpbnV4LW5leHQvdGVzdGxvZ3MvdGVzdF9uZXh0LTIwMTUw NTA4LzIwMTUwNTA4MTAxMDE4L2Jvb3QvdGVncmExMjQtamV0c29uLXRrMS90ZWdyYTEyNC1qZXRz b24tdGsxL3RlZ3JhX2RlZmNvbmZpZ19sb2cudHh0Cj4+IGh0dHA6Ly9udnQucHdzYW4uY29tL2V4 cGVyaW1lbnRhbC9saW51eC1uZXh0L3Rlc3Rsb2dzL3Rlc3RfbmV4dC0yMDE1MDUxMS8yMDE1MDUx MTAwNDUzMy9ib290L3RlZ3JhMTI0LWpldHNvbi10azEvdGVncmExMjQtamV0c29uLXRrMS90ZWdy YV9kZWZjb25maWdfbG9nLnR4dAo+Pgo+PiBSZXZlcnRpbmcgY29tbWl0IDY3MmNiMWQ2YWVjN2Rh Mjc5OWFmZDFiNTI5ZDUxMzZkODRlZDI1NjEgKCJkcm06IEFkZAo+PiByZWZlcmVuY2UgY291bnRp bmcgdG8gYmxvYiBwcm9wZXJ0aWVzIikgZ2V0cyBpdCBib290aW5nIGFnYWluOgo+Pgo+PiBodHRw Oi8vbnZ0LnB3c2FuLmNvbS9wdWIvcHdhbG1zbGV5LXRlc3Rlci90ZXN0bG9ncy90ZXN0XzIwMTUw NTExMjA0NzAzXzM4MzgyMzI0MzFjMjYzNmZmNTM5ZjIzNTRjNjMwYzBjYTNkNGI3YWQvMjAxNTA1 MTEyMDQ3MDMvYm9vdC90ZWdyYTEyNC1qZXRzb24tdGsxL3RlZ3JhMTI0LWpldHNvbi10azEvdGVn cmFfZGVmY29uZmlnX2xvZy50eHQKPj4KPj4gSSdtIG5vdCBhIERSTSBleHBlcnQgYnV0IG1heWJl IG9uZSBvZiB5b3Uga25vd3Mgd2hhdCdzIGdvaW5nIG9uIGhlcmU/Cj4gVGhpZXJyeSBmb3VuZCBh bmQgZml4ZWQgdGhlIHJhdGhlciBzdHVwaWQgYnVnOiBub3QgdXNpbmcgdGhlIGNhY2hlZAo+ICdk ZXYnIHZhcmlhYmxlIGluIHRoZSBvbmUgcGxhY2UgaXQgd291bGQndmUgYmVlbiB1c2VmdWwhCj4K PiBodHRwOi8vcGFzdGUuZGViaWFuLm5ldC8xNzU1MjIvIGlzIHRoZSBmaXgsIHdoaWNoIGhhcyBi ZWVuIHNxdWFzaGVkIGluCj4gdG8gdGhlIGJyYW5jaCBhbmQgc2hvdWxkIGFwcGVhciBpbiAtbmV4 dCBmcm9tIHRvZGF5Lgo+Cj4gVGhhbmtzIGZvciB0aGUgcmVwb3J0LgoKR3JlYXQhICBUaGFua3Mg ZXZlcnlvbmUgZm9yIHRoZSB2ZXJ5IGZhc3QgZml4LgoKCi0gUGF1bAoKX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVsIG1haWxpbmcgbGlzdApk cmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHA6Ly9saXN0cy5mcmVlZGVza3RvcC5v cmcvbWFpbG1hbi9saXN0aW5mby9kcmktZGV2ZWwK From mboxrd@z Thu Jan 1 00:00:00 1970 From: pwalmsley@nvidia.com (Paul Walmsley) Date: Tue, 12 May 2015 09:25:48 -0600 Subject: "drm: Add reference counting to blob properties" breaks Tegra124 Jetson TK1 in linux-next In-Reply-To: References: <55517AE5.30700@nvidia.com> Message-ID: <55521B7C.3030708@nvidia.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Daniel, On 05/12/2015 03:06 AM, Daniel Stone wrote: > Hi Paul, > > On 12 May 2015 at 05:00, Paul Walmsley wrote: >> Tegra124 Jetson TK1 hangs during boot in next-20150508 and beyond: >> >> http://nvt.pwsan.com/experimental/linux-next/testlogs/test_next-20150508/20150508101018/boot/tegra124-jetson-tk1/tegra124-jetson-tk1/tegra_defconfig_log.txt >> http://nvt.pwsan.com/experimental/linux-next/testlogs/test_next-20150511/20150511004533/boot/tegra124-jetson-tk1/tegra124-jetson-tk1/tegra_defconfig_log.txt >> >> Reverting commit 672cb1d6aec7da2799afd1b529d5136d84ed2561 ("drm: Add >> reference counting to blob properties") gets it booting again: >> >> http://nvt.pwsan.com/pub/pwalmsley-tester/testlogs/test_20150511204703_3838232431c2636ff539f2354c630c0ca3d4b7ad/20150511204703/boot/tegra124-jetson-tk1/tegra124-jetson-tk1/tegra_defconfig_log.txt >> >> I'm not a DRM expert but maybe one of you knows what's going on here? > Thierry found and fixed the rather stupid bug: not using the cached > 'dev' variable in the one place it would've been useful! > > http://paste.debian.net/175522/ is the fix, which has been squashed in > to the branch and should appear in -next from today. > > Thanks for the report. Great! Thanks everyone for the very fast fix. - Paul