From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from gate.ebshome.net (gate.ebshome.net [208.106.21.240]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client CN "gate.ebshome.net", Issuer "gate.ebshome.net" (not verified)) by ozlabs.org (Postfix) with ESMTP id 251FADE055 for ; Sat, 12 Jan 2008 09:38:33 +1100 (EST) Date: Fri, 11 Jan 2008 14:38:31 -0800 From: Eugene Surovegin To: Benjamin Herrenschmidt Subject: Re: [PATCH 0/2] [PPC 4xx] L2-cache synchronization for ppc44x Message-ID: <20080111223831.GA2947@gate.ebshome.net> References: <7310408706.20071107014010@emcraft.com> <20071128195037.GB22325@gate.ebshome.net> <200801111824.46920.yur@emcraft.com> <20080111174152.GA17240@gate.ebshome.net> <1200089135.6896.59.camel@pasglop> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <1200089135.6896.59.camel@pasglop> Cc: linuxppc-dev@ozlabs.org, sr@denx.de, dzu@denx.de List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Sat, Jan 12, 2008 at 09:05:35AM +1100, Benjamin Herrenschmidt wrote: > > > > The s/w synchronization algorithms proposed in my patches has no LL PLB > > > limitations as opposed to h/w snooping, but, probably, this is not the best > > > way of how it might be implemented. Even though with these patches the h/w > > > accelerated RAID starts to operate correctly (with L2-cache enabled) there is > > > a performance degradation (induced by loops in the L2-cache synchronization > > > routines) observed in the most cases. So, as a result, there is no benefit > > > from using L2-cache for these, RAID, cases at all. > > > > Thanks a lot for explanation, Yuri. I'd never imagine they were so > > stupid to make new chips with such behaviour. > > Indeed. Now the question is do we want to make that configurable by the > platform so it can select whether to enable snooping, or use this > mechanism (in which case we can disable snooping on the L2) ? I don't think we should panish platforms with sane L2 caches, because there are some brain-dead ones. > Another option would be to make the dma_ops smart enough to know whether > a given device is on the snooped portion of the bus, which would be > easier to do after I merge 32 and 64 bits DMA ops, so we get the ability > to change the dma-ops per bus or per device even. > > What do you guys think ? I like the idea of having smart DMA routines with different per-bus/device behaviour. -- Eugene