Skip to content
  • Nicolai Hähnle's avatar
    radeonsi: add si_debug_options for convenient adding/removing of options · 8bef4df1
    Nicolai Hähnle authored
    
    
    Move the definition of radeonsi_clear_db_cache_before_clear there,
    as well as radeonsi_enable_nir.
    
    This removes the AMD_DEBUG=nir option.
    
    We currently still have two places for options: the driconf machinery
    and AMD_DEBUG/R600_DEBUG. If we are to have a single place for options,
    then the driconf machinery should be preferred since it's more flexible.
    
    The only downside of the driconf machinery was that adding new options
    was quite inconvenient. With this change, a simple boolean option can
    be added with a single line of code, same as for AMD_DEBUG.
    
    One technical limitation of this particular implementation is that while
    almost all driconf features are available, the translation machinery doesn't
    pick up the description strings for options added in si_debvug_options. In
    practice, translations haven't been provided anyway, and this is intended
    for developer options, so I'm not too worried. It could always be added
    later if anybody really cares.
    
    v2:
    - use bool instead of uint8_t for options
    - si_debug_options.inc -> si_debug_options.h
    
    Reviewed-by: default avatarMarek Olšák <marek.olsak@amd.com>
    8bef4df1