Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 21 Feb 2003 00:36:51 +0100
From:      Maxime Henrion <mux@freebsd.org>
To:        Nate Lawson <nate@root.org>
Cc:        src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/pci if_xl.c
Message-ID:  <20030220233651.GQ60813@elvis.mu.org>
In-Reply-To: <Pine.BSF.4.21.0302201520200.52021-100000@root.org>
References:  <20030220212005.GP60813@elvis.mu.org> <Pine.BSF.4.21.0302201520200.52021-100000@root.org>

next in thread | previous in thread | raw e-mail | index | archive | help
Nate Lawson wrote:
> On Thu, 20 Feb 2003, Maxime Henrion wrote:
> > Nate Lawson wrote:
> > > On Wed, 19 Feb 2003, Maxime Henrion wrote:
> > > >   Modified files:
> > > >     sys/pci              if_xl.c 
> > > >   Log:
> > > >   Fix panic on sparc64 introduced in my last commit.  I really
> > > >   wish the busdma APIs were more consistent accross architectures.
> > > >   
> > > >   We should probably move all the other DMA map creations in
> > > >   xl_attach() where we can really handle them failing, since
> > > >   xl_init() is void and shouldn't fail.
> > > 
> > > Please see my patches for adding/fixing error handling for dmamap_load
> > > (posted to -current).  I am going to remove the unnecessary intr alloc
> > > move but otherwise they should be fine.
> > 
> > Where are those patches ?  I only found
> > http://www.root.org/~nate/freebsd/xl.diff, which is not related to
> > busdma at all.
> 
> You're right.  The fixes were for other drivers.

Ah, ok.  I'll take a look at these fixes as soon as possible (hopefully
tomorrow).

Cheers,
Maxime

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-src" in the body of the message




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