diff options
| author | cem <cem@FreeBSD.org> | 2015-12-16 23:23:12 +0000 |
|---|---|---|
| committer | cem <cem@FreeBSD.org> | 2015-12-16 23:23:12 +0000 |
| commit | f64eb0a1eff489953daa256595df542b0f8aeffd (patch) | |
| tree | 4265e46b4e78640d58c18477bbc5be73a1961080 /sys | |
| parent | 0813e34dbc203efa23fb9ee80b8344b8cd776c67 (diff) | |
| download | FreeBSD-src-f64eb0a1eff489953daa256595df542b0f8aeffd.zip FreeBSD-src-f64eb0a1eff489953daa256595df542b0f8aeffd.tar.gz | |
vm_page.h: page busy macro fixups
Minor changes to:
- delete extraneous trailing semicolons from macro definitions, and
- correct spelling of "busying" in panic messages
Submitted by: Ryan Libby <rlibby@gmail.com>
Reviewed by: alc, kib
Sponsored by: EMC / Isilon Storage Division
Differential Revision: https://reviews.freebsd.org/D4577
Diffstat (limited to 'sys')
| -rw-r--r-- | sys/vm/vm_page.h | 17 |
1 files changed, 9 insertions, 8 deletions
diff --git a/sys/vm/vm_page.h b/sys/vm/vm_page.h index 1fa61eb..9abd31c 100644 --- a/sys/vm/vm_page.h +++ b/sys/vm/vm_page.h @@ -514,37 +514,38 @@ void vm_page_lock_assert_KBI(vm_page_t m, int a, const char *file, int line); #define vm_page_assert_sbusied(m) \ KASSERT(vm_page_sbusied(m), \ ("vm_page_assert_sbusied: page %p not shared busy @ %s:%d", \ - (void *)m, __FILE__, __LINE__)); + (m), __FILE__, __LINE__)) #define vm_page_assert_unbusied(m) \ KASSERT(!vm_page_busied(m), \ ("vm_page_assert_unbusied: page %p busy @ %s:%d", \ - (void *)m, __FILE__, __LINE__)); + (m), __FILE__, __LINE__)) #define vm_page_assert_xbusied(m) \ KASSERT(vm_page_xbusied(m), \ ("vm_page_assert_xbusied: page %p not exclusive busy @ %s:%d", \ - (void *)m, __FILE__, __LINE__)); + (m), __FILE__, __LINE__)) #define vm_page_busied(m) \ ((m)->busy_lock != VPB_UNBUSIED) #define vm_page_sbusy(m) do { \ if (!vm_page_trysbusy(m)) \ - panic("%s: page %p failed shared busing", __func__, m); \ + panic("%s: page %p failed shared busying", __func__, \ + (m)); \ } while (0) #define vm_page_tryxbusy(m) \ - (atomic_cmpset_acq_int(&m->busy_lock, VPB_UNBUSIED, \ + (atomic_cmpset_acq_int(&(m)->busy_lock, VPB_UNBUSIED, \ VPB_SINGLE_EXCLUSIVER)) #define vm_page_xbusied(m) \ - ((m->busy_lock & VPB_SINGLE_EXCLUSIVER) != 0) + (((m)->busy_lock & VPB_SINGLE_EXCLUSIVER) != 0) #define vm_page_xbusy(m) do { \ if (!vm_page_tryxbusy(m)) \ - panic("%s: page %p failed exclusive busing", __func__, \ - m); \ + panic("%s: page %p failed exclusive busying", __func__, \ + (m)); \ } while (0) #define vm_page_xunbusy(m) do { \ |
