From owner-freebsd-current@FreeBSD.ORG Mon Nov 17 15:50:55 2008 Return-Path: Delivered-To: freebsd-current@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 968271065674; Mon, 17 Nov 2008 15:50:55 +0000 (UTC) (envelope-from imp@bsdimp.com) Received: from harmony.bsdimp.com (bsdimp.com [199.45.160.85]) by mx1.freebsd.org (Postfix) with ESMTP id 4F3698FC08; Mon, 17 Nov 2008 15:50:55 +0000 (UTC) (envelope-from imp@bsdimp.com) Received: from localhost (localhost [127.0.0.1]) by harmony.bsdimp.com (8.14.2/8.14.1) with ESMTP id mAHFnp1U021194; Mon, 17 Nov 2008 08:49:52 -0700 (MST) (envelope-from imp@bsdimp.com) Date: Mon, 17 Nov 2008 08:51:10 -0700 (MST) Message-Id: <20081117.085110.1649769838.imp@bsdimp.com> To: hselasky@c2i.net From: "M. Warner Losh" In-Reply-To: <200811161408.21562.hselasky@c2i.net> References: <20081108001128.GA1437@icarus.home.lan> <200811081023.10058.hselasky@freebsd.org> <200811161408.21562.hselasky@c2i.net> X-Mailer: Mew version 5.2 on Emacs 21.3 / Mule 5.0 (SAKAKI) Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: freebsd-current@FreeBSD.org, freebsd-usb@FreeBSD.org Subject: Re: [Serious] busdma bug in -current in relation to USB hardware - review wanted X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 17 Nov 2008 15:50:55 -0000 In message: <200811161408.21562.hselasky@c2i.net> Hans Petter Selasky writes: : Hi, : : It turns out my initial patch need to be limited to the use-case only. Before : I start any real work, I want to know if anyone will object if I implement : the following new BUS_DMA flag into FreeBSD's busdma system? : : amd64/amd64/busdma_machdep.c : i386/i386/busdma_machdep.c : arm/arm/busdma_machdep.c : ia64/ia64/busdma_machdep.c : mips/mips/busdma_machdep.c : powerpc/powerpc/busdma_machdep.c : sparc64/sparc64/bus_machdep.c : sun4v/sun4v/bus_machdep.c : sys/bus_dma.h : : /* : * The following flag specifies that no re-alignment of individual : * memory pages is allowed when loaded into DMA. It can only be used : * when "maxsegsz" is equal to "PAGE_SIZE" and "alignment" is less : * than or equal to 1. : * : * Background: Some kinds of DMA hardware only stores the full : * physical address of the first memory page when multiple memory : * pages are loaded into DMA. Consequtive memory pages only gets the : * non-offset part of the physical address updated. The hardware : * computes the amount of data that should be stored in the first : * memory page from the minimum of the total transfer length and : * PAGE_SIZE minus the initial page offset. When the initial page : * offset is not preserved the hardware ends up transferring an : * invalid number of bytes to or from the initial memory page. : */ : #define BUS_DMA_NOREAL 0x400 /* no page re-alignment allowed */ : : : : Without this new feature in busdma the USB hardware will not work when the DMA : data is placed on bounce pages, for example on 64-bit architectures with more : than 4GB of RAM. I'm having trouble understanding the need for this patch. Can you provide a patch to usb2 to use it as well as doing one of the implementations (say i386 or amd64) so that it is easier to judge the problem it is trying to solve? Warner