linux-kernel - Re: has the i915 "black screen" boot issue returned? [BISECTED]

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <201101282112.52798.rjw@sisk.pl>
Date:	Fri, 28 Jan 2011 21:12:52 +0100
From:	"Rafael J. Wysocki" <rjw@...k.pl>
To:	"Robert P. J. Day" <rpjday@...shcourse.ca>
Cc:	Chris Wilson <chris@...is-wilson.co.uk>,
	dri-devel@...ts.freedesktop.org, Len Brown <len.brown@...el.com>,
	linux-kernel@...r.kernel.org
Subject: Re: has the i915 "black screen" boot issue returned? [BISECTED]

On Friday, January 28, 2011, Robert P. J. Day wrote:
> On Fri, 28 Jan 2011, Chris Wilson wrote:
> 
> > On Fri, 28 Jan 2011 09:51:01 -0500 (EST), "Robert P. J. Day" <rpjday@...shcourse.ca> wrote:
> > >   sadly, no change -- still black screen.  again, rebooted
> > > successfully under commit 8a327f23.  just to be clear, here's "git
> > > diff":
> > >
> > > $ git diff
> > > diff --git a/include/linux/acpi_io.h b/include/linux/acpi_io.h
> > > index 7180013..e035f3c 100644
> > > --- a/include/linux/acpi_io.h
> > > +++ b/include/linux/acpi_io.h
> > > @@ -7,7 +7,7 @@
> > >  static inline void __iomem *acpi_os_ioremap(acpi_physical_address phys,
> > >                                             acpi_size size)
> > >  {
> > > -       return ioremap_cache(phys, size);
> > > +       return ioremap(phys, size);
> > >  }
> >
> > Ok, that implies the new mapping is fine and not the cause of the issue.
> >
> > Instead you have some OpRegion related regression hidden until till now
> > because the conflicting mapping disabled it for i915.
> >
> > That would be easy to test by returning early in intel_opregion_setup():
> >
> > diff --git a/drivers/gpu/drm/i915/intel_opregion.c
> > b/drivers/gpu/drm/i915/intel_
> > index 9efccb9..8c93201 100644
> > --- a/drivers/gpu/drm/i915/intel_opregion.c
> > +++ b/drivers/gpu/drm/i915/intel_opregion.c
> > @@ -470,6 +470,8 @@ int intel_opregion_setup(struct drm_device *dev)
> >         u32 asls, mboxes;
> >         int err = 0;
> >
> > +       return -ENOTSUPP;
> > +
> >
> >         pci_read_config_dword(dev->pdev, PCI_ASLS, &asls);
> >         DRM_DEBUG_DRIVER("graphic opregion physical addr: 0x%x\n", asls);
> >         if (asls == 0) {
> >
> 
> so you want me to revert to a stock b705120e before doing the above?

Alternatively, you could take the vanilla Linus' tree and replace
ioremap_cache() with ioremap() in include/linux/acpi_io.h .  Please try that
and see if it makes a difference.

Thanks,
Rafael
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/