Date: Fri, 20 Jul 2007 23:10:08 GMT From: dfilter@FreeBSD.ORG (dfilter service) To: freebsd-amd64@FreeBSD.org Subject: Re: amd64/89550: commit references a PR Message-ID: <200707202310.l6KNA82v025309@freefall.freebsd.org>
next in thread | raw e-mail | index | archive | help
The following reply was made to PR amd64/89550; it has been noted by GNATS.
From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:
Subject: Re: amd64/89550: commit references a PR
Date: Fri, 20 Jul 2007 23:02:24 +0000 (UTC)
se 2007-07-20 23:02:01 UTC
FreeBSD src repository
Modified files:
sys/dev/sym sym_hipd.c
Log:
Fix Symbios driver on amd64: Since amd64 has 64 bit pointers but the same
4KB pages as i386, data structures that just fit in one page on i386 (and
on 64 bit architectures with 8KB pages) can be distributed over two pages
on amd64. This is a porblem in the case of the Symbios driver, since the
SCRIPTS engine in the SCSI chip operates on physical addresses and needs
physically contiguous memory. Earlier patches used contigmalloc on amd64,
but this version replaces part of a structure by a pointer to that data.
In order to not introduce an extra indirection for other architectures,
the change has been made conditional on __amd64__.
Earlier attempts to repair this problem are removed (i.e. the macros that
made amd64 use contigmalloc). The fix was submitted by Jan Mikkelsen and
modified by me to only affect amd64.
PR: 89550
Submitted by: janm at transactionware dot com (Jan Mikkelsen)
Approved by: re (Hiroki Sato)
MFC after: 2 weeks
Revision Changes Path
1.67 +17 -12 src/sys/dev/sym/sym_hipd.c
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200707202310.l6KNA82v025309>