Stlink: Erreur de construction à partir de la branche de développement, problèmes cpuid
J'essaie de compiler commit c7874f805c63c60285f2b190f6589da8ae5e47fa
Et cela a échoué avec ces erreurs:
/builddir/build/BUILD/stlink-org-stlink-c7874f8/tests/usb.c: In function 'main':
/builddir/build/BUILD/stlink-org-stlink-c7874f8/tests/usb.c:34:9: error: 'cpuid.revision' may be used uninitialized in this func
tion [-Werror=maybe-uninitialized]
34 | printf("cpuid:part = %#x, rev = %#x\n", cpuid.part, cpuid.revision);
| ^
/builddir/build/BUILD/stlink-org-stlink-c7874f8/tests/usb.c:31:27: note: 'cpuid.revision' was declared here
31 | cortex_m3_cpuid_t cpuid;
| ^
/builddir/build/BUILD/stlink-org-stlink-c7874f8/tests/usb.c:34:9: error: 'cpuid.part' may be used uninitialized in this function [-Werror=maybe-uninitialized]
34 | printf("cpuid:part = %#x, rev = %#x\n", cpuid.part, cpuid.revision);
| ^
/builddir/build/BUILD/stlink-org-stlink-c7874f8/tests/usb.c:31:27: note: 'cpuid.part' was declared here
31 | cortex_m3_cpuid_t cpuid;
| ^
/builddir/build/BUILD/stlink-org-stlink-c7874f8/tests/usb.c:33:9: error: 'cpuid.variant' may be used uninitialized in this function [-Werror=maybe-uninitialized]
33 | printf("cpuid:impl_id = %0#x, variant = %#x\n", cpuid.implementer_id, cpuid.variant);
| ^
/builddir/build/BUILD/stlink-org-stlink-c7874f8/tests/usb.c:31:27: note: 'cpuid.variant' was declared here
31 | cortex_m3_cpuid_t cpuid;
| ^
/builddir/build/BUILD/stlink-org-stlink-c7874f8/tests/usb.c:33:9: error: 'cpuid.implementer_id' may be used uninitialized in this function [-Werror=maybe-uninitialized]
33 | printf("cpuid:impl_id = %0#x, variant = %#x\n", cpuid.implementer_id, cpuid.variant);
| ^
/builddir/build/BUILD/stlink-org-stlink-c7874f8/tests/usb.c:31:27: note: 'cpuid.implementer_id' was declared here
31 | cortex_m3_cpuid_t cpuid;
| ^
lto1: all warnings being treated as errors
Comment cela peut-il être corrigé ?
Vascom
Tous les 7 commentaires
Sur macOS, le code se construit bien.
En src/common.c
int stlink_cpu_id(stlink_t *sl, cortex_m3_cpuid_t *cpuid) {
uint32_t raw;
if (stlink_read_debug32(sl, STLINK_REG_CM3_CPUID, &raw))
return -1;
cpuid->implementer_id = (raw >> 24) & 0x7f;
cpuid->variant = (raw >> 20) & 0xf;
cpuid->part = (raw >> 4) & 0xfff;
cpuid->revision = raw & 0xf;
return 0;
}
Il semble que cpuid.part et cpuid.revision soient bien initialisés ici. Peut-être qu'un compilateur se comporte mal ?
chenguokai
le 20 avr. 2020
if (stlink_read_debug32(sl, STLINK_REG_CM3_CPUID, &raw))
return -1;
Dans ce cas, cpuid.* reste non initialisé.
Vascom
le 20 avr. 2020
Sur Ubuntu avec succès compilé aussi. La variable cpuid peut utiliser uninited en cas d'erreur de lecture du registre.
--- a/tests/usb.c
+++ b/tests/usb.c
@@ -29,7 +29,11 @@ int main(int ac, char** av) {
printf("-- core_id: %#x\n", sl->core_id);cortex_m3_cpuid_t cpuid;- stlink_cpu_id(sl, &cpuid);
+ si (stlink_cpu_id(sl, &cpuid))
+ {
+ printf("puce inconnue\n");
+ memset(&cpuid, 0, sizeof(cortex_m3_cpuid_t))
+ }
printf("cpuid:impl_id = %0#x, variant = %#x\n", cpuid.implementer_id, cpuid.variant);
printf("cpuid:part = %#x, rev = %#x\n", cpuid.part, cpuid.revision);
Ant-ON
le 20 avr. 2020
Je pense que cela peut être corrigé en ajoutant une vérification de la valeur de retour sur stlink_cpu_id et en initialisant ces valeurs à zéro. Je vais essayer.
chenguokai
le 20 avr. 2020
J'ai -Werror=maybe-uninitialized car lors de la configuration :
-- Performing Test C_SUPPORTS_WMAYBE_UNINITIALIZED
-- Performing Test C_SUPPORTS_WMAYBE_UNINITIALIZED - Success
@Vascom Pouvez- vous tester ma branche https://github.com/chenguokai/stlink/tree/issue937 ?
Si ça marche, j'augmenterai un PR.
chenguokai
le 20 avr. 2020
@Vascom Pouvez- vous tester ma branche https://github.com/chenguokai/stlink/tree/issue937 ?
Si ça marche, j'augmenterai un PR.
Oui cela fonctionne.
Vascom
le 20 avr. 2020
Questions connexes
lkord
·
14Commentaires
renn0xtek9
·
8Commentaires
Hamid-Najafi
·
8Commentaires
chenguokai
·
6Commentaires
smartHarsh
·
9Commentaires