[Cbe-oss-dev] [PATCH] spufs: avoid using backing operation for wbox write

André Detsch adetsch at br.ibm.com
Tue Aug 7 06:08:54 EST 2007


Arnd Bergmann wrote:
> However, we recently had a bug in that area (fixed in 2.6.23-rc1), where Linux
> accidentally saves and restores the channel 0 twice, which can lead to broken
> events. Noguchi-san originally found that problem and the fix that went into
> the kernel is archived at
> http://patchwork.ozlabs.org/cbe-oss-dev/patch?id=12483
 >
> Could the new problem be fixed by this patch as well? What was the symptom
> that this patch was supposed to fix?

Unfortunately, the patch does not fixes the problem.
However, I've misunderstood the SPU events concept. Sorry for the 
confusion. There may be a problem when dealing with events as well, but 
this is not related to the symptom I'm facing here.

The problem I have is a stall on channel 29.
Repeating some tests I've did on a QS20, now on simulator, reveals that, 
although the channel 29 count is being written to the spu in the 
restore_spu_mb function [out_be64(&priv2->spu_chnlcnt_RW, 
csa->spu_chnlcnt_RW[29])], the channel 29 count reported by the 
simulator is not being updated to the value stored in spu_chnlcnt_RW[29].

--
Andre Detsch




More information about the cbe-oss-dev mailing list