Date: Fri, 4 Aug 2006 01:25:14 +0000 From: John Birrell <jb@what-creek.com> To: pdeuskar@FreeBSD.org Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org, cperciva@freebsd.org Subject: Re: cvs commit: src/sys/dev/em if_em.c if_em.h Message-ID: <20060804012514.GA63379@what-creek.com> In-Reply-To: <44D29FB6.9090209@freebsd.org> References: <200608031905.k73J55D2061870@repoman.freebsd.org> <44D29FB6.9090209@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, Aug 03, 2006 at 06:15:34PM -0700, Colin Percival wrote: > Prafulla Deuskar wrote: > > Modified files: > > sys/dev/em if_em.c if_em.h > > Log: > > Revert back changes to made in rev 1.109 of if_em.c which were unnecessary. > > This makes it easier for us to get the changes into -current and to -stable quickly. > > I assume you've discussed this with glebius, who made the original changes; in > the future, please add an "Approved by:", "No objections from:", or "Discussed > with:" line to backout commits to make this more clear. Also Pyun had made a fix for the bus_dmamap_unload problem we see on sun4v. This appears to have been backed out amongst what appears to be largely a white-space change. Are you prepared to do the work to solve the bus_dmamap_unload again? -- John Birrell
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20060804012514.GA63379>