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>