Re: [PATCH v2] Add PPC/4xx "ibm,sdram-4xx-ddr2" EDAC MC Driver
From: Grant Erickson <hidden>
Date: 2008-12-18 23:44:16
On 12/18/08 11:13 AM, Grant Erickson wrote:
On 12/18/08 9:56 AM, Josh Boyer wrote:quoted
So, I am not an expert at the EDAC subsystem. Just looking at the number of functions that get call from this interrupt handler, I'm wondering if a significant amount of time could be spent here. With all the message building and such, it seems like it could be error prone.The message building functions are really best-effort, straight-through, so there should be little room if any for error-proneness. Though, snprintf contained therein is fairly costly as functions go.quoted
Do you have an idea of the duration of one of these events? Could this schedule a work queue to handle it outside of the hard IRQ context?I'll take some measurements for the worst case message length time. I certainly thought about a work queue; however, my gut feeling was that it wasn't warranted. Stay tuned for measurement data.
In general, correctable and uncorrectable ECC errors in and of themselves
should be rare events. So, by extension, the generation of these ECC error
messages should be rare as well.
When one of these CE or UE ECC events occurs, the time required to build the
driver-unique log message is as shown below for a run of ~300 forced CEs at
at a processor frequency of 400 MHz:
Ticks Time / us
--------------------------------
Minimum 4150 10.38
Maximum 9075 22.69
Mean 8024 20.06
Median 8297 20.74
Mode 8869 22.17
Std. Dev. 864 2.16
--------------------------------
Regards,
Grant