Re: [RFT] x86 acpi: normalize segment descriptor register on resume

[Rafael J. Wysocki]

On Sunday, 13 of July 2008, Andi Kleen wrote:
> Andy Lutomirski wrote:
> > Matthew Garrett wrote:
> >> On Sun, Jul 13, 2008 at 11:15:24AM +0200, Ingo Molnar wrote:
> >>
> >>> we still need to find the HAL quirk and disable it, right?
> >>
> >> Not without understanding what the cause is. If the video BIOS calls
> >> are generically broken, then we have a problem.
> >>
> > 
> > The HAL quirk is the very first one here:
> > 
> > http://gitweb.freedesktop.org/?p=hal-info.git;a=blob;f=fdi/information/10freedesktop/20-video-quirk-pm-lenovo.fdi
> > 
> > 
> > I removed it from the HAL config and suspending w/ the hardware button
> > works fine now with -rc9-wl and on Ubuntu's stock .24 kernel.
> 
> Hmm, but the change was not supposed to break the s3 bios. Something
> fishy is going on. It sounds like the s3 bios relies on some earlier
> segment register setup.

Well, we changed the (visible) parts of the segment registers before anyway.

This means that it could only depend on the hidden parts.  However, in that
case if it depended on the hidden part of SS, our stack would be broken, so
the quirk wouldn't work (it uses 'call' to run a BIOS routine).  In turn, if it
depended on the hidden part of DS, our data register would be broken, so the
resume code itself wouldn't work.

This means it could only depend on the hidden part of ES.
 
> If true this means the segment register reset would need to be moved
> later after S3 bios ran.

We can't do that.  If SS contains garbage, the BIOS call itself will reboot
the box and if DS contains garbage, well ...

> Saving/restoring is unfortunately not possible because we cannot save/restore
> the hidden state loaded from the GDT earlier. 
> 
> This is unfortunately a little tricky with the new C wakeup code.
> 
> > I'll file the obvious bug report against HAL, but it might annoy users
> > if new kernel + old HAL = broken system, 
> 
> It's the bad side effect of HAL effectively being an out of tree kernel
> driver (that just by chance happens to run in user space). Really
> all these s3 quirks at least should be in the kernel.
> 
> We can't really do much about that now, but longer term it might be useful
> to invent some mechanism to tell HAL to disable specific quirks from
> the kernel.

They are in the kernel.  In fact, there's a sysctl to switch them on/off and
that's what HAL uses, AFAICS.

Apparently, you can tell HAL not to do that by editing one of its files.

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