svn commit: r347890 - in head/sys/dev: aac bge

Peter Jeremy peter at rulingia.com
Fri May 17 05:36:54 UTC 2019


On 2019-May-16 20:41:29 +0000, Tycho Nightingale <tychon at FreeBSD.org> wrote:
>Author: tychon
>Date: Thu May 16 20:41:28 2019
>New Revision: 347890
>URL: https://svnweb.freebsd.org/changeset/base/347890
>
>Log:
>  reinstate 4GB DMA boundary workarounds for bge and aac
>  
>  Reviewed by:	kib
>  Sponsored by:	Dell EMC Isilon
>  Differential Revision:	https://reviews.freebsd.org/D20277

My main concern with this commit is that there is nothing in the commit
message explaining why this has been done.  D20277 implies that this is
effectively reverting a couple of other commits because these devices
have 64-bit issues - the commit log should document this.

-- 
Peter Jeremy
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 963 bytes
Desc: not available
URL: <http://lists.freebsd.org/pipermail/svn-src-all/attachments/20190517/8e3e52f6/attachment.sig>


More information about the svn-src-all mailing list