powerpc/vas: Cleanup some debug code

Clean up vas.h and the debug code around ifdef vas_debug.

Signed-off-by: Sukadev Bhattiprolu <sukadev@linux.vnet.ibm.com>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
This commit is contained in:
Sukadev Bhattiprolu 2017-11-07 18:23:43 -08:00 committed by Michael Ellerman
parent 51b537124f
commit 0a2c2c24cf
2 changed files with 17 additions and 45 deletions

View File

@ -726,7 +726,10 @@ static void init_winctx_for_rxwin(struct vas_window *rxwin,
static bool rx_win_args_valid(enum vas_cop_type cop, static bool rx_win_args_valid(enum vas_cop_type cop,
struct vas_rx_win_attr *attr) struct vas_rx_win_attr *attr)
{ {
dump_rx_win_attr(attr); pr_debug("Rxattr: fault %d, notify %d, intr %d, early %d, fifo %d\n",
attr->fault_win, attr->notify_disable,
attr->intr_disable, attr->notify_early,
attr->rx_fifo_size);
if (cop >= VAS_COP_TYPE_MAX) if (cop >= VAS_COP_TYPE_MAX)
return false; return false;
@ -1050,7 +1053,8 @@ int vas_paste_crb(struct vas_window *txwin, int offset, bool re)
else else
rc = -EINVAL; rc = -EINVAL;
print_fifo_msg_count(txwin); pr_debug("Txwin #%d: Msg count %llu\n", txwin->winid,
read_hvwc_reg(txwin, VREG(LRFIFO_PUSH)));
return rc; return rc;
} }

View File

@ -258,6 +258,16 @@
#define VAS_NX_UTIL_ADDER_OFFSET 0x180 #define VAS_NX_UTIL_ADDER_OFFSET 0x180
#define VAS_NX_UTIL_ADDER PPC_BITMASK(32, 63) #define VAS_NX_UTIL_ADDER PPC_BITMASK(32, 63)
/*
* VREG(x):
* Expand a register's short name (eg: LPID) into two parameters:
* - the register's short name in string form ("LPID"), and
* - the name of the macro (eg: VAS_LPID_OFFSET), defining the
* register's offset in the window context
*/
#define VREG_SFX(n, s) __stringify(n), VAS_##n##s
#define VREG(r) VREG_SFX(r, _OFFSET)
/* /*
* Local Notify Scope Control Register. (Receive windows only). * Local Notify Scope Control Register. (Receive windows only).
*/ */
@ -385,43 +395,15 @@ struct vas_winctx {
extern struct vas_instance *find_vas_instance(int vasid); extern struct vas_instance *find_vas_instance(int vasid);
/*
* VREG(x):
* Expand a register's short name (eg: LPID) into two parameters:
* - the register's short name in string form ("LPID"), and
* - the name of the macro (eg: VAS_LPID_OFFSET), defining the
* register's offset in the window context
*/
#define VREG_SFX(n, s) __stringify(n), VAS_##n##s
#define VREG(r) VREG_SFX(r, _OFFSET)
#ifdef vas_debug
static inline void dump_rx_win_attr(struct vas_rx_win_attr *attr)
{
pr_err("fault %d, notify %d, intr %d early %d\n",
attr->fault_win, attr->notify_disable,
attr->intr_disable, attr->notify_early);
pr_err("rx_fifo_size %d, max value %d\n",
attr->rx_fifo_size, VAS_RX_FIFO_SIZE_MAX);
}
static inline void vas_log_write(struct vas_window *win, char *name, static inline void vas_log_write(struct vas_window *win, char *name,
void *regptr, u64 val) void *regptr, u64 val)
{ {
if (val) if (val)
pr_err("%swin #%d: %s reg %p, val 0x%016llx\n", pr_debug("%swin #%d: %s reg %p, val 0x%016llx\n",
win->tx_win ? "Tx" : "Rx", win->winid, name, win->tx_win ? "Tx" : "Rx", win->winid, name,
regptr, val); regptr, val);
} }
#else /* vas_debug */
#define vas_log_write(win, name, reg, val)
#define dump_rx_win_attr(attr)
#endif /* vas_debug */
static inline void write_uwc_reg(struct vas_window *win, char *name, static inline void write_uwc_reg(struct vas_window *win, char *name,
s32 reg, u64 val) s32 reg, u64 val)
{ {
@ -450,18 +432,4 @@ static inline u64 read_hvwc_reg(struct vas_window *win,
return in_be64(win->hvwc_map+reg); return in_be64(win->hvwc_map+reg);
} }
#ifdef vas_debug
static void print_fifo_msg_count(struct vas_window *txwin)
{
uint64_t read_hvwc_reg(struct vas_window *w, char *n, uint64_t o);
pr_devel("Winid %d, Msg count %llu\n", txwin->winid,
(uint64_t)read_hvwc_reg(txwin, VREG(LRFIFO_PUSH)));
}
#else /* vas_debug */
#define print_fifo_msg_count(window)
#endif /* vas_debug */
#endif /* _VAS_H */ #endif /* _VAS_H */