src-test/sys/geom/raid/g_raid.c, branch main FreeBSD source tree https://cgit-dev.freebsd.org/src-test/atom?h=main 2020-10-18T16:24:08Z Make g_attach() return ENXIO for orphaned providers; update various 2020-10-18T16:24:08Z Edward Tomasz Napierala trasz@FreeBSD.org 2020-10-18T16:24:08Z urn:sha1:d22ff249d9733ed0ff628efd797f3ea3ec66c913 classes to add missing error checking. Reviewed by: imp MFC after: 2 weeks Sponsored by: NetApp, Inc. Sponsored by: Klara, Inc. Differential Revision: https://reviews.freebsd.org/D26658 sys/geom: consistently use _PATH_DEV instead of hardcoding "/dev/". 2020-07-09T02:52:39Z Xin LI delphij@FreeBSD.org 2020-07-09T02:52:39Z urn:sha1:8510f61acdf83ef524e694b9b0eebeb7ad834162 Reviewed by: cem MFC after: 2 weeks Differential Revision: https://reviews.freebsd.org/D25565 Mark more nodes as CTLFLAG_MPSAFE or CTLFLAG_NEEDGIANT (17 of many) 2020-02-26T14:26:36Z Pawel Biernacki kaktus@FreeBSD.org 2020-02-26T14:26:36Z urn: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 Pass BIO_SPEEDUP through all the geom layers 2020-01-17T01:15:55Z Warner Losh imp@FreeBSD.org 2020-01-17T01:15:55Z urn:sha1:8b522bdae65ca1b2e5db20e9aeee0b5acd89e7ed While some geom layers pass unknown commands down, not all do. For the ones that don't, pass BIO_SPEEDUP down to the providers that constittue the geom, as applicable. No changes to vinum or virstor because I was unsure how to add this support, and I'm also unsure how to test these. gvinum doesn't implement BIO_FLUSH either, so it may just be poorly maintained. gvirstor is for testing and not supportig BIO_SPEEDUP is fine. Reviewed by: chs Differential Revision: https://reviews.freebsd.org/D23183 GEOM: Reduce unnecessary log interleaving with sbufs 2019-08-07T19:28:35Z Conrad Meyer cem@FreeBSD.org 2019-08-07T19:28:35Z urn:sha1:ac03832ef3afa26cc8e23bf5cfeaa32501b21c3e Similar to what was done for device_printfs in r347229. Convert g_print_bio() to a thin shim around g_format_bio(), which acts on an sbuf; documented in g_bio.9. Reviewed by: markj Discussed with: rlibby Sponsored by: Dell EMC Isilon Differential Revision: https://reviews.freebsd.org/D21165 Use sbuf_cat() in GEOM confxml generation. 2019-06-19T15:36:02Z Alexander Motin mav@FreeBSD.org 2019-06-19T15:36:02Z urn:sha1:49ee0fcea52c43f055ce8d63bc6f72dbb91c509d When it comes to megabytes of text, difference between sbuf_printf() and sbuf_cat() becomes substantial. MFC after: 2 weeks Sponsored by: iXsystems, Inc. Use g_handleattr() to reply to GEOM::candelete queries. 2019-01-02T15:52:16Z Mark Johnston markj@FreeBSD.org 2019-01-02T15:52:16Z urn:sha1:438622af062bc61836f46c138292ceab68b335b3 g_handleattr() fills out bp->bio_completed; otherwise, g_getattr() returns an error in response to the query. This caused BIO_DELETE support to not be propagated through stacked configurations, e.g., a gconcat of gmirror volumes would not handle BIO_DELETE even when the gmirrors do. g_io_getattr() was not affected by the problem. PR: 232676 Reported and tested by: noah.bergbauer@tum.de MFC after: 1 week Extend stripeoffset and stripesize of GEOMs from u_int to off_t 2018-10-27T16:14:42Z Eugene Grosbein eugen@FreeBSD.org 2018-10-27T16:14:42Z urn:sha1:6d305ab0b289686aeab9774cae15ef1e2da5c7be GEOM's stripeoffset overflows at 4 gigabyte margin (2^32) because of its u_int type. This leads to incorrect data in the output generated by "sysctl kern.geom.confxml" command, "graid list" etc. when GEOM array has volumes larger than 4G, for example. This change does not affect ABI but changes KBI. No MFC planned. Differential Revision: https://reviews.freebsd.org/D13426 Do pass removing some write-only variables from the kernel. 2017-12-25T04:48:39Z Alexander Kabaev kan@FreeBSD.org 2017-12-25T04:48:39Z urn:sha1:151ba7933aab5c8edc094715bffd9b97d84d2407 This reduces noise when kernel is compiled by newer GCC versions, such as one used by external toolchain ports. Reviewed by: kib, andrew(sys/arm and sys/arm64), emaste(partial), erj(partial) Reviewed by: jhb (sys/dev/pci/* sys/kern/vfs_aio.c and sys/kern/kern_synch.c) Differential Revision: https://reviews.freebsd.org/D10385 sys/geom: adoption of SPDX licensing ID tags. 2017-11-27T15:17:37Z Pedro F. Giffuni pfg@FreeBSD.org 2017-11-27T15:17:37Z urn:sha1:3728855a0f66545c4aa90dc0d478dc403bdfa667 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.