diff options
| author | Eric Anholt <[email protected]> | 2012-10-12 12:41:56 -0700 |
|---|---|---|
| committer | Eric Anholt <[email protected]> | 2012-10-16 13:13:44 -0700 |
| commit | 1fe71848b6ba0bba4def2887d977ac413030a5fb (patch) | |
| tree | 63cfba0f388b4d620b45745126eb7964c67a456d | |
| parent | 85067d4babd6fd8a131088b8c0b673bff021a502 (diff) | |
intel: Add a macro for printing a debug warning once.
There are a number of places where some obscure piece of the code is not
currently worth fixing, and we have some workaround behavior available. It's
nicer for users to do some lame workaround than to just assert, but without
asserts we never knew when the workaround was at fault.
This should give us a nice compromise: Execute the workaround, but mention
that the obscure workaround was hit.
Reviewed-by: Kenneth Graunke <[email protected]>
| -rw-r--r-- | src/mesa/drivers/dri/intel/intel_context.h | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/src/mesa/drivers/dri/intel/intel_context.h b/src/mesa/drivers/dri/intel/intel_context.h index b732696bdaf..eeefadffb29 100644 --- a/src/mesa/drivers/dri/intel/intel_context.h +++ b/src/mesa/drivers/dri/intel/intel_context.h @@ -483,6 +483,17 @@ extern int INTEL_DEBUG; dbg_printf(__VA_ARGS__); \ } while(0) +#define WARN_ONCE(cond, fmt...) do { \ + if (unlikely(cond)) { \ + static bool _warned = false; \ + if (!_warned) { \ + fprintf(stderr, "WARNING: "); \ + fprintf(stderr, fmt); \ + _warned = true; \ + } \ + } \ +} while (0) + #define PCI_CHIP_845_G 0x2562 #define PCI_CHIP_I830_M 0x3577 #define PCI_CHIP_I855_GM 0x3582 |