[lkml]   [2005]   [May]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [patch] properly stop devices before poweroff

    > >>Without this patch, Linux provokes emergency disk shutdowns and
    > >>similar nastiness. It was in SuSE kernels for some time, IIRC.
    > >>
    > >
    > >
    > >With this patch when running `halt -p' my ia64 Tiger (using
    > >tiger_defconfig) gets a stream of badnesses in iosapic_unregister_intr()
    > >and then hangs up.
    > >
    > >Unfortunately it all seems to happen after the serial port has been
    > >disabled because nothing comes out. I set the console to a squitty font
    > >and took a piccy. See
    > >
    > >
    > >I guess it's an ia64 problem. I'll leave the patch in -mm for now.
    > >
    > I guess the stream of badness was occured as follows:
    > pcibios_disable_device() for ia64 assumes that pci_enable_device()
    > and pci_disable_device() are balanced. But with 'properly stop
    > devices before power off' patch, pci_disable_device() becomes to be
    > called twice for e1000 device at halt time, through reboot_notifier_list
    > callback and through device_suspend(). As a result,
    > iosapic_unregister_intr()
    > was called for already unregistered gsi and then stream of badness
    > was displayed.
    > I think the following patch will remove this stream of badness. I'm
    > sorry but I have not checked if the stream of badness is actually
    > removed because I'm on vacation and I can't look at my display
    > (I'm working via remote console). Could you try this patch?
    > By the way, I don't think this stream of badness is related to hang up,
    > because the problem (hang up) was reproduced even on my test kernel that
    > doesn't call pcibios_disable_device().

    Looks good to me. Plus I guess we should remove reboot notifier from
    e1000... It should be obsoleted by driver model.

    Boycott Kodak -- for their patent abuse against Java.
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2005-05-01 22:02    [W:0.021 / U:47.700 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site