From owner-freebsd-hackers@FreeBSD.ORG Mon May 21 23:36:02 2012 Return-Path: Delivered-To: hackers@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id 77B971065674 for ; Mon, 21 May 2012 23:36:02 +0000 (UTC) (envelope-from kabaev@gmail.com) Received: from mail-qa0-f49.google.com (mail-qa0-f49.google.com [209.85.216.49]) by mx1.freebsd.org (Postfix) with ESMTP id 29B258FC15 for ; Mon, 21 May 2012 23:36:02 +0000 (UTC) Received: by qabj40 with SMTP id j40so2497682qab.15 for ; Mon, 21 May 2012 16:36:01 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=date:from:to:cc:subject:message-id:in-reply-to:references:x-mailer :mime-version:content-type; bh=OBfHjsa9EDKos5hwfjJOdWcg1JI6ECJENmfwMLA9sRE=; b=E6DOW4Zn+Nz816cYE0pfCKhLoExgnz5fPYF4HC0Qu1icNeNqnjf4DkWxW5ts5hfuDR CKufXH54P/UtpfB/9TqHac+9bKO/eEIgDsWbdaxwZ60KN5517VgRPiFfMBZyp0tlOL4H Or3kHAWDyrzpiZqU7Soxr7LmOjev+9U/90Kgg7CPbtaNViw2J5i/tSo4Q4Y7onCGDyLJ 7UXH0VkNiFu9DhdQ4h0EhJ9T0Q3oZGLYIWgD0FM9M3AUoxBHkTF3vF1HE3hVtPIjIwDD nLTdWdbyeGydR7j2Z5jcuplo/lU53kw6MwxxGnh6yqPZBVNB3KXtN4A43Z+rBtWwE/AX cRPQ== Received: by 10.224.115.208 with SMTP id j16mr41621545qaq.84.1337643361366; Mon, 21 May 2012 16:36:01 -0700 (PDT) Received: from kan.dyndns.org (c-24-63-226-98.hsd1.ma.comcast.net. [24.63.226.98]) by mx.google.com with ESMTPS id u2sm41860340qaa.13.2012.05.21.16.35.59 (version=SSLv3 cipher=OTHER); Mon, 21 May 2012 16:36:00 -0700 (PDT) Date: Mon, 21 May 2012 19:35:48 -0400 From: Alexander Kabaev To: Mark Tinguely Message-ID: <20120521193548.0b03a39a@kan.dyndns.org> In-Reply-To: References: X-Mailer: Claws Mail 3.8.0 (GTK+ 2.24.6; amd64-portbld-freebsd10.0) Mime-Version: 1.0 Content-Type: multipart/signed; micalg=PGP-SHA1; boundary="Sig_/pHCWVeAVip/ysIJVHmMEvUv"; protocol="application/pgp-signature" Cc: hackers@freebsd.org, Svatopluk Kraus Subject: Re: ARM + CACHE_LINE_SIZE + DMA X-BeenThere: freebsd-hackers@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Technical Discussions relating to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 21 May 2012 23:36:02 -0000 --Sig_/pHCWVeAVip/ysIJVHmMEvUv Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable On Thu, 17 May 2012 11:01:34 -0500 Mark Tinguely wrote: > On Thu, May 17, 2012 at 8:20 AM, Svatopluk Kraus > wrote: > > Hi, > > > > I'm working on DMA bus implementation for ARM11mpcore platform. I've > > looked at implementation in ARM tree, but IMHO it only works with > > some assumptions. There is a problem with DMA on memory block which > > is not aligned on CACHE_LINE_SIZE (start and end) if memory is not > > coherent. > > > > Let's have a buffer for DMA which is no aligned on CACHE_LINE_SIZE. > > Then first cache line associated with the buffer can be divided into > > two parts, A and B, where A is a memory we know nothing about it > > and B is buffer memory. The same stands for last cache line > > associatted with the buffer. We have no problem if a memory is > > coherent. Otherwise it depends on memory attributes. > > > > 1. [no cache] attribute > > No problem as memory is coherent. > > > > 2. [write throught] attribute > > The part A can be invalidated without loss of any data. It's not > > problem too. > > > > 3. [write back] attribute > > In general, there is no way how to keep both parts consistent. At > > the start of DMA transaction, the cache line is written back and > > invalidated. However, as we know nothing about memory associated > > with part A of the cache line, the cache line can be filled again > > at any time and messing up DMA transaction if flushed. Even if the > > cache line is only filled but not flushed during DMA transaction, > > we must make it coherent with memory after that. There is a trick > > with saving part A of the line into temporary buffer, invalidating > > the line, and restoring part A in current ARM (MIPS) > > implementation. However, if somebody is writting to memory > > associated with part A of the line during this trick, the part A > > will be messed up. Moreover, the part A can be part of another DMA > > transaction. > > > > To safely use DMA with no coherent memory, a memory with [no cache] > > or [write throught] attributes can be used without problem. A > > memory with [write back] attribute must be aligned on > > CACHE_LINE_SIZE. > > > > However, for example mbuf, a buffer for DMA can be part of a > > structure which can be aligned on CACHE_LINE_SIZE, but not the > > buffer itself. We can know that nobody will write to the structure > > during DMA transaction, so it's safe to use the buffer event if > > it's not aligned on CACHE_LINE_SIZE. > > > > So, in practice, if DMA buffer is not aligned on CACHE_LINE_SIZE and > > we want to avoid bounce pages overhead, we must support additional > > information to DMA transaction. It should be easy to support the > > information about drivers data buffers. However, what about OS data > > buffers like mentioned mbufs? > > > > The question is following. Is or can be guaranteed for all or at > > least well-known OS data buffers which can be part of DMA access > > that the not CACHE_LINE_SIZE aligned buffers are surrounded by data > > which belongs to the same object as the buffer and the data is not > > written by OS when given to a driver? > > > > Any answer is appreciated. However, 'bounce pages' is not an answer. > > > > Thanks, Svata >=20 > Sigh. A several ideas by several people, but a good answer has not > been created yet. SMP will make this worse. >=20 > To make things worse, there are drivers that use memory from the > stack as DMA buffers. >=20 > I was hoping that mbufs are pretty well self-contained, unless you > expect to modify them while DMA is happening. >=20 > This is on my to-do list. >=20 > --Mark. Drivers that do DMA from memory that was not allocated by proper busdma methods or load buffers for DMA using not properly constrained busdma tags are broken drivers. We did not have a busdma tag inheritance from parent bus to child devices before, but now we should just take advantage of that and just make cache line alignment a requirement for the platform. USB is firmly in that 'broken' category btw and is currently being worked around by the USB_HOST_ALIGN hack on MIPS, which suffers from the very same cache coherency issues you describe. --=20 Alexander Kabaev --Sig_/pHCWVeAVip/ysIJVHmMEvUv Content-Type: application/pgp-signature; name=signature.asc Content-Disposition: attachment; filename=signature.asc -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.18 (FreeBSD) iD8DBQFPutFeQ6z1jMm+XZYRAqB9AJ49ZfOlIH3tV+xHt3sX7QwEE04nhACeP4k3 YMdpbSyHoLsLAtCU9oYy9bQ= =ResP -----END PGP SIGNATURE----- --Sig_/pHCWVeAVip/ysIJVHmMEvUv--