From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751100AbcEDEjR (ORCPT ); Wed, 4 May 2016 00:39:17 -0400 Received: from mout.gmx.net ([212.227.15.19]:53207 "EHLO mout.gmx.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750708AbcEDEjQ (ORCPT ); Wed, 4 May 2016 00:39:16 -0400 Date: Wed, 4 May 2016 06:39:02 +0200 From: Stefan Lippers-Hollmann To: Linus Torvalds Cc: Mauro Carvalho Chehab , Linux Media Mailing List , Linux Kernel Mailing List Subject: Re: [GIT PULL for v4.6-rc1] media updates Message-ID: <20160504063902.0af2f4d7@mir> In-Reply-To: References: <20160315080552.3cc5d146@recife.lan> <20160503233859.0f6506fa@mir> X-Mailer: Claws Mail 3.13.2 (GTK+ 2.24.30; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-Provags-ID: V03:K0:Dx6h8ioWr5hVRfqA8hvuNEAbB32M2MbpxV+WSXspFbfWMMl8e2T 1Gcpx7XoOdSGlepEVwFOMdp3o9wBmaqNzycNlN0/u/PZSgjcNmJkuK0enlKPyJWYDDn1k2m x+aywqgKdpcmbYH4Fsfio0YmpsaqVaEEoprUdqwm5WUxYUFlyFAW6uxziVPkHGDaQ+sSlWE QXV2lvrf54Hjn9SX1Akwg== X-UI-Out-Filterresults: notjunk:1;V01:K0:amcyUL/MAnE=:xNWL7RGIen98NVzoCFljNQ 2P0IatduL9kL7oLIocyA8Y/v3yGXlOq5Vj4xkzbY/Y3Q9EXwZVTO7YXjCx57yOri+RzpQPfsj 5B8Jjy48TIFkvcK08eYUOVHRf7x3FBXEpeTD4n4+GiEANKzuPZxf3bdcR7m3R73n2nQTDaCfN D9A/tVDuoYxNS02LmOCqTMWSrX039zz0a+eROePcI60hjqdYr1elgBMBGC/RTOhuC5R8uqmjA Cij4W0v3DtzE5oPeXd2/e5Zg0mXC8V3MbRMFS1PNlVCfH38TLW2d9uvLCUdsMysH4FJ2ARLO3 Cf/3vGVxdfk9Ww9PSpmeohHwEvaFvMQQhCfsfqdJpUtgAMcCILMmeuYmajlt8XlwTCfi6+cwE 21YDVjEnYxTe5HAGFth96h7/mpXJ3pixDZrv0F/mFl+XIgaeklifnKoUcgsCOhB58NqUvC7Ap liBI0pVYJCHoKm4IClZ/BWcVhOptSVG9ZkESyMIFEXjWw3q1xkTt0bbnayTfKqQHLaAWHpzWv 0lZp9lq2mFq2jwz2TasEU4vq7W5kh21Azz4TExnuJz9rH5bzjDfeilIe4ibjpfduzXzfIDm+i kOH2O7C9Xma4OV5K1/QIoAb6bcTbY+JFg/EvhTAIeVuUI1Ldf4qA/HEzLtQEIR35Jiek2uMVo mkDhhO6+IGeTgMB8qwsgfWxRupd3jZ4EhWDTMbOB9BYxaIgOfvONjTq5zay3K8xCgqr4VINxs oFfCpoRzi4Ez8A//Y8S1d3TMyDTSiQh9iaREN9zUNpuVJLUjQMlE9rRmovxSsib94gwq6C3fP 9W08DGi Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi On 2016-05-03, Linus Torvalds wrote: > On Tue, May 3, 2016 at 2:38 PM, Stefan Lippers-Hollmann wrote: > > Hi > > [...] > >> Mauro Carvalho Chehab (95): > > [...] > >> [media] use v4l2_mc_usb_media_device_init() on most USB devices > > [...] > > > > This change, as part of v4.6-rc6-85-g1248ded, breaks two systems, each > > equipped with a TeVii s480 (dvb_usb_dw2102) DVB-S2 card, for me (kernel > > v4.5.3-rc1 is fine): > > From the oops it looks like the "->prev" pointer in one of the list > heads in 'mdev' is NULL. > > > [ 5.041915] BUG: unable to handle kernel NULL pointer dereference at (null) > > [ 5.041921] IP: [] media_gobj_create+0xb8/0x100 [media] > > I can't tell *which* list head it is, but it looks like there's a > missing call to media_device_init() which is what should have > initialized those list heads. > > Of course, maybe that list pointer got initialized but then > overwritten by NULL for some other reason. Just as a cross-check, this (incomplete, but au0828, cx231xx and em28xx aren't needed/ loaded on my system) crude revert avoids the problem for me on v4.6-rc6-113-g83858a7. --- a/drivers/media/usb/dvb-usb/dvb-usb-dvb.c +++ b/drivers/media/usb/dvb-usb/dvb-usb-dvb.c @@ -107,7 +107,15 @@ static int dvb_usb_media_device_init(str if (!mdev) return -ENOMEM; - media_device_usb_init(mdev, udev, d->desc->name); + mdev->dev = &udev->dev; + strlcpy(mdev->model, d->desc->name, sizeof(mdev->model)); + if (udev->serial) + strlcpy(mdev->serial, udev->serial, sizeof(mdev->serial)); + strcpy(mdev->bus_info, udev->devpath); + mdev->hw_revision = le16_to_cpu(udev->descriptor.bcdDevice); + mdev->driver_version = LINUX_VERSION_CODE; + + media_device_init(mdev); dvb_register_media_controller(&adap->dvb_adap, mdev); --- a/drivers/media/usb/dvb-usb-v2/dvb_usb_core.c +++ b/drivers/media/usb/dvb-usb-v2/dvb_usb_core.c @@ -412,7 +412,15 @@ static int dvb_usbv2_media_device_init(s if (!mdev) return -ENOMEM; - media_device_usb_init(mdev, udev, d->name); + mdev->dev = &udev->dev; + strlcpy(mdev->model, d->name, sizeof(mdev->model)); + if (udev->serial) + strlcpy(mdev->serial, udev->serial, sizeof(mdev->serial)); + strcpy(mdev->bus_info, udev->devpath); + mdev->hw_revision = le16_to_cpu(udev->descriptor.bcdDevice); + mdev->driver_version = LINUX_VERSION_CODE; + + media_device_init(mdev); dvb_register_media_controller(&adap->dvb_adap, mdev); While testing this, I also noticed that not just the dvb_usb based dvb_usb_dw2102 is affected, but that also the dvb_usb_v2 based rtl2832 triggers this Oops on its own (given that just reverting drivers/media/usb/dvb-usb/dvb-usb-dvb.c wasn't enough). Regards Stefan Lippers-Hollmann