Преглед изворни кода

drm/nouveau/nouveau: bios pointers may be unaligned, use proper accessors

This can show up on SPARC or other architectures that don't handle
unaligned accesses. The kernel normally fixes these up, but it shouldn't
have to.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96836
Signed-off-by: Ilia Mirkin <imirkin@alum.mit.edu>
Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
Ilia Mirkin пре 9 година
родитељ
комит
9936aeeaeb
2 измењених фајлова са 4 додато и 6 уклоњено
  1. 2 4
      drivers/gpu/drm/nouveau/nouveau_bios.h
  2. 2 2
      drivers/gpu/drm/nouveau/nvkm/subdev/mxm/mxms.c

+ 2 - 4
drivers/gpu/drm/nouveau/nouveau_bios.h

@@ -31,10 +31,8 @@
 
 
 #define DCB_LOC_ON_CHIP 0
 #define DCB_LOC_ON_CHIP 0
 
 
-#define ROM16(x) le16_to_cpu(*(u16 *)&(x))
-#define ROM32(x) le32_to_cpu(*(u32 *)&(x))
-#define ROM48(x) ({ u8 *p = &(x); (u64)ROM16(p[4]) << 32 | ROM32(p[0]); })
-#define ROM64(x) le64_to_cpu(*(u64 *)&(x))
+#define ROM16(x) get_unaligned_le16(&(x))
+#define ROM32(x) get_unaligned_le32(&(x))
 #define ROMPTR(d,x) ({            \
 #define ROMPTR(d,x) ({            \
 	struct nouveau_drm *drm = nouveau_drm((d)); \
 	struct nouveau_drm *drm = nouveau_drm((d)); \
 	ROM16(x) ? &drm->vbios.data[ROM16(x)] : NULL; \
 	ROM16(x) ? &drm->vbios.data[ROM16(x)] : NULL; \

+ 2 - 2
drivers/gpu/drm/nouveau/nvkm/subdev/mxm/mxms.c

@@ -23,8 +23,8 @@
  */
  */
 #include "mxms.h"
 #include "mxms.h"
 
 
-#define ROM16(x) le16_to_cpu(*(u16 *)&(x))
-#define ROM32(x) le32_to_cpu(*(u32 *)&(x))
+#define ROM16(x) get_unaligned_le16(&(x))
+#define ROM32(x) get_unaligned_le32(&(x))
 
 
 static u8 *
 static u8 *
 mxms_data(struct nvkm_mxm *mxm)
 mxms_data(struct nvkm_mxm *mxm)