d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Unbreak 64-bit little-endian systems that do require alignment.

Browse Source 
The fix involves using le16dec(), le32dec(), le16enc() and
le32enc(). This eliminates invalid casts and duplicated logic.
This commit is contained in:
marcel 2006-12-21 05:40:46 +00:00
parent edaada5e3a
commit 24b8c057ed
1 changed files with 5 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
sys/fs/msdosfs/bpb.h
View File

@ -92,25 +92,12 @@ struct bpb710 {
* use the macros for the big-endian case.
*/
#include <machine/endian.h>
#include <sys/endian.h>
#if (BYTE_ORDER == LITTLE_ENDIAN)
#define getushort(x) *((u_int16_t *)(x))
#define getulong(x) *((u_int32_t *)(x))
#define putushort(p, v) (*((u_int16_t *)(p)) = (v))
#define putulong(p, v) (*((u_int32_t *)(p)) = (v))
#else
#define getushort(x) (((u_int8_t *)(x))[0] + (((u_int8_t *)(x))[1] << 8))
#define getulong(x) (u_int32_t)(((u_int8_t *)(x))[0] + (((u_int8_t *)(x))[1] << 8) \
+ (((u_int8_t *)(x))[2] << 16) \
+ (((u_int8_t *)(x))[3] << 24))
#define putushort(p, v) (((u_int8_t *)(p))[0] = (v), \
((u_int8_t *)(p))[1] = (v) >> 8)
#define putulong(p, v) (((u_int8_t *)(p))[0] = (v), \
((u_int8_t *)(p))[1] = (v) >> 8, \
((u_int8_t *)(p))[2] = (v) >> 16,\
((u_int8_t *)(p))[3] = (v) >> 24)
#endif
#define getushort(x) le16dec(x)
#define getulong(x) le32dec(x)
#define putushort(p, v) le16enc(p, v)
#define putulong(p, v) le32enc(p, v)
/*
* BIOS Parameter Block (BPB) for DOS 3.3