git: 55c57a7811ec - main - rman: Remove an outdated comment that no longer applies

Jessica Clarke jrtc27 at FreeBSD.org
Mon Jul 5 15:17:53 UTC 2021


The branch main has been updated by jrtc27:

URL: https://cgit.FreeBSD.org/src/commit/?id=55c57a7811ec3c2dfcbfd69216a8eaad4e9be8bc

commit 55c57a7811ec3c2dfcbfd69216a8eaad4e9be8bc
Author:     Jessica Clarke <jrtc27 at FreeBSD.org>
AuthorDate: 2021-07-05 15:15:03 +0000
Commit:     Jessica Clarke <jrtc27 at FreeBSD.org>
CommitDate: 2021-07-05 15:15:03 +0000

    rman: Remove an outdated comment that no longer applies
    
    Since commit 2dd1bdf1834c in 2016 the r_start and r_end fields have been
    rman_res_t, which was briefly unsigned long, but commit da1b038af9f9
    changed the typedef to be uintmax_t instead. C99 is also something we
    assume these days.
    
    Reviewed by:    imp
    Differential Revision:  https://reviews.freebsd.org/D30808
---
 sys/kern/subr_rman.c | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/sys/kern/subr_rman.c b/sys/kern/subr_rman.c
index a76dbf020ee3..a8f5188e7f54 100644
--- a/sys/kern/subr_rman.c
+++ b/sys/kern/subr_rman.c
@@ -79,11 +79,7 @@ __FBSDID("$FreeBSD$");
 /*
  * We use a linked list rather than a bitmap because we need to be able to
  * represent potentially huge objects (like all of a processor's physical
- * address space).  That is also why the indices are defined to have type
- * `unsigned long' -- that being the largest integral type in ISO C (1990).
- * The 1999 version of C allows `long long'; we may need to switch to that
- * at some point in the future, particularly if we want to support 36-bit
- * addresses on IA32 hardware.
+ * address space).
  */
 struct resource_i {
 	struct resource		r_r;


More information about the dev-commits-src-all mailing list