src/sys/dev/hptiop, branch releng/13.3FreeBSD source treehttps://cgit-dev.freebsd.org/src/atom?h=releng%2F13.32023-08-23T17:43:25Zsys: Remove $FreeBSD$: one-line .c pattern2023-08-23T17:43:25ZWarner Loshimp@FreeBSD.org2023-08-22T01:31:27Zurn:sha1:3bc80996974a61a4223eae4c1ccd47b6ee32a48a
Remove /^[\s*]*__FBSDID\("\$FreeBSD\$"\);?\s*\n/
Similar commit in current:
(cherry picked from commit 685dc743dc3b)
spdx: The BSD-2-Clause-FreeBSD identifier is obsolete, drop -FreeBSD2023-07-25T15:13:49ZWarner Loshimp@FreeBSD.org2023-05-10T15:40:58Zurn:sha1:caa41f641755c935b036e17440a3b49329c904ed
The SPDX folks have obsoleted the BSD-2-Clause-FreeBSD identifier. Catch
up to that fact and revert to their recommended match of BSD-2-Clause.
Discussed with: pfg
MFC After: 3 days
Sponsored by: Netflix
(cherry picked from commit 4d846d260e2b9a3d4d0a701462568268cbfe7a5b)
hptiop: Store softc pointer in si_drv1 of cdev.2022-05-19T21:31:53ZJohn Baldwinjhb@FreeBSD.org2022-04-21T17:29:13Zurn:sha1:b4600b13780847f8c9e4369ee1fa2bf7b8913399
Rather than fetching the softc using the controller's unit number as
an index into the devclass.
Reviewed by: imp
Differential Revision: https://reviews.freebsd.org/D34993
(cherry picked from commit 517a8a715a8aa94071c058491d22c464a7962f74)
hptiop: Remove Giant locking around ioctls.2021-12-24T01:05:03ZAlexander Motinmav@FreeBSD.org2021-12-10T01:35:21Zurn:sha1:93ce10ed26282edc9647be20ad72d62f855f4bd1
The methods have their own locking and don't require Giant.
MFC after: 2 weeks
(cherry picked from commit e06c67eef8faae3322c06056960562fde88fd7aa)
sys/dev: further adoption of SPDX licensing ID tags.2017-11-27T14:52:40ZPedro F. Giffunipfg@FreeBSD.org2017-11-27T14:52:40Zurn:sha1:718cf2ccb9956613756ab15d7a0e28f2c8e91cab
Mainly focus on files that use BSD 2-Clause license, however the tool I
was using misidentified many licenses so this was mostly a manual - error
prone - task.
The Software Package Data Exchange (SPDX) group provides a specification
to make it easier for automated tools to detect and summarize well known
opensource licenses. We are gradually adopting the specification, noting
that the tags are considered only advisory and do not, in any way,
superceed or replace the license texts.
sys/dev: Replace zero with NULL for pointers.2017-02-20T03:43:12ZPedro F. Giffunipfg@FreeBSD.org2017-02-20T03:43:12Zurn:sha1:4d24901ac9131267fd188c5e1ee19a219847238a
Makes things easier to read, plus architectures may set NULL to something
different than zero.
Found with: devel/coccinelle
MFC after: 3 weeks
Always null-terminate ccb_pathinq.(sim_vid|hba_vid|dev_name)2017-01-04T20:26:42ZAlan Somersasomers@FreeBSD.org2017-01-04T20:26:42Zurn:sha1:4195c7de2471d6212744bcbf128d659e94801d17
The sim_vid, hba_vid, and dev_name fields of struct ccb_pathinq are
fixed-length strings. AFAICT the only place they're read is in
sbin/camcontrol/camcontrol.c, which assumes they'll be null-terminated.
However, the kernel doesn't null-terminate them. A bunch of copy-pasted code
uses strncpy to write them, and doesn't guarantee null-termination. For at
least 4 drivers (mpr, mps, ciss, and hyperv), the hba_vid field actually
overflows. You can see the result by doing "camcontrol negotiate da0 -v".
This change null-terminates those fields everywhere they're set in the
kernel. It also shortens a few strings to ensure they'll fit within the
16-character field.
PR: 215474
Reported by: Coverity
CID: 1009997 1010000 1010001 1010002 1010003 1010004 1010005
CID: 1331519 1010006 1215097 1010007 1288967 1010008 1306000
CID: 1211924 1010009 1010010 1010011 1010012 1010013 1010014
CID: 1147190 1010017 1010016 1010018 1216435 1010020 1010021
CID: 1010022 1009666 1018185 1010023 1010025 1010026 1010027
CID: 1010028 1010029 1010030 1010031 1010033 1018186 1018187
CID: 1010035 1010036 1010042 1010041 1010040 1010039
Reviewed by: imp, sephe, slm
MFC after: 4 weeks
Sponsored by: Spectra Logic Corp
Differential Revision: https://reviews.freebsd.org/D9037
Differential Revision: https://reviews.freebsd.org/D9038
Use imin() macro from sys/libkern.h.2016-05-04T01:43:07ZMarcelo Araujoaraujo@FreeBSD.org2016-05-04T01:43:07Zurn:sha1:b3956646bf5c79d5ef5d878f8265c08468e87970
MFC after: 2 weeks.
etc: minor spelling fixes.2016-05-02T16:47:28ZPedro F. Giffunipfg@FreeBSD.org2016-05-02T16:47:28Zurn:sha1:b790c1938dd94d20de89e148169aab4ae3edd771
Mostly comments but also some user-visible strings.
MFC after: 2 weeks
Replace several bus_alloc_resource() calls using default arguments with bus_alloc_resource_any()2016-02-19T03:37:56ZJustin Hibbitsjhibbits@FreeBSD.org2016-02-19T03:37:56Zurn:sha1:43cd61606b6bfae52bb09856277751103bfa28fd
Since these calls only use default arguments, bus_alloc_resource_any() is the
right call.
Differential Revision: https://reviews.freebsd.org/D5306