lkml.org 
[lkml]   [2004]   [Oct]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectConcerns about our pci_{save,restore}_state()
From
Date
Hi Greg !

I was looking at our "generic" pci_save_state() and pci_restore_state()
and I have various concerns with them, I was wondering what you though
about them...

- We should always write the command register after all the BARs,
typically that mean write it back _last_
- We shouldn't write to the BIST register, it is defined as having
side effects and writing to it any value may trigger a BIST on the
card, with all the possible bad consequences that has
- What about saving/restoring more registers ? I'm not sure wether it
should be the responsibility of the driver to save and restore things
above dword 15, but we should at least deal with the case of P2P bridges
who have more "standard" registers

In addition, we currently have no mecanism to save/restore the state of
P2P bridges. Shouldn't we do that in pci_device_suspend() if there is no
driver attached ?

Ben.


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

\
 
 \ /
  Last update: 2005-03-22 14:07    [W:0.175 / U:0.076 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site