Re: [PATCH] fix tulip suspend/resume

Benjamin Herrenschmidt
2005-06-08T23:03:03Z

> > I think we should also use the pm_message_t defines. We will need to
> > add PMSG_FREEZE eventually. I decided to default to the current state
> > rather than panic. Does this patch look ok?
>
> No.

Hrm... I don't follow you anymore here ...

> case PM_EVENT_ON:
> return PCI_D0;
> case PM_EVENT_FREEZE:
> case PM_EVENT_SUSPEND:
> return PCI_D3hot;

What are these new PM_EVENT_* things ? I though we defined PMSG_* ?

> You passed invalid argument; I see no reason why you should paper over
> it and risk continuing. This happens during system suspend; it is
> quite possible that user will not see your printk when machine powers
> off just after that; and remember that it will not be in syslog after
> resume.

Crap. I don't think a BUG() makes any useful help neither in this place,
and when I locally turn PMSG_FREEZE to something sane I suddenly blow up
in there (and I wonder in how many other places).

Ben.