Skip to contentSkip to conversation

linux-next: build failure after merge of the drm-misc tree

DISCUSSION Stephen Rothwell 3 participants 4 messages Oct 12, 2021 – Oct 14, 2021 Linux-next tree

lore
Share & tools
lore

Stephen Rothwell

author
#0 lore raw

Hi all,

After merging the drm-misc tree, today's linux-next build (x86_64 allmodconfig) failed like this:

drivers/gpu/drm/nouveau/nvkm/subdev/mmu/vmmgp100.c: In function 'gp100_vmm_fault_cancel':
drivers/gpu/drm/nouveau/nvkm/subdev/mmu/vmmgp100.c:491:6: error: unused variable 'inst' [-Werror=unused-variable]

  491 |  u32 inst, aper;
      |      ^~~~

cc1: all warnings being treated as errors

Caused by commit

404046cf4805 ("drm/nouveau/mmu/gp100-: drop unneeded assignment in the if condition.")

I have used the drm-misc tree from next-20211011 for today.

Signature · 2 lines
Cheers,
Stephen Rothwell
#1 lore raw

uhh, that's on me. I will send out a patch today. I just noticed that the config file I used for testing had WERROR disabled.

Quoted 21 lines · Stephen Rothwell

Hi all,

After merging the drm-misc tree, today's linux-next build (x86_64 allmodconfig) failed like this:

drivers/gpu/drm/nouveau/nvkm/subdev/mmu/vmmgp100.c: In function 'gp100_vmm_fault_cancel':
drivers/gpu/drm/nouveau/nvkm/subdev/mmu/vmmgp100.c:491:6: error: unused variable 'inst' [-Werror=unused-variable]

  491 |  u32 inst, aper;
      |      ^~~~

cc1: all warnings being treated as errors

Caused by commit

404046cf4805 ("drm/nouveau/mmu/gp100-: drop unneeded assignment in the if condition.")

I have used the drm-misc tree from next-20211011 for today.

--
Cheers,
Stephen Rothwell

#2 lore raw
In reply to Stephen Rothwell

Hi,

I review the code.

It seems I forget to delete the definition of the variable "inst",I'm sry for that.: (

I'll submit another patch soon.

Quoted 1 line

Hi all,> > After merging the drm-misc tree, today's linux-next build (x86_64> allmodconfig) failed like this:> > drivers/gpu/drm/nouveau/nvkm/subdev/mmu/vmmgp100.c: In function 'gp100_vmm_fault_cancel':> drivers/gpu/drm/nouveau/nvkm/subdev/mmu/vmmgp100.c:491:6: error: unused variable 'inst' [-Werror=unused-variable]> 491 | u32 inst, aper;> | ^~~~> cc1: all warnings being treated as errors> > Caused by commit> > 404046cf4805 ("drm/nouveau/mmu/gp100-: drop unneeded assignment in the if condition.")>> I have used the drm-misc tree from next-20211011 for today.> > -- > Cheers,> Stephen Rothwell

#3 lore raw
Quoted 9 lines · On Thu, Oct 14, 2021 at 5:02 AM <[email protected]> wrote:

Hi,

I review the code.

It seems I forget to delete the definition of the variable "inst",I'm sry for that.: (

I'll submit another patch soon.

I already wrote the patch and pushed it:
https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit?id=381ba6a6baf104b572379c6b2deab884555104d4

Quoted 21 lines
Quoted 21 lines

Hi all,

After merging the drm-misc tree, today's linux-next build (x86_64 allmodconfig) failed like this:

drivers/gpu/drm/nouveau/nvkm/subdev/mmu/vmmgp100.c: In function 'gp100_vmm_fault_cancel':
drivers/gpu/drm/nouveau/nvkm/subdev/mmu/vmmgp100.c:491:6: error: unused variable 'inst' [-Werror=unused-variable]

  491 |  u32 inst, aper;
      |      ^~~~

cc1: all warnings being treated as errors

Caused by commit

404046cf4805 ("drm/nouveau/mmu/gp100-: drop unneeded assignment in the if condition.")

I have used the drm-misc tree from next-20211011 for today.

--
Cheers,
Stephen Rothwell