Discussion :

[hampoireau]

Bonjour,
pour le développement d'un module pour le noyau linux sur ARM, pour la lecture "brute" d'un zone de mémoire flash,
j'ai un buffer non pas en char* mais d'une structure représenant la forme des données en mémoire flash.

Voici ma structure:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
typedef struct 
{
  //métadonnées
  uint8_t id;
  ...
  uint32_t klen;
  uint32_t vlen;
 
  //données
  uint8_t data[1];
}page;

Comme la taille de la page n'est pas connue statiquement, et elle peut varier voici comment je l'utilise:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
page *mon_buffer=vmalloc( page_length );
read_flash(mon_buffer, page_length)

Ainsi je n'ai pas besoin de spécifier une par une les métadonnées et je peux accéder directement au donnée utiles via

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

mon_buffer->data+offset_data

.

Cela est-il corrècte ? peut il y a avoir un cas qui mettrais en défaut ce choix de mise en oeuvre (si oui lequel ?) ?
Et au sens plus "conventionel" cela pose t-il un probleme de "violation" des conventions ... ?

Merci

[Médinoc]

L'important, c'est comment du détermines l'allocation.

Normalement, tu es censé définir la taille comme offsetof(page, data) + taille_des_data.
offsetof étant défini dans <stddef.h>.

[diogene]

La construction prévue par le standard ne serait-elle pas plutôt :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
typedef struct 
{
  //métadonnées
  uint8_t id;
  ...
  uint32_t klen;
  uint32_t vlen;
 
  //données
  uint8_t data[];   // sans préciser la dimension
}page;

[Médinoc]

Sur les compilos assez récent pour le supporter.

La seule différence, c'est le résultat d'un sizeof(page), il me semble. Dans tous les cas ça fait ce qu'il faut pour éviter les problèmes d'alignement.

[hampoireau]

Salut,
merci de vos réponses,
j'ai changé le

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

uint8_t data[1];

en

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

uint8_t data[];

@Médinoc:
J'utilise la structure comme une représentation "brute" des données en mémoire flash (pour la lecture et l'écriture, ce qui évite pas mal de casts et pas mal d'erreurs), sur la machien de test, chaque page fait 2048 octets (mais peut changer selon l'architecture).
Les métadonnées ayant une taille constante de 16 octets (en contant les "bouches trous" de l'alignement) donc:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

offsetof(page, data) + taille_des_data =  page_length

Donc normalement en faisant ça, je respecte les standards ?

[Médinoc]

Ça a l'air, tant que les contraintes d'alignement de la structure sont précisées correctement pour bien correspondre à ce que tu lis (et il n'y a rien dans le standard pour cela, ça dépend du compilateur).