pragma pack(1) funktioniert nicht

[Geri (Gast)]

Hallo


sizeof(DC) liefert mir unter gcc auf einem ARM LPC1768 compiliert eine 
Länge von 70 Byte. Bisher waren es 62 Byte.

GNU Tools ARM Embedded\7 2017-q4-major

DataPacket.DataLen  = sizeof(DC); 

#pragma pack(1)
typedef struct
{
  uint16_t HardwareMajorVersion;
  uint16_t HardwareMinorVersion;
  uint16_t FirmwareMajorVersion;
  uint16_t FirmwareMinorVersion;
  uint8_t AxCnt;
  uint32_t CpuClockRate;
  uint8_t CpuName[12];
  uint32_t FifoBufSize;
  uint8_t CompilationDate[22];
  uint8_t FirmwareName[11];
}TDeviceContext;
#pragma pack()

Habe bereits in der gcc Dok gelesen, allerlei andere Direktive versucht 
jedoch keine Besserung.

Habt ihr vielleicht eine Idee wie man den Compiler dazu bewegt die 
Struktur zu packen?

Vielen Dank und beste Grüße
Geri

[Jörg W. (dl8dtl) (Moderator)]

Geri schrieb:
> Habt ihr vielleicht eine Idee wie man den Compiler dazu bewegt die
> Struktur zu packen?

typedef struct
{
  uint16_t HardwareMajorVersion;
  uint16_t HardwareMinorVersion;
  uint16_t FirmwareMajorVersion;
  uint16_t FirmwareMinorVersion;
  uint8_t AxCnt;
  uint32_t CpuClockRate;
  uint8_t CpuName[12];
  uint32_t FifoBufSize;
  uint8_t CompilationDate[22];
  uint8_t FirmwareName[11];
}TDeviceContext __attribute__((packed));

[AbcAbc (Gast)]

Siehe Doku:
Attribut richtig anwenden
struct S { short f[3]; } _attribute_ ((packed));

[Jörg W. (dl8dtl) (Moderator)]

Aber ist natürlich auf einer 32-Bit-Maschine eine blöde Idee, die 
verschiedenen Elemente in so einer Form zu mischen. Irgendwer muss dann 
hingehen, und CpuClockRate aus ihren einzelnen Bytes zusammenpuzzeln, 
statt dass man einfach einen 32-bit-Wert aus dem Speicher liest.

[Geri (Gast)]

Hallo zusammen

@Jörg W.

wenn ich das so mache, dann erhalte ich beim Compilieren folgende 
Warnung:

"../Controller/devinfo.h(383): warning: empty declaration"

und die sizeof liefert immer noch 70 Byte.


>Aber ist natürlich auf einer 32-Bit-Maschine eine blöde Idee, die
>verschiedenen Elemente in so einer Form zu mischen. Irgendwer muss dann
>hingehen, und CpuClockRate aus ihren einzelnen Bytes zusammenpuzzeln,
>statt dass man einfach einen 32-bit-Wert aus dem Speicher liest.
Die Datenstruktur wird nur einmal vom uC über USB ausgetauscht. Die 
Größe des USB-Datenpaketes beträgt 64 Byte. Die Struktur passt so halt 
genau rein...


@AbcAbc: bisher ließ sich sich der Code auf diese Weise unter gcc immer 
einwandfrei compilieren und funktionierte auch.

Beste Grüße

Geri

[Luther B. (luther-blissett)]

AbcAbc schrieb:
> Siehe Doku:
> Attribut richtig anwenden
> struct S { short f[3]; } _attribute_ ((packed));

Zeit wieder einen Blick in die Doku zu werfen. gcc unterstützt #pragma 
pack() seit Version 4.0 - die kam 2005 raus. Ich glaube nicht, daß der 
Compiler vom OP so alt ist. Eventuell liegt das Problem doch woanders.

[Geri (Gast)]

@Luther B. Hattest recht:)


Das Problem lag daran, dass ich das den Compiler-Direktive "GCC" 
vergessen hatte. Unter dieser Option wurden die Standard-Datentypen im 
Programm anders angelegt.

#ifdef TG_XP
  typedef unsigned __int16 uint16_t;
  typedef __int16 int16_t;
  typedef __int8 int8_t;
  typedef unsigned char uint8_t;
  typedef __int32 int32_t;
  typedef unsigned __int32 uint32_t;
  typedef unsigned __int64 uint64_t;
  typedef __int64 int64_t;
#elif GCC
  typedef unsigned short int uint16_t;
  typedef short int int16_t;
  typedef char int8_t;
  typedef unsigned char uint8_t;
  typedef int int32_t;
  typedef unsigned int uint32_t;
  typedef unsigned long long uint64_t;
  typedef long long int64_t;
#else
  typedef unsigned int uint16_t;
  typedef int int16_t;
  typedef int int8_t;
  typedef unsigned char uint8_t;
  typedef long int32_t;
  typedef unsigned long uint32_t;
  typedef unsigned long long uint64_t;
  typedef long long int64_t;
#endif

Nun klappt es. Vielen Dank nochmals

Geri

[Jörg W. (dl8dtl) (Moderator)]

Geri schrieb:
> Die Größe des USB-Datenpaketes beträgt 64 Byte. Die Struktur passt so
> halt genau rein...

Das kannst du auch ganz ohne "pack" haben, indem du die Felder passend 
sortierst:

typedef struct
{
  uint32_t CpuClockRate;
  uint32_t FifoBufSize;
  uint16_t HardwareMajorVersion;
  uint16_t HardwareMinorVersion;
  uint16_t FirmwareMajorVersion;
  uint16_t FirmwareMinorVersion;
  uint8_t CpuName[12];
  uint8_t CompilationDate[22];
  uint8_t FirmwareName[11];
  uint8_t AxCnt;
}
TDeviceContext

[A. S. (Gast)]

Dein uint16_t ist (warum auch immer) 4 Byte groß.

[Jörg W. (dl8dtl) (Moderator)]

Ist sowieso eine doofe Idee, diese Typen selbst zu definieren. Dafür 
nimmt man <stdint.h>.

[Zweig (Gast)]

Jörg W. schrieb:
> Ist sowieso eine doofe Idee, diese Typen selbst zu definieren. Dafür
> nimmt man <stdint.h>.

!!! ganz ganz genau. !!!

[Dirk B. (dirkb2)]

Definiere int8_t als signed char, die 16 Bit Typen über short.

Bedenke auch, dass ein long auf 64-Bit Linux auch 64-Bit groß ist.

[Geri (Gast)]

Vielen Dank für eure Tipps!

>Ist sowieso eine doofe Idee, diese Typen selbst zu definieren. Dafür
>nimmt man <stdint.h>.


Stimmt, ein Relikt aus alten Tagen. Habe nun umgestellt auf <stdint.h>. 
Hoffe, damit wird alles besser:)

Beste Grüße
Geri

[Niklas G. (erlkoenig)]

Siehe auch Serialisierung ... "packed" ist böse :-)

Geri schrieb:
> #elif GCC

Wenn schon dann "__GNUC__" - hilft aber auch nicht so wirklich, da der 
GCC auf verschiedenen Architekturen verschiedene Integer-Größen nutzt. 
stdint.h ist hier natürlich das richtige.