From owner-cvs-all@FreeBSD.ORG Wed Nov 29 10:58:10 2006 Return-Path: X-Original-To: cvs-all@freebsd.org Delivered-To: cvs-all@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id 1929716A412; Wed, 29 Nov 2006 10:58:10 +0000 (UTC) (envelope-from iedowse@iedowse.com) Received: from nowhere.iedowse.com (nowhere.iedowse.com [82.195.144.75]) by mx1.FreeBSD.org (Postfix) with SMTP id D583D43CA5; Wed, 29 Nov 2006 10:58:07 +0000 (GMT) (envelope-from iedowse@iedowse.com) Received: from localhost ([127.0.0.1] helo=iedowse.com) by nowhere.iedowse.com via local-iedowse id ; 29 Nov 2006 10:58:06 +0000 (GMT) To: "Daan Vreeken [PA4DAN]" In-Reply-To: Your message of "Tue, 28 Nov 2006 14:53:26 +0100." <200611281453.26820.Danovitsch@vitsch.net> Date: Wed, 29 Nov 2006 10:58:04 +0000 From: Ian Dowse Message-ID: <200611291058.aa80677@nowhere.iedowse.com> Cc: scottl@samsco.org, src-committers@freebsd.org, cvs-src@freebsd.org, cvs-all@freebsd.org, "M. Warner Losh" Subject: Re: cvs commit: src/sys/dev/usb usbdi.c X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 29 Nov 2006 10:58:10 -0000 In message <200611281453.26820.Danovitsch@vitsch.net>, "Daan Vreeken [PA4DAN]" writes: >I don't believe my patch is at the right place where it's placed now. Any >ideas from someone with more bus_dma-foo and/or usb-foo are highly >appreciated :) >If more debugging information is needed, just ask. Hi Daan, Thanks for all the details - I'll try to put together a patch in the next few days that adds bus_dmamap_sync() calls whereever there are shared access ordering requirements in the host controller drivers. As Olivier mentioned, bus_dma(9) says that this should be done even for BUS_DMA_COHERENT allocations, so adding them may fix problems on other platforms too. BTW it looks like some of the packet descriptor structures in the HC drivers currently throw away their bus_dma information, since the strutures are never freed, so adding the bus_dmamap_sync() calls will involve keeping track of them now. Ian