src/sys/dev/agp, branch releng/12.4 FreeBSD source tree https://cgit-dev.freebsd.org/src/atom?h=releng%2F12.4 2022-07-19T18:02:00Z Adjust agp_find_device() definition in agp.c to avoid clang 15 warning 2022-07-19T18:02:00Z Dimitry Andric dim@FreeBSD.org 2022-07-16T14:46:21Z urn:sha1:381951ce6dc8ab3b13f39757ecbce41b1fea5e53 With clang 15, the following -Werror warning is produced: sys/dev/agp/agp.c:910:16: error: a function declaration without a prototype is deprecated in all versions of C [-Werror,-Wstrict-prototypes] agp_find_device() ^ void This is because agp_find_device() is declared with a (void) argument list, and defined with an empty argument list. Make the definition match the declaration. MFC after: 3 days (cherry picked from commit c0817e2aba26ceb683e9c649c22d7fcd9a921231) Eliminate the arena parameter to kmem_free(). Implicitly this corrects an 2018-08-25T19:38:08Z Alan Cox alc@FreeBSD.org 2018-08-25T19:38:08Z urn:sha1:49bfa624ac861d7fafc87f06dc2cdf4972aed854 error in the function hypercall_memfree(), where the wrong arena was being passed to kmem_free(). Introduce a per-page flag, VPO_KMEM_EXEC, to mark physical pages that are mapped in kmem with execute permissions. Use this flag to determine which arena the kmem virtual addresses are returned to. Eliminate UMA_SLAB_KRWX. The introduction of VPO_KMEM_EXEC makes it redundant. Update the nearby comment for UMA_SLAB_KERNEL. Reviewed by: kib, markj Discussed with: jeff Approved by: re (marius) Differential Revision: https://reviews.freebsd.org/D16845 Eliminate kmem_alloc_contig()'s unused arena parameter. 2018-08-20T15:57:27Z Alan Cox alc@FreeBSD.org 2018-08-20T15:57:27Z urn:sha1:44d0efb2154d884c85919229f28b31d50d25a350 Reviewed by: hselasky, kib, markj Discussed with: jeff Differential Revision: https://reviews.freebsd.org/D16799 Oops. r338030 didn't eliminate the unused arena argument from all of 2018-08-18T22:35:19Z Alan Cox alc@FreeBSD.org 2018-08-18T22:35:19Z urn:sha1:94d0f0877db6bf0511dc0198e78b870618f42d3e kmem_alloc_attr()'s callers. Correct that mistake. sys/dev: further adoption of SPDX licensing ID tags. 2017-11-27T14:52:40Z Pedro F. Giffuni pfg@FreeBSD.org 2017-11-27T14:52:40Z urn: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. Add a helper function to agp(4) which installs a single GTT entry. 2017-07-09T22:53:24Z Mark Johnston markj@FreeBSD.org 2017-07-09T22:53:24Z urn:sha1:8ee96d1452c6b93767920a59035d4cb66a6a6b68 Submitted by: dumbbell MFC after: 1 week sys/dev: Replace zero with NULL for pointers. 2017-02-20T03:43:12Z Pedro F. Giffuni pfg@FreeBSD.org 2017-02-20T03:43:12Z urn: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 Move bogus_page declaration to vm_page.h and initialization to vm_page.c. 2017-01-04T22:27:19Z Gleb Smirnoff glebius@FreeBSD.org 2017-01-04T22:27:19Z urn:sha1:bfc8c24c7350da9093271263d46897862b259068 Reviewed by: kib sys: use our roundup2/rounddown2() macros when param.h is available. 2016-04-21T19:57:40Z Pedro F. Giffuni pfg@FreeBSD.org 2016-04-21T19:57:40Z urn:sha1:d9c9c81c083a76a65c6cacf8fcbc0511e2ffa489 rounddown2 tends to produce longer lines than the original code and when the code has a high indentation level it was not really advantageous to do the replacement. This tries to strike a balance between readability using the macros and flexibility of having the expressions, so not everything is converted. dev/agp: use our nitems() macro when it is avaliable through param.h. 2016-04-19T23:31:35Z Pedro F. Giffuni pfg@FreeBSD.org 2016-04-19T23:31:35Z urn:sha1:4ec642f1d61bbab5998f7b19281396ee925f82f0 Consistently capitalize the macros used in the driver. No functional change.