From nobody Mon Jun 5 16:22:32 2023 X-Original-To: dev-commits-src-all@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4QZf5j0MrSz4ZQtT; Mon, 5 Jun 2023 16:22:33 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mxrelay.nyi.freebsd.org", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4QZf5h6jj5z4PGl; Mon, 5 Jun 2023 16:22:32 +0000 (UTC) (envelope-from git@FreeBSD.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1685982152; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=r/VIe/sOmcy3SSrwwbaGzYOhfzWRvsZKVp4gamkF9kE=; b=Oc3fs6txO4L9PoPqhoYqtvWkMSCM4iIQsCSw8DygUQZYiap1mnMAZ+X1g+GukSWugkbo3V +KO7+Mvzobn/Hij9Qdxk5neFB7EFJP37j9CzQdwAIXqwcgHkFeSzIKT6b5SDXTeDkDrWQA m55MugSvvec2DmyzE6xXexgtle0LnrNEfO4DAvwXewRdRLmul2MdhOsKWszWfV6KP+Le/s 8RJTfj/sLr0SBmiebPBl/5eMEbY0qhtlDC0j1Ej3BTpSRq41MCr5hXU0H0jTLFbgtpmiyp tciVOtkS1djfXguJTQaS04umMXjDQoTPzx1K04BM6nXTyURil7/9p8cJIx7rdQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1685982152; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=r/VIe/sOmcy3SSrwwbaGzYOhfzWRvsZKVp4gamkF9kE=; b=YuBjHcxwkjKPeRcmP/KOQa+iz/UaTLcNrPnAaAiVs72FNlok720PvzeXj34f1L3f0yrB2d xUl8YkU0ho/XczxFEg1HwInWyKH6BjTxNTg8Xery3AGe8rhmCAZDvEBd/rYWJfu70uvKfz cR2auMa5Kvt2HrT7kka2d6/mzgS7Vt4d2LmCJ0mShsUzQEWrZiah0Qu58zaeU8P1nTipzx C/Yc13BdbV4Mg/Vs6u7LKxwlIzj3ZurwAT3ngVAuseaUFB9or15rC+rGHQiS+zriBpD17P ll0mtu0PV3fC95e3kgsR+pRgs9uQquZR/+tzhn33/kLmDszQg1tpXTlrpkU6pg== ARC-Authentication-Results: i=1; mx1.freebsd.org; none ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1685982152; a=rsa-sha256; cv=none; b=LACfT90vYu1wqP5AAQYGAzwPKmMMKG77LloTGCDygrgfJrz6SC/CQCmPUZD3LUVsynANGl 4CAiuzifSoInJjYVJ4bl46GBUFTDgZ/2u/VyAXh6lwF3xS6SWYxVX8f6z28J9srfiJPV4q Hzc57xkKpKlgvizvI8dH0b02SV8ejcA12sqkzTgQxEGqPaijqMLtEpAFoLrJt3aSHVLrt+ rWcMyH/FMf5i2v8GXwy8gBraRW1/ia26yUPFShf/pGTLr62nbLVeFj1xBA5bPJMslt5NlO D1Z7+5sgsUZIjIPAcXDPl8/xF2b0ym5GEix6+5lfYTOWPXEm62Zk0PvJsoRCKg== Received: from gitrepo.freebsd.org (gitrepo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:5]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) by mxrelay.nyi.freebsd.org (Postfix) with ESMTPS id 4QZf5h5nLbzvJ1; Mon, 5 Jun 2023 16:22:32 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from gitrepo.freebsd.org ([127.0.1.44]) by gitrepo.freebsd.org (8.16.1/8.16.1) with ESMTP id 355GMWeB000110; Mon, 5 Jun 2023 16:22:32 GMT (envelope-from git@gitrepo.freebsd.org) Received: (from git@localhost) by gitrepo.freebsd.org (8.16.1/8.16.1/Submit) id 355GMW7J000109; Mon, 5 Jun 2023 16:22:32 GMT (envelope-from git) Date: Mon, 5 Jun 2023 16:22:32 GMT Message-Id: <202306051622.355GMW7J000109@gitrepo.freebsd.org> To: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org From: Mark Johnston Subject: git: 6062d9faf23f - main - vm_phys: Change the return type of vm_phys_unfree_page() to bool List-Id: Commit messages for all branches of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-all List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-src-all@freebsd.org X-BeenThere: dev-commits-src-all@freebsd.org MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Git-Committer: markj X-Git-Repository: src X-Git-Refname: refs/heads/main X-Git-Reftype: branch X-Git-Commit: 6062d9faf23f11e0655f3a222c204725e3111fe8 Auto-Submitted: auto-generated X-ThisMailContainsUnwantedMimeParts: N The branch main has been updated by markj: URL: https://cgit.FreeBSD.org/src/commit/?id=6062d9faf23f11e0655f3a222c204725e3111fe8 commit 6062d9faf23f11e0655f3a222c204725e3111fe8 Author: Mark Johnston AuthorDate: 2023-06-05 14:40:15 +0000 Commit: Mark Johnston CommitDate: 2023-06-05 16:22:11 +0000 vm_phys: Change the return type of vm_phys_unfree_page() to bool This is in keeping with the trend of removing uses of boolean_t, and the sole caller was implicitly converting it to a "bool". No functional change intended. Reviewed by: dougm, alc, imp, kib MFC after: 2 weeks Differential Revision: https://reviews.freebsd.org/D40401 --- sys/vm/vm_page.c | 8 ++++---- sys/vm/vm_phys.c | 14 +++++++------- sys/vm/vm_phys.h | 2 +- 3 files changed, 12 insertions(+), 12 deletions(-) diff --git a/sys/vm/vm_page.c b/sys/vm/vm_page.c index 4b967a94aa1f..5d822d34ed7c 100644 --- a/sys/vm/vm_page.c +++ b/sys/vm/vm_page.c @@ -327,7 +327,7 @@ vm_page_blacklist_add(vm_paddr_t pa, bool verbose) { struct vm_domain *vmd; vm_page_t m; - int ret; + bool found; m = vm_phys_paddr_to_vm_page(pa); if (m == NULL) @@ -335,15 +335,15 @@ vm_page_blacklist_add(vm_paddr_t pa, bool verbose) vmd = vm_pagequeue_domain(m); vm_domain_free_lock(vmd); - ret = vm_phys_unfree_page(m); + found = vm_phys_unfree_page(m); vm_domain_free_unlock(vmd); - if (ret != 0) { + if (found) { vm_domain_freecnt_inc(vmd, -1); TAILQ_INSERT_TAIL(&blacklist_head, m, listq); if (verbose) printf("Skipping page with pa 0x%jx\n", (uintmax_t)pa); } - return (ret); + return (found); } /* diff --git a/sys/vm/vm_phys.c b/sys/vm/vm_phys.c index 42875d12ba81..8db6529b8c80 100644 --- a/sys/vm/vm_phys.c +++ b/sys/vm/vm_phys.c @@ -1291,12 +1291,12 @@ vm_phys_scan_contig(int domain, u_long npages, vm_paddr_t low, vm_paddr_t high, /* * Search for the given physical page "m" in the free lists. If the search - * succeeds, remove "m" from the free lists and return TRUE. Otherwise, return - * FALSE, indicating that "m" is not in the free lists. + * succeeds, remove "m" from the free lists and return true. Otherwise, return + * false, indicating that "m" is not in the free lists. * * The free page queues must be locked. */ -boolean_t +bool vm_phys_unfree_page(vm_page_t m) { struct vm_freelist *fl; @@ -1319,12 +1319,12 @@ vm_phys_unfree_page(vm_page_t m) if (pa >= seg->start) m_set = &seg->first_page[atop(pa - seg->start)]; else - return (FALSE); + return (false); } if (m_set->order < order) - return (FALSE); + return (false); if (m_set->order == VM_NFREEORDER) - return (FALSE); + return (false); KASSERT(m_set->order < VM_NFREEORDER, ("vm_phys_unfree_page: page %p has unexpected order %d", m_set, m_set->order)); @@ -1350,7 +1350,7 @@ vm_phys_unfree_page(vm_page_t m) vm_freelist_add(fl, m_tmp, order, 0); } KASSERT(m_set == m, ("vm_phys_unfree_page: fatal inconsistency")); - return (TRUE); + return (true); } /* diff --git a/sys/vm/vm_phys.h b/sys/vm/vm_phys.h index c1f6514e92e4..a294bbaad80a 100644 --- a/sys/vm/vm_phys.h +++ b/sys/vm/vm_phys.h @@ -79,7 +79,7 @@ void vm_phys_register_domains(int ndomains, struct mem_affinity *affinity, int *locality); vm_page_t vm_phys_scan_contig(int domain, u_long npages, vm_paddr_t low, vm_paddr_t high, u_long alignment, vm_paddr_t boundary, int options); -boolean_t vm_phys_unfree_page(vm_page_t m); +bool vm_phys_unfree_page(vm_page_t m); int vm_phys_mem_affinity(int f, int t); void vm_phys_early_add_seg(vm_paddr_t start, vm_paddr_t end); vm_paddr_t vm_phys_early_alloc(int domain, size_t alloc_size);