>> I would expect that if the device is not in D0, we should remember the >> hardware updates that need to be made, and if the device returns to >> D0, we should apply the updates then. If that's the case this is not >> an error and we shouldn't warn about it. > [...] > > During system suspend all IRQ affinities are changed to the boot CPU as > other CPUs are disabled, and then I think they are reverted during > system resume. This certainly used to be done while most devices were > suspended. That's why we have to check for the device power state here. > If suspend/resume is still done in the same order then we shouldn't log > a warning about this. > > Yijing, why not add a check for this in pci_enable_msi() itself if you > think it's that important to warn about? This is a good idea, better than just warn a inconspicuous message. Thanks! Yijing. -- Thanks! Yijing -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html