Date: Thu, 29 Oct 2015 06:39:59 -0700 From: John Baldwin <jhb@freebsd.org> To: "Conrad E. Meyer" <cem@freebsd.org> Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r290130 - head/sys/dev/ntb/ntb_hw Message-ID: <2379504.sY8Moxo3YJ@ralph.baldwin.cx> In-Reply-To: <201510290416.t9T4GSG7044279@repo.freebsd.org> References: <201510290416.t9T4GSG7044279@repo.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Thursday, October 29, 2015 04:16:28 AM Conrad E. Meyer wrote: > Author: cem > Date: Thu Oct 29 04:16:28 2015 > New Revision: 290130 > URL: https://svnweb.freebsd.org/changeset/base/290130 > > Log: > ntb: Do not attempt to set write-combining on MWs > > AMD64 pmap assumes ranges will be in the DMAP, which isn't necessarily > true for NTB memory windows (especially 64-bit BARs). > > Suggested by: pmap_change_attr_locked -> kassert_panic > Sponsored by: EMC / Isilon Storage Division Hmm, pmap_mapdev_attr() does the right thing, but there isn't a good way to call that directly (and still get a valid bus handle/tag). I have on my todo to add a bus_map_resource() that would let you allocate a bus handle/tag that 1) is a subset of a resource, and 2) specifies a memory attribute. It would allow you to do this sanely. -- John Baldwin
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?2379504.sY8Moxo3YJ>