Page MenuHomeFreeBSD
Files F137574680

D21076.id.diff
No OneTemporary
Actions

D21076.id.diff
View Options

Index: head/sys/dev/sfxge/common/ef10_rx.c
===================================================================
--- head/sys/dev/sfxge/common/ef10_rx.c
+++ head/sys/dev/sfxge/common/ef10_rx.c
@@ -1131,12 +1131,12 @@
rc = ENOTSUP;
goto fail9;
}
- if (!IS_P2ALIGNED(es_max_dma_len,
+ if (!EFX_IS_P2ALIGNED(uint32_t, es_max_dma_len,
EFX_RX_ES_SUPER_BUFFER_BUF_ALIGNMENT)) {
rc = EINVAL;
goto fail10;
}
- if (!IS_P2ALIGNED(es_buf_stride,
+ if (!EFX_IS_P2ALIGNED(uint32_t, es_buf_stride,
EFX_RX_ES_SUPER_BUFFER_BUF_ALIGNMENT)) {
rc = EINVAL;
goto fail11;
Index: head/sys/dev/sfxge/common/efsys.h
===================================================================
--- head/sys/dev/sfxge/common/efsys.h
+++ head/sys/dev/sfxge/common/efsys.h
@@ -84,10 +84,6 @@
#define B_TRUE TRUE
#endif
-#ifndef IS_P2ALIGNED
-#define IS_P2ALIGNED(v, a) ((((uintptr_t)(v)) & ((uintptr_t)(a) - 1)) == 0)
-#endif
-
#ifndef IS2P
#define ISP2(x) (((x) & ((x) - 1)) == 0)
#endif
@@ -375,7 +371,8 @@
uint32_t *addr; \
\
_NOTE(CONSTANTCONDITION) \
- KASSERT(IS_P2ALIGNED(_offset, sizeof (efx_dword_t)), \
+ KASSERT(EFX_IS_P2ALIGNED(size_t, _offset, \
+ sizeof (efx_dword_t)), \
("not power of 2 aligned")); \
\
addr = (void *)((_esmp)->esm_base + (_offset)); \
@@ -394,7 +391,8 @@
uint64_t *addr; \
\
_NOTE(CONSTANTCONDITION) \
- KASSERT(IS_P2ALIGNED(_offset, sizeof (efx_qword_t)), \
+ KASSERT(EFX_IS_P2ALIGNED(size_t, _offset, \
+ sizeof (efx_qword_t)), \
("not power of 2 aligned")); \
\
addr = (void *)((_esmp)->esm_base + (_offset)); \
@@ -413,7 +411,8 @@
uint32_t *addr; \
\
_NOTE(CONSTANTCONDITION) \
- KASSERT(IS_P2ALIGNED(_offset, sizeof (efx_qword_t)), \
+ KASSERT(EFX_IS_P2ALIGNED(size_t, _offset, \
+ sizeof (efx_qword_t)), \
("not power of 2 aligned")); \
\
addr = (void *)((_esmp)->esm_base + (_offset)); \
@@ -435,7 +434,8 @@
uint64_t *addr; \
\
_NOTE(CONSTANTCONDITION) \
- KASSERT(IS_P2ALIGNED(_offset, sizeof (efx_oword_t)), \
+ KASSERT(EFX_IS_P2ALIGNED(size_t, _offset, \
+ sizeof (efx_oword_t)), \
("not power of 2 aligned")); \
\
addr = (void *)((_esmp)->esm_base + (_offset)); \
@@ -457,7 +457,8 @@
uint32_t *addr; \
\
_NOTE(CONSTANTCONDITION) \
- KASSERT(IS_P2ALIGNED(_offset, sizeof (efx_oword_t)), \
+ KASSERT(EFX_IS_P2ALIGNED(size_t, _offset, \
+ sizeof (efx_oword_t)), \
("not power of 2 aligned")); \
\
addr = (void *)((_esmp)->esm_base + (_offset)); \
@@ -482,7 +483,8 @@
uint32_t *addr; \
\
_NOTE(CONSTANTCONDITION) \
- KASSERT(IS_P2ALIGNED(_offset, sizeof (efx_dword_t)), \
+ KASSERT(EFX_IS_P2ALIGNED(size_t, _offset, \
+ sizeof (efx_dword_t)), \
("not power of 2 aligned")); \
\
EFSYS_PROBE2(mem_writed, unsigned int, (_offset), \
@@ -501,7 +503,8 @@
uint64_t *addr; \
\
_NOTE(CONSTANTCONDITION) \
- KASSERT(IS_P2ALIGNED(_offset, sizeof (efx_qword_t)), \
+ KASSERT(EFX_IS_P2ALIGNED(size_t, _offset, \
+ sizeof (efx_qword_t)), \
("not power of 2 aligned")); \
\
EFSYS_PROBE3(mem_writeq, unsigned int, (_offset), \
@@ -521,7 +524,8 @@
uint32_t *addr; \
\
_NOTE(CONSTANTCONDITION) \
- KASSERT(IS_P2ALIGNED(_offset, sizeof (efx_qword_t)), \
+ KASSERT(EFX_IS_P2ALIGNED(size_t, _offset, \
+ sizeof (efx_qword_t)), \
("not power of 2 aligned")); \
\
EFSYS_PROBE3(mem_writeq, unsigned int, (_offset), \
@@ -543,7 +547,8 @@
uint64_t *addr; \
\
_NOTE(CONSTANTCONDITION) \
- KASSERT(IS_P2ALIGNED(_offset, sizeof (efx_oword_t)), \
+ KASSERT(EFX_IS_P2ALIGNED(size_t, _offset, \
+ sizeof (efx_oword_t)), \
("not power of 2 aligned")); \
\
EFSYS_PROBE5(mem_writeo, unsigned int, (_offset), \
@@ -565,7 +570,8 @@
uint32_t *addr; \
\
_NOTE(CONSTANTCONDITION) \
- KASSERT(IS_P2ALIGNED(_offset, sizeof (efx_oword_t)), \
+ KASSERT(EFX_IS_P2ALIGNED(size_t, _offset, \
+ sizeof (efx_oword_t)), \
("not power of 2 aligned")); \
\
EFSYS_PROBE5(mem_writeo, unsigned int, (_offset), \
@@ -617,7 +623,8 @@
#define EFSYS_BAR_READD(_esbp, _offset, _edp, _lock) \
do { \
_NOTE(CONSTANTCONDITION) \
- KASSERT(IS_P2ALIGNED(_offset, sizeof (efx_dword_t)), \
+ KASSERT(EFX_IS_P2ALIGNED(size_t, _offset, \
+ sizeof (efx_dword_t)), \
("not power of 2 aligned")); \
\
_NOTE(CONSTANTCONDITION) \
@@ -641,7 +648,8 @@
#define EFSYS_BAR_READQ(_esbp, _offset, _eqp) \
do { \
_NOTE(CONSTANTCONDITION) \
- KASSERT(IS_P2ALIGNED(_offset, sizeof (efx_qword_t)), \
+ KASSERT(EFX_IS_P2ALIGNED(size_t, _offset, \
+ sizeof (efx_qword_t)), \
("not power of 2 aligned")); \
\
SFXGE_BAR_LOCK(_esbp); \
@@ -661,7 +669,8 @@
#define EFSYS_BAR_READO(_esbp, _offset, _eop, _lock) \
do { \
_NOTE(CONSTANTCONDITION) \
- KASSERT(IS_P2ALIGNED(_offset, sizeof (efx_oword_t)), \
+ KASSERT(EFX_IS_P2ALIGNED(size_t, _offset, \
+ sizeof (efx_oword_t)), \
("not power of 2 aligned")); \
\
_NOTE(CONSTANTCONDITION) \
@@ -691,7 +700,8 @@
#define EFSYS_BAR_READQ(_esbp, _offset, _eqp) \
do { \
_NOTE(CONSTANTCONDITION) \
- KASSERT(IS_P2ALIGNED(_offset, sizeof (efx_qword_t)), \
+ KASSERT(EFX_IS_P2ALIGNED(size_t, _offset, \
+ sizeof (efx_qword_t)), \
("not power of 2 aligned")); \
\
SFXGE_BAR_LOCK(_esbp); \
@@ -714,7 +724,8 @@
#define EFSYS_BAR_READO(_esbp, _offset, _eop, _lock) \
do { \
_NOTE(CONSTANTCONDITION) \
- KASSERT(IS_P2ALIGNED(_offset, sizeof (efx_oword_t)), \
+ KASSERT(EFX_IS_P2ALIGNED(size_t, _offset, \
+ sizeof (efx_oword_t)), \
("not power of 2 aligned")); \
\
_NOTE(CONSTANTCONDITION) \
@@ -750,7 +761,8 @@
#define EFSYS_BAR_WRITED(_esbp, _offset, _edp, _lock) \
do { \
_NOTE(CONSTANTCONDITION) \
- KASSERT(IS_P2ALIGNED(_offset, sizeof (efx_dword_t)), \
+ KASSERT(EFX_IS_P2ALIGNED(size_t, _offset, \
+ sizeof (efx_dword_t)), \
("not power of 2 aligned")); \
\
_NOTE(CONSTANTCONDITION) \
@@ -782,7 +794,8 @@
#define EFSYS_BAR_WRITEQ(_esbp, _offset, _eqp) \
do { \
_NOTE(CONSTANTCONDITION) \
- KASSERT(IS_P2ALIGNED(_offset, sizeof (efx_qword_t)), \
+ KASSERT(EFX_IS_P2ALIGNED(size_t, _offset, \
+ sizeof (efx_qword_t)), \
("not power of 2 aligned")); \
\
SFXGE_BAR_LOCK(_esbp); \
@@ -810,7 +823,8 @@
#define EFSYS_BAR_WRITEQ(_esbp, _offset, _eqp) \
do { \
_NOTE(CONSTANTCONDITION) \
- KASSERT(IS_P2ALIGNED(_offset, sizeof (efx_qword_t)), \
+ KASSERT(EFX_IS_P2ALIGNED(size_t, _offset, \
+ sizeof (efx_qword_t)), \
("not power of 2 aligned")); \
\
SFXGE_BAR_LOCK(_esbp); \
@@ -854,7 +868,8 @@
#define EFSYS_BAR_WC_WRITEQ(_esbp, _offset, _eqp) \
do { \
_NOTE(CONSTANTCONDITION) \
- KASSERT(IS_P2ALIGNED(_offset, sizeof (efx_qword_t)), \
+ KASSERT(EFX_IS_P2ALIGNED(size_t, _offset, \
+ sizeof (efx_qword_t)), \
("not power of 2 aligned")); \
\
(void) (_esbp); \
@@ -869,7 +884,8 @@
#define EFSYS_BAR_WRITEO(_esbp, _offset, _eop, _lock) \
do { \
_NOTE(CONSTANTCONDITION) \
- KASSERT(IS_P2ALIGNED(_offset, sizeof (efx_oword_t)), \
+ KASSERT(EFX_IS_P2ALIGNED(size_t, _offset, \
+ sizeof (efx_oword_t)), \
("not power of 2 aligned")); \
\
_NOTE(CONSTANTCONDITION) \
@@ -915,7 +931,8 @@
#define EFSYS_BAR_WRITEO(_esbp, _offset, _eop, _lock) \
do { \
_NOTE(CONSTANTCONDITION) \
- KASSERT(IS_P2ALIGNED(_offset, sizeof (efx_oword_t)), \
+ KASSERT(EFX_IS_P2ALIGNED(size_t, _offset, \
+ sizeof (efx_oword_t)), \
("not power of 2 aligned")); \
\
_NOTE(CONSTANTCONDITION) \
Index: head/sys/dev/sfxge/common/efx.h
===================================================================
--- head/sys/dev/sfxge/common/efx.h
+++ head/sys/dev/sfxge/common/efx.h
@@ -64,6 +64,10 @@
#define EFX_P2ALIGN(_type, _value, _align) \
((_type)(_value) & -(_type)(_align))
+/* Test if value is power of 2 aligned. */
+#define EFX_IS_P2ALIGNED(_type, _value, _align) \
+ ((((_type)(_value)) & ((_type)(_align) - 1)) == 0)
+
/* Return codes */
typedef __success(return == 0) int efx_rc_t;

File Metadata

Mime Type
text/plain
Expires
Tue, Nov 25, 9:59 AM (5 h, 39 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
26123412
Default Alt Text
D21076.id.diff (8 KB)

Event Timeline