Changeset View
Changeset View
Standalone View
Standalone View
sys/sys/obm.h
- This file was added.
/*- | |||||
* SPDX-License-Identifier: BSD-2-Clause-FreeBSD | |||||
* | |||||
* Copyright (c) 2020 The FreeBSD Foundation | |||||
* All rights reserved. | |||||
* | |||||
* This software was developed by Konstantin Belousov <kib@FreeBSD.org> | |||||
* under sponsorship from the FreeBSD Foundation. | |||||
* | |||||
* Redistribution and use in source and binary forms, with or without | |||||
* modification, are permitted provided that the following conditions | |||||
* are met: | |||||
* 1. Redistributions of source code must retain the above copyright | |||||
* notice, this list of conditions and the following disclaimer. | |||||
* 2. Redistributions in binary form must reproduce the above copyright | |||||
* notice, this list of conditions and the following disclaimer in the | |||||
* documentation and/or other materials provided with the distribution. | |||||
* | |||||
* THIS SOFTWARE IS PROVIDED BY THE AUTHOR AND CONTRIBUTORS ``AS IS'' AND | |||||
* ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE | |||||
* IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE | |||||
* ARE DISCLAIMED. IN NO EVENT SHALL THE AUTHOR OR CONTRIBUTORS BE LIABLE | |||||
* FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL | |||||
* DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS | |||||
* OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) | |||||
* HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT | |||||
* LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY | |||||
* OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF | |||||
* SUCH DAMAGE. | |||||
* | |||||
* $FreeBSD$ | |||||
*/ | |||||
#ifndef _SYS_OBM_H | |||||
#define _SYS_OBM_H | |||||
/* One-Byte Lock */ | |||||
#ifdef _KERNEL | |||||
#include <sys/systm.h> | |||||
#include <sys/_lock.h> | |||||
#include <sys/_obm.h> | |||||
#include <sys/proc.h> | |||||
#include <machine/atomic.h> | |||||
#ifndef LOCK_DEBUG | |||||
#error "LOCK_DEBUG not defined, include <sys/lock.h> before <sys/obm.h>" | |||||
#endif | |||||
void obm_init_lo(struct lock_object *lo, const char *name); | |||||
markj: Did you consider making a structure to wrap the uint8_t lock state, instead of passing a naked… | |||||
void obm_init(obm_lock_t *obm); | |||||
bool _obm_trylock(obm_lock_t *obm, struct lock_object *lo | |||||
LOCK_FILE_LINE_ARG_DEF); | |||||
void _obm_lock(obm_lock_t *obm, struct lock_object *lo | |||||
LOCK_FILE_LINE_ARG_DEF); | |||||
void _obm_unlock(obm_lock_t *obm, struct lock_object *lo | |||||
LOCK_FILE_LINE_ARG_DEF); | |||||
Done Inline ActionsAre you sure that we have atomic_*_char on all platforms? I do not see it on arm64 for instance, though it does have atomic_*_8. Maybe the latter is a bit more clear anyway. markj: Are you sure that we have atomic_*_char on all platforms? I do not see it on arm64 for instance… | |||||
__used static void | |||||
obm_assert_locked(obm_lock_t *obm) | |||||
{ | |||||
#ifdef INVARIANTS | |||||
uint8_t v; | |||||
v = atomic_load_8(&obm->lk); | |||||
MPASS(v == OBM_LOCKED || v == (OBM_LOCKED | OBM_CONTESTED)); | |||||
#endif | |||||
} | |||||
#if (LOCK_DEBUG > 0) | |||||
#define obm_trylock(obm, lo) \ | |||||
_obm_trylock(obm, lo, __FILE__, __LINE__) | |||||
#define obm_lock(obm, lo) \ | |||||
_obm_lock(obm, lo, __FILE__, __LINE__) | |||||
#define obm_unlock(obm, lo) \ | |||||
_obm_unlock(obm, lo, __FILE__, __LINE__) | |||||
#else | |||||
#define obm_trylock(obm, lo) \ | |||||
_obm_trylock(obm, lo) | |||||
#define obm_lock(obm, lo) \ | |||||
_obm_lock(obm, lo) | |||||
#define obm_unlock(obm, lo) \ | |||||
_obm_unlock(obm, lo) | |||||
#endif | |||||
#endif | |||||
#endif |
Did you consider making a structure to wrap the uint8_t lock state, instead of passing a naked uint8_t around?