From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tomi Valkeinen Subject: Re: omapdrm/pvr kernel crash with camera capture and display application Date: Tue, 2 Dec 2014 15:15:27 +0200 Message-ID: <547DBB6F.6030503@ti.com> References: Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1643590836==" Return-path: Received: from bear.ext.ti.com (bear.ext.ti.com [192.94.94.41]) by gabe.freedesktop.org (Postfix) with ESMTP id 3F95C6E8B8 for ; Tue, 2 Dec 2014 05:22:02 -0800 (PST) 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: Vikas Patil , Rob Clark , "dri-devel@lists.freedesktop.org" List-Id: dri-devel@lists.freedesktop.org --===============1643590836== Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="AgOCD0swGCh38PFp761Epwmvpifs7LjI8" --AgOCD0swGCh38PFp761Epwmvpifs7LjI8 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Hi, On 02/12/14 14:41, Vikas Patil wrote: > Hi All, >=20 > What I found is UnwrapExtMemoryCallBack() function from > eurasia_km\services4\srvkm\common\devicemem.c always calls > omap_gem_put_pages(), however just before the crash it calls > omap_gem_put_paddr() and it crashes in it due to NULL pointer > dereference of &obj->dev->struct_mutex. instead of > omap_gem_put_pages(). Does this gives any clue on what might be going > wrong? What OMAP_BO_TILED_MASK signifies? OMAP_BO_TILED_MASK tells that the buffer has been allocated with TILER. I'm not really familiar with this stuff, but I presume a matching operation is done in PVRSRVImportGEMKM(). But but... If &obj->dev->struct_mutex causes a NULL ref, it sounds to me that the memory has already been released, or somehow else a NULL has gotten into there. Even if the code would accidentally call omap_gem_put_paddr instead of omap_gem_put_pages, the struct_mutex should be fine. Tomi --AgOCD0swGCh38PFp761Epwmvpifs7LjI8 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJUfbtvAAoJEPo9qoy8lh71AYIP/0zhpUfS55D+HZCC4B/7dipH Tkn2AaplvuK7HGXYANkQuUFZ2nPQpoK6n180lHTrowoRxQCQnxtiTnhUrNP9YqPc Y2sGZMsbLaFrof/xA/e/Np2qVVmvamc1A6Lvb0pm5Z/RpNy+UDjY9kLoyKbJCe6D OKLPzGPad9zEEIukjk632AzEVkWz77RtV5DQAkYytobUDklXG6yRbzFe3CLjzeIr QKR2yxo/+atZPHgh4AEW/iT11aaZixSB/jkVpS+XtWZXe07BbJX8I4WucM11hBBY /P5kK8X5mz+m/e7M62oxqF2cp2YTyFS3ICY0HjDR+7nLQ8YE0veQZP7R7eyQDCqL ewj9KLbXRz2UDK0STISENP6tJNBy8iYZ0huif+QVhEUO3jiX5tlwwXVyekgyVf1X 01lmTK4hqg1N1Ni0zz1e13/vHglQ/fKQsXvPAOI18VjaR7yHPBu3xRNLjy0V2PeZ JLMPr3HEUMEWIF1WkT5uSZUH2e2KnR3Qc9Jop5LgmvLvjiZ29RTCQhfTZsrCIFTo 5S8tch9gCNhZZKtu7sj7jIOt8YlF5VdD4CrNR3pNE81q2uy30woSwWYptVu9LOGN 9Fy7u6ioZS0hpBhrlTk/MQzOOPY+diUJvJsBJAcysXlYPxr3wq2wlWTk5dR5gHyy tHngq3KUc9Mjbtzh3e18 =bTau -----END PGP SIGNATURE----- --AgOCD0swGCh38PFp761Epwmvpifs7LjI8-- --===============1643590836== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVs IG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHA6Ly9saXN0 cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9kcmktZGV2ZWwK --===============1643590836==--