Date: Thu, 29 Oct 2015 11:32:52 -0700 From: Conrad Meyer <cem@FreeBSD.org> To: John Baldwin <jhb@freebsd.org> Cc: Steven Wahl <Steve_Wahl@dell.com>, "svn-src-all@freebsd.org" <svn-src-all@freebsd.org>, "kib@freebsd.org" <kib@freebsd.org>, Eric Van Gyzen <Eric.VanGyzen@compellent.com>, "?Alan L. Cox" <alc@freebsd.org> Subject: Re: svn commit: r290130 - head/sys/dev/ntb/ntb_hw Message-ID: <CAG6CVpW=55Cat4jpXdyfPtrpphSgXB3Pcnr%2BMun9v5nWXp9Q-Q@mail.gmail.com> In-Reply-To: <1968754.iCngWsIWpR@ralph.baldwin.cx> References: <bbf3208d2a4d40998c2154402208c016@mspexmb1.Beer.Town> <1968754.iCngWsIWpR@ralph.baldwin.cx>
next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, Oct 29, 2015 at 10:02 AM, John Baldwin <jhb@freebsd.org> wrote: > On Thursday, October 29, 2015 04:42:20 PM Steven Wahl wrote: >> We ran into this exact problem, pmap_change_attr not working right with large bars. I had been working up to seeing if this compiles on the current head, introducing myself to the community, seeing if this would be accepted. >> >> But looks like it's needed sooner, so in case it might save you some time, here's the patch we developed for this problem. >> >> --> Steve Wahl, Dell Compellent, Eden Prairie, MN Thanks! > Adding alc@, but on first blush this looks correct to me. Looks good to me, too. I've thrown it up on phabricator: https://reviews.freebsd.org/D4030 . Best, Conrad
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAG6CVpW=55Cat4jpXdyfPtrpphSgXB3Pcnr%2BMun9v5nWXp9Q-Q>