Browse Source

For DYNAMIC_ARCH don't use sbgemm_r as parameter.c doesn't get build

Presently, DYNAMIC_ARCH doesn't build if the build includes Intel
Cooperlake or Sapphire Rapids cores. This is because their init
function reference sbgemm_r which is defined in
driver/other/parameter.c.
This file is not built when using DYNAMIC_ARCH.
The value is the one that blas_set_parameter() would set on build
without DYNAMIC_ARCH.
There seems to be some duplication between blas_set_parameter() and
the kernel specific init_parameter() calls. Some consolidation would
be in order here.

Signed-off-by: Egbert Eich <eich@suse.com>
pull/3570/head
Egbert Eich 4 years ago
parent
commit
815c7ab15e
1 changed files with 8 additions and 0 deletions
  1. +8
    -0
      param.h

+ 8
- 0
param.h View File

@@ -1803,7 +1803,11 @@ USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
#define SBGEMM_DEFAULT_UNROLL_M 32
#define SBGEMM_DEFAULT_P 256
#define SBGEMM_DEFAULT_Q 1024
#ifndef DYNAMIC_ARCH
#define SBGEMM_DEFAULT_R sbgemm_r
#else
#define SBGEMM_DEFAULT_R 43280
#endif

#ifdef ARCH_X86

@@ -1933,7 +1937,11 @@ USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
#define SBGEMM_DEFAULT_UNROLL_M 16
#define SBGEMM_DEFAULT_P 384
#define SBGEMM_DEFAULT_Q 768
#ifndef DYNAMIC_ARCH
#define SBGEMM_DEFAULT_R sbgemm_r
#else
#define SBGEMM_DEFAULT_R 43280
#endif

#ifdef ARCH_X86



Loading…
Cancel
Save