Re: [visorchipset] invalid opcode: 0000 [#1] PREEMPT SMP

[Greg Kroah-Hartman <gregkh@linuxfoundation.org>]

On Mon, Apr 07, 2014 at 12:23:47PM -0700, Greg Kroah-Hartman wrote:
> On Mon, Apr 07, 2014 at 09:24:37AM -0500, Ken Cox wrote:
> > 
> > On 04/07/2014 09:09 AM, Greg Kroah-Hartman wrote:
> > >On Mon, Apr 07, 2014 at 07:17:25PM +0800, Fengguang Wu wrote:
> > >>Hi Ken,
> > >>
> > >>I got the below dmesg and the first bad commit is
> > >>
> > >>git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> > >>
> > >>commit 12e364b9f08aa335dc7716ce74113e834c993765
> > >>Author:     Ken Cox <jkc@redhat.com>
> > >>AuthorDate: Tue Mar 4 07:58:07 2014 -0600
> > >>Commit:     Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> > >>CommitDate: Tue Mar 4 16:58:21 2014 -0800
> > >>
> > >>     staging: visorchipset driver to provide registration and other services
> > >I think Sasha has already sent a fix to resolve this issue that I'll be
> > >sending to Linus in a day or so.
> > >
> > >Ken, is Sasha's patch going to resolve this issue as well?  It looks
> > >like people haven't tested what happens when the module is loaded
> > >without the hardware present in the system :(
> > You are exactly right.  The driver needs to check for hardware early on
> > before trying to use it.  Unfortunately, Sasha's patch will not resolve this
> > one.  I'll work with Ben Romer to get a patch out ASAP.
> 
> Wait, in looking at this closer, I don't see any of the "normal"
> hardware checks to determine that this really is a valid piece of
> hardware present, before it starts to just go and initialize a whole
> bunch of things (sysfs busses, proc files and directories, and other
> things.)
> 
> That's not ok, and it's obvious it's starting to affect people's work
> systems.
> 
> How about I just mark the whole thing BROKEN for now, disabling the
> build, until "correct" hardware probing can be added to the driver, so
> no one else gets hurt by this?

In looking at it further, that seems like the best thing to do for now,
we can slowly enable the driver back after things like proper device
probing is fixed up so as to not break people's boxes.

thanks,

greg k-h