FreeBSD source tree https://cgit-dev.freebsd.org/src/atom?h=releng%2F14.3 2024-01-02T00:29:56Z 2024-01-02T00:29:56Z Mark Johnston markj@FreeBSD.org 2023-12-26T01:42:58Z urn:sha1:494fe2e050a69455f33f76a365bd884f3a32842b Try to copy out output values before handling errors, and check that we did so successfully. In particular, it doesn't seem sensible to ignore errors here, otherwise userspace won't have any way to refer to the allocations. This is in preparation for annotating copyin() and related functions with __result_use_check. Reviewed by: royger MFC after: 1 week Differential Revision: https://reviews.freebsd.org/D43145 (cherry picked from commit 6cdff09c0d70f780a738dbd3d87deb3b13ec8446) 2023-08-16T17:54:36Z Warner Losh imp@FreeBSD.org 2023-08-16T17:54:36Z urn:sha1:685dc743dc3b5645e34836464128e1c0558b404b Remove /^[\s*]*__FBSDID\("\$FreeBSD\$"\);?\s*\n/ 2022-05-06T22:46:58Z John Baldwin jhb@FreeBSD.org 2022-05-06T22:46:58Z urn:sha1:f929eb1ed50f079f1638cdabe511ac2e347f541a 2021-10-20T01:22:56Z Mark Johnston markj@FreeBSD.org 2021-10-20T00:23:39Z urn:sha1:a4667e09e6520dc2c4b0b988051f060fed695a91 Remove page zeroing code from consumers and stop specifying VM_ALLOC_NOOBJ. In a few places, also convert an allocation loop to simply use VM_ALLOC_WAITOK. Similarly, convert vm_page_alloc_domain() callers. Note that callers are now responsible for assigning the pindex. Reviewed by: alc, hselasky, kib MFC after: 1 week Sponsored by: The FreeBSD Foundation Differential Revision: https://reviews.freebsd.org/D31986 2020-09-08T23:38:49Z Konstantin Belousov kib@FreeBSD.org 2020-09-08T23:38:49Z urn:sha1:fbf2a778760f24565eab950ced3e2ad1df4cf844 Future changes would require additional initialization of OBJT_PHYS objects, and vm_object_allocate() is not suitable for it. Reviewed by: markj Tested by: pho Sponsored by: The FreeBSD Foundation MFC after: 1 week Differential revision: https://reviews.freebsd.org/D24652 2020-09-01T21:45:08Z Mateusz Guzik mjg@FreeBSD.org 2020-09-01T21:45:08Z urn:sha1:6c7cae4a73b883ec0410e882bb56622c3090592c 2019-12-22T06:56:44Z Jeff Roberson jeff@FreeBSD.org 2019-12-22T06:56:44Z urn:sha1:3cf3b4e6419d14ed25c0b67b84763b3be937c8b5 removed from objects including calls to free. Pages must not be xbusy when freed and not on an object. Strengthen assertions to match these expectations. In practice very little code had to change busy handling to meet these rules but we can now make stronger guarantees to busy holders and avoid conditionally dropping busy in free. Refine vm_page_remove() and vm_page_replace() semantics now that we have stronger guarantees about busy state. This removes redundant and potentially problematic code that has proliferated. Discussed with: markj Reviewed by: kib Differential Revision: https://reviews.freebsd.org/D22822 2019-10-15T03:45:41Z Jeff Roberson jeff@FreeBSD.org 2019-10-15T03:45:41Z urn:sha1:0012f373e43db2341c20329163ed2d5ad3b0f341 Atomics are used for page busy and valid state when the shared busy is held. The details of the locking protocol and valid and dirty synchronization are in the updated vm_page.h comments. Reviewed by: kib, markj Tested by: pho Sponsored by: Netflix, Intel Differential Revision: https://reviews.freebsd.org/D21594 2019-10-15T03:35:11Z Jeff Roberson jeff@FreeBSD.org 2019-10-15T03:35:11Z urn:sha1:63e9755548e4feebf798686ab8bce0cdaaaf7b46 This is the first in a series of patches that promotes the page busy field to a first class lock that no longer requires the object lock for consistency. Reviewed by: kib, markj Tested by: pho Sponsored by: Netflix, Intel Differential Revision: https://reviews.freebsd.org/D21548 2019-09-10T19:08:01Z Jeff Roberson jeff@FreeBSD.org 2019-09-10T19:08:01Z urn:sha1:c75757481f831d2b28741c7f4f62176a76444c2e - VM_ALLOC_NOCREAT will grab without creating a page. - vm_page_grab_valid() will grab and page in if necessary. - vm_page_busy_acquire() automates some busy acquire loops. Discussed with: alc, kib, markj Tested by: pho (part of larger branch) Sponsored by: Netflix Differential Revision: https://reviews.freebsd.org/D21546