Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 2 Oct 2003 09:58:10 +1000
From:      Peter Jeremy <peter.jeremy@alcatel.com.au>
To:        Murray Stokely <murray@freebsd.org>, stable@freebsd.org, qa@freebsd.org
Subject:   Re: FreeBSD 4.9 RC1 (i386) now available
Message-ID:  <20031001235809.GY91404@gsmx07.alcatel.com.au>
In-Reply-To: <20030929160905.GB42388@falcon.midgard.homeip.net>
References:  <20030929151905.GD3743@freebsdmall.com> <20030929160905.GB42388@falcon.midgard.homeip.net>

next in thread | previous in thread | raw e-mail | index | archive | help
On 2003-Sep-29 18:09:05 +0200, Erik Trulsson <ertr1013@student.uu.se> wrote:
>On Mon, Sep 29, 2003 at 08:19:05AM -0700, Murray Stokely wrote:
>> Not all FTP sites have the first release candidate, but it is at least
>> available from ftp.freebsd.org.  Please download and install this
>> candidate and help us find bugs BEFORE we call it 4.9-RELEASE.
>
><sarcasm>
>You mean apart from the minor bug that non-pci kernels using ata won't even
>compile, and hasn't worked for the last three weeks or so?
></sarcasm>
>
>See
>
>http://docs.freebsd.org/cgi/getmsg.cgi?fetch=44649+0+archive/2003/freebsd-stable/20030928.freebsd-stable
>
>or
>
>http://docs.freebsd.org/cgi/getmsg.cgi?fetch=816008+0+archive/2003/cvs-all/20030921.cvs-all
>
>for details and a patch.

Whilst both Erik and I independently came up with the same patch, upon
reflection, I'm not sure that this is the correct patch.  None of the
callers to ata_dmastart() check for a return value and therefore this
probably should be a void function - so the code in ata-isa.c is
correct and the remaining declaration and definitions are incorrect.

In either case, I would request that this be fixed before 4.9-RELEASE.

Peter



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20031001235809.GY91404>