ec2_functests: Remove TODO and enable the config-variations test

Signed-off-by: willieyz <willie.zhao@chelpis.com>

add changes

Signed-off-by: willieyz <willie.zhao@chelpis.com>

Author: willieyz

.github/workflows/ci.yml

@@ -513,32 +513,28 @@ jobs:
             ec2_volume_size: 20
             compile_mode: native
             opt: all
-            # TODO: This config do not exist for now, will be available after PR #607 ("Add Runtime Dispatch") is merged.
-            # config_variations: 'native-cap-CPUID_AVX2'
+            config_variations: 'native-cap-CPUID_AVX2'
           - name: Intel Xeon 4th gen (t3)
             ec2_instance_type: t3.small
             ec2_ami: ubuntu-latest (x86_64)
             ec2_volume_size: 20
             compile_mode: native
             opt: all
-            # TODO: This config do not exist for now, will be available after PR #607 ("Add Runtime Dispatch") is merged.
-            # config_variations: 'native-cap-CPUID_AVX2'
+            config_variations: 'native-cap-CPUID_AVX2'
           - name: Graviton2 (c6g.medium)
             ec2_instance_type: c6g.medium
             ec2_ami: ubuntu-latest (aarch64)
             ec2_volume_size: 20
             compile_mode: native
             opt: all
-            # TODO: This config do not exist for now, will be available after PR #607 ("Add Runtime Dispatch") is merged.
-            # config_variations: 'native-cap-ON native-cap-OFF native-cap-ID_AA64PFR1_EL1'
+            config_variations: 'native-cap-ON native-cap-OFF native-cap-ID_AA64PFR1_EL1'
           - name: Graviton3 (c7g.medium)
             ec2_instance_type: c7g.medium
             ec2_ami: ubuntu-latest (aarch64)
             ec2_volume_size: 20
             compile_mode: native
             opt: all
-            # TODO: This config do not exist for now, will be available after PR #607 ("Add Runtime Dispatch") is merged.
-            # config_variations: 'native-cap-ID_AA64PFR1_EL1'
+            config_variations: 'native-cap-ID_AA64PFR1_EL1'
     name: Platform tests  (${{ matrix.target.name }})
     permissions:
       contents: 'read'

mldsa/src/native/api.h

@@ -27,8 +27,8 @@
 /* Backends may return MLD_NATIVE_FUNC_FALLBACK to signal to the frontend that
  * the target/parameters are unsupported; typically, this would be because of
  * dependencies on CPU features not detected on the host CPU. In this case,
- * the frontend falls back to the default C implementation. 
- * 
+ * the frontend falls back to the default C implementation.
+ *
  * IMPORTANT: Backend implementations must ensure that the decision of whether
  * to fallback (return MLD_NATIVE_FUNC_FALLBACK) or not must never depend on
  * the input data itself. Fallback decisions may only depend on system

mldsa/src/sys.h

@@ -216,15 +216,10 @@
 /* System capability enumeration */
 typedef enum
 {
-#if defined(MLD_SYS_X86_64)
-  MLD_SYS_CAP_AVX2
-#elif defined(MLD_SYS_AARCH64)
+  /* x86_64 */
+  MLD_SYS_CAP_AVX2,
+  /* AArch64 */
   MLD_SYS_CAP_SHA3
-#else
-  /* C90 does not allow empty enums, so use a dummy value
-   * for architectures other than AArch64 and x86_64. */
-  MLD_SYS_CAP_DUMMY
-#endif
 } mld_sys_cap;
 
 #if !defined(MLD_CONFIG_CUSTOM_CAPABILITY_FUNC)