Re: [PATCH] drm/amdgpu/display: make various arrays static, makes object smaller

From: Joe Perches
Date: Tue Oct 08 2019 - 10:00:20 EST


On Tue, 2019-10-08 at 13:56 +0000, Harry Wentland wrote:
[]
> > diff --git a/drivers/gpu/drm/amd/display/dc/bios/bios_parser.c b/drivers/gpu/drm/amd/display/dc/bios/bios_parser.c
> []
> > @@ -2745,7 +2745,7 @@ static enum bp_result bios_get_board_layout_info(
> > struct bios_parser *bp;
> > enum bp_result record_result;
> >
> > - const unsigned int slot_index_to_vbios_id[MAX_BOARD_SLOTS] = {
> > + static const unsigned int slot_index_to_vbios_id[MAX_BOARD_SLOTS] = {
>
> Won't this break the multi-GPU case where you'll have multiple driver
> instances with different layout?

As the array is read-only, how could that happen?