Commit 25feda6f authored by Kees Cook's avatar Kees Cook Committed by Karol Herbst

drm/nouveau/disp: More DP_RECEIVER_CAP_SIZE array fixes

More arrays (and arguments) for dcpd were set to 16, when it looks like
DP_RECEIVER_CAP_SIZE (15) should be used. Fix the remaining cases, seen
with GCC 13:

../drivers/gpu/drm/nouveau/nvif/outp.c: In function 'nvif_outp_acquire_dp':
../include/linux/fortify-string.h:57:33: warning: array subscript 'unsigned char[16][0]' is partly outside array bounds of 'u8[15]' {aka 'unsigned char[15]'} [-Warray-bounds=]
   57 | #define __underlying_memcpy     __builtin_memcpy
      |                                 ^
...
../drivers/gpu/drm/nouveau/nvif/outp.c:140:9: note: in expansion of macro 'memcpy'
  140 |         memcpy(args.dp.dpcd, dpcd, sizeof(args.dp.dpcd));
      |         ^~~~~~
../drivers/gpu/drm/nouveau/nvif/outp.c:130:49: note: object 'dpcd' of size [0, 15]
  130 | nvif_outp_acquire_dp(struct nvif_outp *outp, u8 dpcd[DP_RECEIVER_CAP_SIZE],
      |                                              ~~~^~~~~~~~~~~~~~~~~~~~~~~~~~

Fixes: 81344372 ("drm/nouveau/disp: move DP link config into acquire")
Cc: Ben Skeggs <bskeggs@redhat.com>
Cc: Lyude Paul <lyude@redhat.com>
Cc: Karol Herbst <kherbst@redhat.com>
Cc: David Airlie <airlied@gmail.com>
Cc: Daniel Vetter <daniel@ffwll.ch>
Cc: Dave Airlie <airlied@redhat.com>
Cc: "Gustavo A. R. Silva" <gustavo@embeddedor.com>
Cc: dri-devel@lists.freedesktop.org
Cc: nouveau@lists.freedesktop.org
Signed-off-by: default avatarKees Cook <keescook@chromium.org>
Reviewed-by: default avatarGustavo A. R. Silva <gustavoars@kernel.org>
Reviewed-by: default avatarKarol Herbst <kherbst@redhat.com>
Signed-off-by: default avatarKarol Herbst <git@karolherbst.de>
Link: https://patchwork.freedesktop.org/patch/msgid/20230204184307.never.825-kees@kernel.org
parent 0d686838
...@@ -2,6 +2,8 @@ ...@@ -2,6 +2,8 @@
#ifndef __NVIF_IF0012_H__ #ifndef __NVIF_IF0012_H__
#define __NVIF_IF0012_H__ #define __NVIF_IF0012_H__
#include <drm/display/drm_dp.h>
union nvif_outp_args { union nvif_outp_args {
struct nvif_outp_v0 { struct nvif_outp_v0 {
__u8 version; __u8 version;
...@@ -63,7 +65,7 @@ union nvif_outp_acquire_args { ...@@ -63,7 +65,7 @@ union nvif_outp_acquire_args {
__u8 hda; __u8 hda;
__u8 mst; __u8 mst;
__u8 pad04[4]; __u8 pad04[4];
__u8 dpcd[16]; __u8 dpcd[DP_RECEIVER_CAP_SIZE];
} dp; } dp;
}; };
} v0; } v0;
......
...@@ -3,6 +3,7 @@ ...@@ -3,6 +3,7 @@
#define __NVKM_DISP_OUTP_H__ #define __NVKM_DISP_OUTP_H__
#include "priv.h" #include "priv.h"
#include <drm/display/drm_dp.h>
#include <subdev/bios.h> #include <subdev/bios.h>
#include <subdev/bios/dcb.h> #include <subdev/bios/dcb.h>
#include <subdev/bios/dp.h> #include <subdev/bios/dp.h>
...@@ -42,7 +43,7 @@ struct nvkm_outp { ...@@ -42,7 +43,7 @@ struct nvkm_outp {
bool aux_pwr_pu; bool aux_pwr_pu;
u8 lttpr[6]; u8 lttpr[6];
u8 lttprs; u8 lttprs;
u8 dpcd[16]; u8 dpcd[DP_RECEIVER_CAP_SIZE];
struct { struct {
int dpcd; /* -1, or index into SUPPORTED_LINK_RATES table */ int dpcd; /* -1, or index into SUPPORTED_LINK_RATES table */
......
...@@ -146,7 +146,7 @@ nvkm_uoutp_mthd_release(struct nvkm_outp *outp, void *argv, u32 argc) ...@@ -146,7 +146,7 @@ nvkm_uoutp_mthd_release(struct nvkm_outp *outp, void *argv, u32 argc)
} }
static int static int
nvkm_uoutp_mthd_acquire_dp(struct nvkm_outp *outp, u8 dpcd[16], nvkm_uoutp_mthd_acquire_dp(struct nvkm_outp *outp, u8 dpcd[DP_RECEIVER_CAP_SIZE],
u8 link_nr, u8 link_bw, bool hda, bool mst) u8 link_nr, u8 link_bw, bool hda, bool mst)
{ {
int ret; int ret;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment