From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:43986) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Qudha-0002mf-3R for qemu-devel@nongnu.org; Sat, 20 Aug 2011 01:02:35 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1QudhY-00016U-QP for qemu-devel@nongnu.org; Sat, 20 Aug 2011 01:02:34 -0400 Received: from mail.mc.net ([209.172.128.24]:47140) by eggs.gnu.org with smtp (Exim 4.71) (envelope-from ) id 1QudhY-00016M-L1 for qemu-devel@nongnu.org; Sat, 20 Aug 2011 01:02:32 -0400 Message-ID: <4E4F4041.5000208@mc.net> Date: Sat, 20 Aug 2011 00:04:01 -0500 From: Bob Breuer MIME-Version: 1.0 References: <4E43FF4E.7080501@mc.net> <4E48F1A0.3080904@siriusit.co.uk> <4E493D74.6020303@mc.net> <4E4ECD0A.70207@siriusit.co.uk> In-Reply-To: <4E4ECD0A.70207@siriusit.co.uk> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] sparc32_dma: correctly initialize ledma base address List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Mark Cave-Ayland wrote: > On 15/08/11 16:38, Bob Breuer wrote: > >> Depends on the rom. The SS-5 rom always sets it correctly, whereas the >> SS-20 rom only sets it when you do "boot net". Also, this is just the >> top 8 bits of the address. The DMA2 documentation[1] for E_BASE_ADDR >> states that these upper address bits default to 0xff, even though it >> seems to incorrectly define it as bits 7:0 in the register instead of >> 31:24. > > Nice one - looks like I missed this when reading the documentation. At > least the choice of default address now makes sense. It might also make sense to modify the dma address calculations to use only the top 8 bits from ledma. Not sure if anything will care about that though. >> If you follow Artyom's blog, at [2] it was assumed that the bogus dbri >> device was the culprit (which is also why I went down the path of >> implementing the dbri device), when in reality, the selftest failure >> was preventing "boot net" from running and fixing the ledma register >> settings. > > Okay - I think I see ;) In that case, I'd say this patch should be > applied if Blue hasn't already done it (*sigh* I really miss the git web > interface on qemu.org). http://repo.or.cz/w/qemu.git might be the next best thing, not sure if there is a mirroring delay with it or not. Bob