src-test/sys/dev/ata, branch mainFreeBSD source treehttps://cgit-dev.freebsd.org/src-test/atom?h=main2020-11-28T12:12:51ZMake MAXPHYS tunable. Bump MAXPHYS to 1M.2020-11-28T12:12:51ZKonstantin Belousovkib@FreeBSD.org2020-11-28T12:12:51Zurn:sha1:cd8537910406e68d4719136a5b0cf6d23bb1b23b
Replace MAXPHYS by runtime variable maxphys. It is initialized from
MAXPHYS by default, but can be also adjusted with the tunable kern.maxphys.
Make b_pages[] array in struct buf flexible. Size b_pages[] for buffer
cache buffers exactly to atop(maxbcachebuf) (currently it is sized to
atop(MAXPHYS)), and b_pages[] for pbufs is sized to atop(maxphys) + 1.
The +1 for pbufs allow several pbuf consumers, among them vmapbuf(),
to use unaligned buffers still sized to maxphys, esp. when such
buffers come from userspace (*). Overall, we save significant amount
of otherwise wasted memory in b_pages[] for buffer cache buffers,
while bumping MAXPHYS to desired high value.
Eliminate all direct uses of the MAXPHYS constant in kernel and driver
sources, except a place which initialize maxphys. Some random (and
arguably weird) uses of MAXPHYS, e.g. in linuxolator, are converted
straight. Some drivers, which use MAXPHYS to size embeded structures,
get private MAXPHYS-like constant; their convertion is out of scope
for this work.
Changes to cam/, dev/ahci, dev/ata, dev/mpr, dev/mpt, dev/mvs,
dev/siis, where either submitted by, or based on changes by mav.
Suggested by: mav (*)
Reviewed by: imp, mav, imp, mckusick, scottl (intermediate versions)
Tested by: pho
Sponsored by: The FreeBSD Foundation
Differential revision: https://reviews.freebsd.org/D27225
ata(4): Release the ioport resource if device initialization fails2020-11-24T17:13:08ZMark Johnstonmarkj@FreeBSD.org2020-11-24T17:13:08Zurn:sha1:938a9d7c4f8028c9eb03e0ec4e20a9d1e1109709
PR: 251346
Submitted by: janm@transactionware.com
MFC after: 1 week
ata: clean up empty lines in .c and .h files2020-09-01T21:40:47ZMateusz Guzikmjg@FreeBSD.org2020-09-01T21:40:47Zurn:sha1:1529c5096670b9c0d687a30a57f58c08a614ba01Tag pccard drivers with gone in 13.2020-08-20T17:19:40ZWarner Loshimp@FreeBSD.org2020-08-20T17:19:40Zurn:sha1:e0d14216c1124fccfe4500cc6d27ea91f459e83b
MFC After: 3 days
Reviewed by: emaste, brooks, adrian (on twitter)
Differential Revision: https://reviews.freebsd.org/D26095
Mark more nodes as CTLFLAG_MPSAFE or CTLFLAG_NEEDGIANT (17 of many)2020-02-26T14:26:36ZPawel Biernackikaktus@FreeBSD.org2020-02-26T14:26:36Zurn:sha1:7029da5c36f2d3cf6bb6c81bf551229f416399e8
r357614 added CTLFLAG_NEEDGIANT to make it easier to find nodes that are
still not MPSAFE (or already are but aren’t properly marked).
Use it in preparation for a general review of all nodes.
This is non-functional change that adds annotations to SYSCTL_NODE and
SYSCTL_PROC nodes using one of the soon-to-be-required flags.
Mark all obvious cases as MPSAFE. All entries that haven't been marked
as MPSAFE before are by default marked as NEEDGIANT
Approved by: kib (mentor, blanket)
Commented by: kib, gallatin, melifaro
Differential Revision: https://reviews.freebsd.org/D23718
Remove sparc64 kernel support2020-02-03T17:35:11ZWarner Loshimp@FreeBSD.org2020-02-03T17:35:11Zurn:sha1:58aa35d42975c298ca0adba705c042596303c9f5
Remove all sparc64 specific files
Remove all sparc64 ifdefs
Removee indireeect sparc64 ifdefs
Avoid a tautological bitwise compare.2019-12-17T00:10:19ZBrooks Davisbrooks@FreeBSD.org2019-12-17T00:10:19Zurn:sha1:e1fa71e579a0668dfc4165ab4f5edc85014dbeff
This looks like a bit of debugging code that sliped into the initial
import of the new ATA framework. This changes the behavior to omit a
line of output that appears to have been intended for omission.
Reviewed by: mav
MFC after: 3 days
Sponsored by: DARPA, AFRL
Differential Revision: https://reviews.freebsd.org/D22845
Use a void * argument to callout handlers instead of timeout_t casts.2019-12-05T18:47:29ZJohn Baldwinjhb@FreeBSD.org2019-12-05T18:47:29Zurn:sha1:65d2f9c12b22a774dda7e01577e0a8b88378a6ef
Reviewed by: kib
Differential Revision: https://reviews.freebsd.org/D22684
GC ATA_REQUEST_TIMEOUT option remnants2019-02-16T01:48:38ZAndriy Voskoboinykavos@FreeBSD.org2019-02-16T01:48:38Zurn:sha1:06da0ce08476454412372bd8d48a9947a395a1c5
It was removed from code in r249083 and from sys/conf/options in r249213.
PR: 222170
MFC after: 3 days
Reset indentiation of ata_suspend() such that its clear we fall through2018-07-03T16:17:59ZSean Brunosbruno@FreeBSD.org2018-07-03T16:17:59Zurn:sha1:a263a0427feaa595b512b174330b915a6c9aaaa3
this function and that we aren't supposed to be controlled by the first
if() conditional. Found with gcc.
No functional change is intended with this commit.