Bug 9058 - Ocassionall panic due to incorrect AC line voltage measurement
: Ocassionall panic due to incorrect AC line voltage measurement
Status: RESOLVED FIXED
Product: SB Transporter
Classification: Unclassified
Component: Setup
: 60
: All All
: -- minor (vote)
: Investigating
Assigned To: Chris Owens
: TestCase
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-08-06 09:54 UTC by Sean Adams
Modified: 2009-09-08 09:19 UTC (History)
0 users

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Sean Adams 2008-08-06 09:54:49 UTC
In the latest firmware there is a bug which causes the firmware to sometimes incorrectly read the line voltage monitoring ADC. Probably a race condition with the other SPI config interface, which share one of those pins.
Comment 1 Blackketter Dean 2008-08-06 12:04:27 UTC
Sean: are you planning on fixing this in 7.2?
Comment 2 Sean Adams 2008-08-06 13:46:08 UTC
working on it now
Comment 3 Sean Adams 2008-08-06 14:52:37 UTC
I can't figure out why we get bad readings. It's pretty clear that it's something to do with the changes around V. 43 or so, but the spinlock code looks correct. I added a sanity check to require two consistent consecutive readings, which should prevent the over-voltage panic.

I can't say for sure if this bug is actually fixed because it could take thousands of hours to reproduce. We'll have to keep an eye on this, and we can close it after it's been proven in the field for a month. Leaving the bug open as a reminder.
Comment 4 Blackketter Dean 2008-08-08 10:13:45 UTC
Sean:  Do you consider this fixed with your change?
Comment 5 Sean Adams 2008-08-08 10:23:03 UTC
Yes but I can't test it. We should still track this issue in some way...
Comment 6 Blackketter Dean 2008-08-08 12:20:09 UTC
Chris: How do you want to track this?  We should do some basic verification of the voltage, then keep an eye on it post 7.2.
Comment 7 Andy Grundman 2008-08-08 14:34:57 UTC
Fixed in firmware 59.
Comment 8 Chris Owens 2008-08-08 14:50:07 UTC
I'll check and see what we have for transporter fw test cases and throw something in there.