From owner-cvs-all@FreeBSD.ORG Thu Nov 22 22:49:10 2007 Return-Path: Delivered-To: cvs-all@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 6946416A418; Thu, 22 Nov 2007 22:49:10 +0000 (UTC) (envelope-from sobomax@FreeBSD.org) Received: from sippysoft.com (gk.360sip.com [72.236.70.226]) by mx1.freebsd.org (Postfix) with ESMTP id 30C0E13C45B; Thu, 22 Nov 2007 22:49:09 +0000 (UTC) (envelope-from sobomax@FreeBSD.org) Received: from [192.168.0.3] ([204.244.149.125]) (authenticated bits=0) by sippysoft.com (8.13.8/8.13.8) with ESMTP id lAMMn3ew029830 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Thu, 22 Nov 2007 14:49:05 -0800 (PST) (envelope-from sobomax@FreeBSD.org) Message-ID: <4746075D.9000108@FreeBSD.org> Date: Thu, 22 Nov 2007 14:49:01 -0800 From: Maxim Sobolev Organization: Sippy Software, Inc. User-Agent: Thunderbird 2.0.0.9 (Windows/20071031) MIME-Version: 1.0 To: Robert Watson References: <200711212115.lALLF16n017990@repoman.freebsd.org> <4744DE58.9090005@FreeBSD.org> <4744EF09.4060009@FreeBSD.org> <474533A4.3030106@deepcore.dk> <20071122082847.N60495@fledge.watson.org> In-Reply-To: <20071122082847.N60495@fledge.watson.org> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 8bit Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org, =?ISO-8859-1?Q?S=F8ren_Schmidt?= , S?ren Schmidt Subject: Re: cvs commit: src/sys/dev/ata ata-all.h ata-chipset.c ata-dma.c ata-pci.c ata-pci.h atapi-cd.c atapi-fd.c atapi-tape.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: Thu, 22 Nov 2007 22:49:10 -0000 Robert Watson wrote: > On Thu, 22 Nov 2007, Søren Schmidt wrote: > >>>>> FreeBSD src repository >>>>> >>>>> Modified files: (Branch: RELENG_7) >>>>> sys/dev/ata ata-all.h ata-chipset.c ata-dma.c ata-pci.c >>>>> ata-pci.h atapi-cd.c atapi-fd.c atapi-tape.c Log: >>>>> Update with the latest fixes from -current. >>>> >>>> Please don't forget to merge at least atapi-cd.c fix into RELENG_6. >>>> Currently CD is not working in Parallels with 6.3-PRERELEASE. >>> >>> Also, I see some "ad0: TIMEOUT - WRITE_DMA (retrying)" messages with >>> Parallels using latest 6.3-PRERELEASE snapshot. Not sure if it's due >>> to some changes between 6.2 and 6.3, but I've never seen this with >>> 6.2. Any ideas? >> >> I have a complete backport of all the latest fixes done for 6.3 >> awaiting approval. >> >> I've seen such a timeout once on 6.3 before I merged the latest >> changes, since then I've not seen any, not that it means much though :) > > I see them any time I do anything involving significant disk load under > Parallels--I've always assumed it was because there are several layers > of indirection, including HFS+ and an expanding Parallels disk image, > beween the FreeBSD ATA driver and hardware, leading to significantly > longer delays than the ATA driver would expect from normal hardware. > That said, I'm fine with being wrong and seeing a bug fixed :-). I > usually get several each time I buildkernel, such as last night: > > ad0: TIMEOUT - READ_DMA retrying (1 retry left) LBA=5208191 > ad0: TIMEOUT - READ_DMA retrying (1 retry left) LBA=23670111 > ad0: TIMEOUT - READ_DMA retrying (1 retry left) LBA=5843715 > ad0: TIMEOUT - WRITE_DMA retrying (1 retry left) LBA=40407615 Assuming that there are no actual bugs in the code, should not it make sense to detect virtual hardware and treat it a little bit differently with respect to timeouts? Given the growing popularity of virtualization technology it's doesn't sound like a worthless project. -Maxim