linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* linux-next: build failures after merge of the vfs tree
@ 2020-06-16  0:33 Stephen Rothwell
  2020-06-16  0:34 ` Stephen Rothwell
  0 siblings, 1 reply; 15+ messages in thread
From: Stephen Rothwell @ 2020-06-16  0:33 UTC (permalink / raw)
  To: Al Viro; +Cc: Linux Next Mailing List, Linux Kernel Mailing List

[-- Attachment #1: Type: text/plain, Size: 2266 bytes --]

Hi all,

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

drivers/dma/uniphier-xdmac.c: In function 'uniphier_xdmac_prep_dma_memcpy':
drivers/dma/uniphier-xdmac.c:293:7: error: implicit declaration of function 'kzalloc'; did you mean 'kvzalloc'? [-Werror=implicit-function-declaration]
  293 |  xd = kzalloc(struct_size(xd, nodes, nr), GFP_NOWAIT);
      |       ^~~~~~~
      |       kvzalloc
drivers/dma/uniphier-xdmac.c:293:5: warning: assignment to 'struct uniphier_xdmac_desc *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
  293 |  xd = kzalloc(struct_size(xd, nodes, nr), GFP_NOWAIT);
      |     ^
drivers/dma/uniphier-xdmac.c: In function 'uniphier_xdmac_prep_slave_sg':
drivers/dma/uniphier-xdmac.c:349:5: warning: assignment to 'struct uniphier_xdmac_desc *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
  349 |  xd = kzalloc(struct_size(xd, nodes, sg_len), GFP_NOWAIT);
      |     ^
drivers/dma/uniphier-xdmac.c:373:4: error: implicit declaration of function 'kfree'; did you mean 'kvfree'? [-Werror=implicit-function-declaration]
  373 |    kfree(xd);
      |    ^~~~~
      |    kvfree

Also, from the arm multi_v7_defconfig build:

drivers/dma/st_fdma.c: In function 'st_fdma_free_desc':
drivers/dma/st_fdma.c:233:2: error: implicit declaration of function 'kfree'; did you mean 'vfree'? [-Werror=implicit-function-declaration]
  233 |  kfree(fdesc);
      |  ^~~~~
      |  vfree
drivers/dma/st_fdma.c: In function 'st_fdma_alloc_desc':
drivers/dma/st_fdma.c:242:10: error: implicit declaration of function 'kzalloc'; did you mean 'vzalloc'? [-Werror=implicit-function-declaration]
  242 |  fdesc = kzalloc(struct_size(fdesc, node, sg_len), GFP_NOWAIT);
      |          ^~~~~~~
      |          vzalloc
drivers/dma/st_fdma.c:242:8: warning: assignment to 'struct st_fdma_desc *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
  242 |  fdesc = kzalloc(struct_size(fdesc, node, sg_len), GFP_NOWAIT);
      |        ^

Caused by commit

  f0187db056dc ("iov_iter: Move unnecessary inclusion of crypto/hash.h")

I have reverted that commit for today.

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: linux-next: build failures after merge of the vfs tree
  2020-06-16  0:33 linux-next: build failures after merge of the vfs tree Stephen Rothwell
@ 2020-06-16  0:34 ` Stephen Rothwell
  2020-06-16  1:05   ` Herbert Xu
  0 siblings, 1 reply; 15+ messages in thread
From: Stephen Rothwell @ 2020-06-16  0:34 UTC (permalink / raw)
  To: Al Viro; +Cc: Linux Next Mailing List, Linux Kernel Mailing List, Herbert Xu

[-- Attachment #1: Type: text/plain, Size: 2469 bytes --]

[Just adding Herbert to cc]

On Tue, 16 Jun 2020 10:33:30 +1000 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
>
> Hi all,
> 
> After merging the vfs tree, today's linux-next build (x86_64 allmodconfig)
> failed like this:
> 
> drivers/dma/uniphier-xdmac.c: In function 'uniphier_xdmac_prep_dma_memcpy':
> drivers/dma/uniphier-xdmac.c:293:7: error: implicit declaration of function 'kzalloc'; did you mean 'kvzalloc'? [-Werror=implicit-function-declaration]
>   293 |  xd = kzalloc(struct_size(xd, nodes, nr), GFP_NOWAIT);
>       |       ^~~~~~~
>       |       kvzalloc
> drivers/dma/uniphier-xdmac.c:293:5: warning: assignment to 'struct uniphier_xdmac_desc *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
>   293 |  xd = kzalloc(struct_size(xd, nodes, nr), GFP_NOWAIT);
>       |     ^
> drivers/dma/uniphier-xdmac.c: In function 'uniphier_xdmac_prep_slave_sg':
> drivers/dma/uniphier-xdmac.c:349:5: warning: assignment to 'struct uniphier_xdmac_desc *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
>   349 |  xd = kzalloc(struct_size(xd, nodes, sg_len), GFP_NOWAIT);
>       |     ^
> drivers/dma/uniphier-xdmac.c:373:4: error: implicit declaration of function 'kfree'; did you mean 'kvfree'? [-Werror=implicit-function-declaration]
>   373 |    kfree(xd);
>       |    ^~~~~
>       |    kvfree
> 
> Also, from the arm multi_v7_defconfig build:
> 
> drivers/dma/st_fdma.c: In function 'st_fdma_free_desc':
> drivers/dma/st_fdma.c:233:2: error: implicit declaration of function 'kfree'; did you mean 'vfree'? [-Werror=implicit-function-declaration]
>   233 |  kfree(fdesc);
>       |  ^~~~~
>       |  vfree
> drivers/dma/st_fdma.c: In function 'st_fdma_alloc_desc':
> drivers/dma/st_fdma.c:242:10: error: implicit declaration of function 'kzalloc'; did you mean 'vzalloc'? [-Werror=implicit-function-declaration]
>   242 |  fdesc = kzalloc(struct_size(fdesc, node, sg_len), GFP_NOWAIT);
>       |          ^~~~~~~
>       |          vzalloc
> drivers/dma/st_fdma.c:242:8: warning: assignment to 'struct st_fdma_desc *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
>   242 |  fdesc = kzalloc(struct_size(fdesc, node, sg_len), GFP_NOWAIT);
>       |        ^
> 
> Caused by commit
> 
>   f0187db056dc ("iov_iter: Move unnecessary inclusion of crypto/hash.h")
> 
> I have reverted that commit for today.

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: linux-next: build failures after merge of the vfs tree
  2020-06-16  0:34 ` Stephen Rothwell
@ 2020-06-16  1:05   ` Herbert Xu
  2020-06-16  3:38     ` Al Viro
  0 siblings, 1 reply; 15+ messages in thread
From: Herbert Xu @ 2020-06-16  1:05 UTC (permalink / raw)
  To: Stephen Rothwell
  Cc: Al Viro, Linux Next Mailing List, Linux Kernel Mailing List, netdev

On Tue, Jun 16, 2020 at 10:34:40AM +1000, Stephen Rothwell wrote:
> [Just adding Herbert to cc]
> 
> On Tue, 16 Jun 2020 10:33:30 +1000 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> >
> > Hi all,
> > 
> > After merging the vfs tree, today's linux-next build (x86_64 allmodconfig)
> > failed like this:

Thanks Stephen, here is an incremental patch to fix these up.

---8<---
Because linux/uio.h included crypto/hash.h a number of header
files that should have been included weren't.  This patch adds
linux/slab.h where kmalloc/kfree are used, as well as a forward
declaration in linux/socket.h for struct file.

Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
Fixes: 	f0187db056dc ("iov_iter: Move unnecessary inclusion of...")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>

diff --git a/drivers/dma/st_fdma.c b/drivers/dma/st_fdma.c
index 67087dbe2f9f..962b6e05287b 100644
--- a/drivers/dma/st_fdma.c
+++ b/drivers/dma/st_fdma.c
@@ -15,6 +15,7 @@
 #include <linux/platform_device.h>
 #include <linux/interrupt.h>
 #include <linux/remoteproc.h>
+#include <linux/slab.h>
 
 #include "st_fdma.h"
 
diff --git a/drivers/dma/uniphier-xdmac.c b/drivers/dma/uniphier-xdmac.c
index 7b2f8a8c2d31..16b19654873d 100644
--- a/drivers/dma/uniphier-xdmac.c
+++ b/drivers/dma/uniphier-xdmac.c
@@ -12,6 +12,7 @@
 #include <linux/of.h>
 #include <linux/of_dma.h>
 #include <linux/platform_device.h>
+#include <linux/slab.h>
 
 #include "dmaengine.h"
 #include "virt-dma.h"
diff --git a/include/linux/socket.h b/include/linux/socket.h
index 04d2bc97f497..e9cb30d8cbfb 100644
--- a/include/linux/socket.h
+++ b/include/linux/socket.h
@@ -10,6 +10,7 @@
 #include <linux/compiler.h>		/* __user			*/
 #include <uapi/linux/socket.h>
 
+struct file;
 struct pid;
 struct cred;
 struct socket;
-- 
Email: Herbert Xu <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt

^ permalink raw reply related	[flat|nested] 15+ messages in thread

* Re: linux-next: build failures after merge of the vfs tree
  2020-06-16  1:05   ` Herbert Xu
@ 2020-06-16  3:38     ` Al Viro
  2020-06-16 14:38       ` Herbert Xu
  0 siblings, 1 reply; 15+ messages in thread
From: Al Viro @ 2020-06-16  3:38 UTC (permalink / raw)
  To: Herbert Xu
  Cc: Stephen Rothwell, Linux Next Mailing List,
	Linux Kernel Mailing List, netdev

On Tue, Jun 16, 2020 at 11:05:02AM +1000, Herbert Xu wrote:
> On Tue, Jun 16, 2020 at 10:34:40AM +1000, Stephen Rothwell wrote:
> > [Just adding Herbert to cc]
> > 
> > On Tue, 16 Jun 2020 10:33:30 +1000 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> > >
> > > Hi all,
> > > 
> > > After merging the vfs tree, today's linux-next build (x86_64 allmodconfig)
> > > failed like this:
> 
> Thanks Stephen, here is an incremental patch to fix these up.

Folded and pushed

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: linux-next: build failures after merge of the vfs tree
  2020-06-16  3:38     ` Al Viro
@ 2020-06-16 14:38       ` Herbert Xu
  2020-06-16 15:14         ` Al Viro
  2020-06-17  6:57         ` Stephen Rothwell
  0 siblings, 2 replies; 15+ messages in thread
From: Herbert Xu @ 2020-06-16 14:38 UTC (permalink / raw)
  To: Al Viro
  Cc: Stephen Rothwell, Linux Next Mailing List,
	Linux Kernel Mailing List, netdev

On Tue, Jun 16, 2020 at 04:38:49AM +0100, Al Viro wrote:
>
> Folded and pushed

Thanks Al.  Here's another one that I just got, could you add this
one too?

diff --git a/drivers/mtd/nand/raw/cadence-nand-controller.c b/drivers/mtd/nand/raw/cadence-nand-controller.c
index c405722adfe1..c4f273e2fe78 100644
--- a/drivers/mtd/nand/raw/cadence-nand-controller.c
+++ b/drivers/mtd/nand/raw/cadence-nand-controller.c
@@ -17,6 +17,7 @@
 #include <linux/mtd/rawnand.h>
 #include <linux/of_device.h>
 #include <linux/iopoll.h>
+#include <linux/slab.h>
 
 /*
  * HPNFC can work in 3 modes:

Cheers,
-- 
Email: Herbert Xu <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt

^ permalink raw reply related	[flat|nested] 15+ messages in thread

* Re: linux-next: build failures after merge of the vfs tree
  2020-06-16 14:38       ` Herbert Xu
@ 2020-06-16 15:14         ` Al Viro
  2020-06-17  6:57         ` Stephen Rothwell
  1 sibling, 0 replies; 15+ messages in thread
From: Al Viro @ 2020-06-16 15:14 UTC (permalink / raw)
  To: Herbert Xu
  Cc: Stephen Rothwell, Linux Next Mailing List,
	Linux Kernel Mailing List, netdev

On Wed, Jun 17, 2020 at 12:38:07AM +1000, Herbert Xu wrote:
> On Tue, Jun 16, 2020 at 04:38:49AM +0100, Al Viro wrote:
> >
> > Folded and pushed
> 
> Thanks Al.  Here's another one that I just got, could you add this
> one too?

Done...

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: linux-next: build failures after merge of the vfs tree
  2020-06-16 14:38       ` Herbert Xu
  2020-06-16 15:14         ` Al Viro
@ 2020-06-17  6:57         ` Stephen Rothwell
  2020-06-17  7:03           ` Herbert Xu
  1 sibling, 1 reply; 15+ messages in thread
From: Stephen Rothwell @ 2020-06-17  6:57 UTC (permalink / raw)
  To: Herbert Xu
  Cc: Al Viro, Linux Next Mailing List, Linux Kernel Mailing List, netdev

[-- Attachment #1: Type: text/plain, Size: 1064 bytes --]

Hi Herbert,

On Wed, 17 Jun 2020 00:38:07 +1000 Herbert Xu <herbert@gondor.apana.org.au> wrote:
>
> On Tue, Jun 16, 2020 at 04:38:49AM +0100, Al Viro wrote:
> >
> > Folded and pushed  
> 
> Thanks Al.  Here's another one that I just got, could you add this
> one too?
> 
> diff --git a/drivers/mtd/nand/raw/cadence-nand-controller.c b/drivers/mtd/nand/raw/cadence-nand-controller.c
> index c405722adfe1..c4f273e2fe78 100644
> --- a/drivers/mtd/nand/raw/cadence-nand-controller.c
> +++ b/drivers/mtd/nand/raw/cadence-nand-controller.c
> @@ -17,6 +17,7 @@
>  #include <linux/mtd/rawnand.h>
>  #include <linux/of_device.h>
>  #include <linux/iopoll.h>
> +#include <linux/slab.h>
>  
>  /*
>   * HPNFC can work in 3 modes:

Presumably another include needed:

arch/s390/lib/test_unwind.c:49:2: error: implicit declaration of function 'kmalloc' [-Werror=implicit-function-declaration]
arch/s390/lib/test_unwind.c:99:2: error: implicit declaration of function 'kfree' [-Werror=implicit-function-declaration]

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: linux-next: build failures after merge of the vfs tree
  2020-06-17  6:57         ` Stephen Rothwell
@ 2020-06-17  7:03           ` Herbert Xu
  2020-06-17  7:31             ` Stephen Rothwell
  2020-06-17  8:06             ` Stephen Rothwell
  0 siblings, 2 replies; 15+ messages in thread
From: Herbert Xu @ 2020-06-17  7:03 UTC (permalink / raw)
  To: Stephen Rothwell
  Cc: Al Viro, Linux Next Mailing List, Linux Kernel Mailing List, netdev

On Wed, Jun 17, 2020 at 04:57:15PM +1000, Stephen Rothwell wrote:
> 
> Presumably another include needed:
> 
> arch/s390/lib/test_unwind.c:49:2: error: implicit declaration of function 'kmalloc' [-Werror=implicit-function-declaration]
> arch/s390/lib/test_unwind.c:99:2: error: implicit declaration of function 'kfree' [-Werror=implicit-function-declaration]

Hi Stephen:

It's not clear how this file manages to include linux/uio.h but
here is a patch for it anyway:

diff --git a/arch/s390/lib/test_unwind.c b/arch/s390/lib/test_unwind.c
index 32b7a30b2485..eb382ceaa116 100644
--- a/arch/s390/lib/test_unwind.c
+++ b/arch/s390/lib/test_unwind.c
@@ -9,6 +9,7 @@
 #include <linux/kallsyms.h>
 #include <linux/kthread.h>
 #include <linux/module.h>
+#include <linux/slab.h>
 #include <linux/string.h>
 #include <linux/kprobes.h>
 #include <linux/wait.h>
-- 
Email: Herbert Xu <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt

^ permalink raw reply related	[flat|nested] 15+ messages in thread

* Re: linux-next: build failures after merge of the vfs tree
  2020-06-17  7:03           ` Herbert Xu
@ 2020-06-17  7:31             ` Stephen Rothwell
  2020-06-17  7:38               ` Herbert Xu
  2020-06-17  8:06             ` Stephen Rothwell
  1 sibling, 1 reply; 15+ messages in thread
From: Stephen Rothwell @ 2020-06-17  7:31 UTC (permalink / raw)
  To: Herbert Xu
  Cc: Al Viro, Linux Next Mailing List, Linux Kernel Mailing List, netdev

[-- Attachment #1: Type: text/plain, Size: 1117 bytes --]

Hi Herbert,

On Wed, 17 Jun 2020 17:03:17 +1000 Herbert Xu <herbert@gondor.apana.org.au> wrote:
>
> On Wed, Jun 17, 2020 at 04:57:15PM +1000, Stephen Rothwell wrote:
> > 
> > Presumably another include needed:
> > 
> > arch/s390/lib/test_unwind.c:49:2: error: implicit declaration of function 'kmalloc' [-Werror=implicit-function-declaration]
> > arch/s390/lib/test_unwind.c:99:2: error: implicit declaration of function 'kfree' [-Werror=implicit-function-declaration]  
> 

And more (these are coming from other's builds):

  drivers/remoteproc/qcom_q6v5_mss.c:772:3: error: implicit declaration of function 'kfree' [-Werror,-Wimplicit-function-declaration]
  drivers/remoteproc/qcom_q6v5_mss.c:808:2: error: implicit declaration of function 'kfree' [-Werror,-Wimplicit-function-declaration]
  drivers/remoteproc/qcom_q6v5_mss.c:1195:2: error: implicit declaration of function 'kfree' [-Werror,-Wimplicit-function-declaration]

They may have other causes as they are full linux-next builds (not just
after the merge of the vfs tree), but the timing is suspicious.
-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: linux-next: build failures after merge of the vfs tree
  2020-06-17  7:31             ` Stephen Rothwell
@ 2020-06-17  7:38               ` Herbert Xu
  2020-06-18  0:08                 ` Stephen Rothwell
  0 siblings, 1 reply; 15+ messages in thread
From: Herbert Xu @ 2020-06-17  7:38 UTC (permalink / raw)
  To: Stephen Rothwell
  Cc: Al Viro, Linux Next Mailing List, Linux Kernel Mailing List, netdev

On Wed, Jun 17, 2020 at 05:31:02PM +1000, Stephen Rothwell wrote:
> > > 
> > > Presumably another include needed:
> > > 
> > > arch/s390/lib/test_unwind.c:49:2: error: implicit declaration of function 'kmalloc' [-Werror=implicit-function-declaration]
> > > arch/s390/lib/test_unwind.c:99:2: error: implicit declaration of function 'kfree' [-Werror=implicit-function-declaration]  
> 
> And more (these are coming from other's builds):
> 
>   drivers/remoteproc/qcom_q6v5_mss.c:772:3: error: implicit declaration of function 'kfree' [-Werror,-Wimplicit-function-declaration]
>   drivers/remoteproc/qcom_q6v5_mss.c:808:2: error: implicit declaration of function 'kfree' [-Werror,-Wimplicit-function-declaration]
>   drivers/remoteproc/qcom_q6v5_mss.c:1195:2: error: implicit declaration of function 'kfree' [-Werror,-Wimplicit-function-declaration]
> 
> They may have other causes as they are full linux-next builds (not just
> after the merge of the vfs tree), but the timing is suspicious.

OK, here's a patch for both of these together:

diff --git a/arch/s390/lib/test_unwind.c b/arch/s390/lib/test_unwind.c
index 32b7a30b2485..eb382ceaa116 100644
--- a/arch/s390/lib/test_unwind.c
+++ b/arch/s390/lib/test_unwind.c
@@ -9,6 +9,7 @@
 #include <linux/kallsyms.h>
 #include <linux/kthread.h>
 #include <linux/module.h>
+#include <linux/slab.h>
 #include <linux/string.h>
 #include <linux/kprobes.h>
 #include <linux/wait.h>
diff --git a/drivers/remoteproc/qcom_q6v5_mss.c b/drivers/remoteproc/qcom_q6v5_mss.c
index feb70283b6a2..903b2bb97e12 100644
--- a/drivers/remoteproc/qcom_q6v5_mss.c
+++ b/drivers/remoteproc/qcom_q6v5_mss.c
@@ -26,6 +26,7 @@
 #include <linux/reset.h>
 #include <linux/soc/qcom/mdt_loader.h>
 #include <linux/iopoll.h>
+#include <linux/slab.h>
 
 #include "remoteproc_internal.h"
 #include "qcom_common.h"
-- 
Email: Herbert Xu <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt

^ permalink raw reply related	[flat|nested] 15+ messages in thread

* Re: linux-next: build failures after merge of the vfs tree
  2020-06-17  7:03           ` Herbert Xu
  2020-06-17  7:31             ` Stephen Rothwell
@ 2020-06-17  8:06             ` Stephen Rothwell
  1 sibling, 0 replies; 15+ messages in thread
From: Stephen Rothwell @ 2020-06-17  8:06 UTC (permalink / raw)
  To: Herbert Xu
  Cc: Al Viro, Linux Next Mailing List, Linux Kernel Mailing List, netdev

[-- Attachment #1: Type: text/plain, Size: 868 bytes --]

Hi Herbert,

On Wed, 17 Jun 2020 17:03:17 +1000 Herbert Xu <herbert@gondor.apana.org.au> wrote:
>
> On Wed, Jun 17, 2020 at 04:57:15PM +1000, Stephen Rothwell wrote:
> > 
> > Presumably another include needed:
> > 
> > arch/s390/lib/test_unwind.c:49:2: error: implicit declaration of function 'kmalloc' [-Werror=implicit-function-declaration]
> > arch/s390/lib/test_unwind.c:99:2: error: implicit declaration of function 'kfree' [-Werror=implicit-function-declaration]  
> 
> Hi Stephen:
> 
> It's not clear how this file manages to include linux/uio.h but

arch/s390/lib/test_unwind.c
arch/s390/include/asm/unwind.h
include/linux/ftrace.h
include/linux/kallsyms.h
include/linux/module.h
include/linux/elf.h
arch/s390/include/asm/elf.h
include/linux/compat.h
include/linux/socket.h
include/linux/uio.h

:-(

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: linux-next: build failures after merge of the vfs tree
  2020-06-17  7:38               ` Herbert Xu
@ 2020-06-18  0:08                 ` Stephen Rothwell
  2020-06-30  1:58                   ` Stephen Rothwell
  0 siblings, 1 reply; 15+ messages in thread
From: Stephen Rothwell @ 2020-06-18  0:08 UTC (permalink / raw)
  To: Herbert Xu
  Cc: Al Viro, Linux Next Mailing List, Linux Kernel Mailing List, netdev

[-- Attachment #1: Type: text/plain, Size: 2135 bytes --]

Hi all,

On Wed, 17 Jun 2020 17:38:45 +1000 Herbert Xu <herbert@gondor.apana.org.au> wrote:
>
> On Wed, Jun 17, 2020 at 05:31:02PM +1000, Stephen Rothwell wrote:
> > > > 
> > > > Presumably another include needed:
> > > > 
> > > > arch/s390/lib/test_unwind.c:49:2: error: implicit declaration of function 'kmalloc' [-Werror=implicit-function-declaration]
> > > > arch/s390/lib/test_unwind.c:99:2: error: implicit declaration of function 'kfree' [-Werror=implicit-function-declaration]    
> > 
> > And more (these are coming from other's builds):
> > 
> >   drivers/remoteproc/qcom_q6v5_mss.c:772:3: error: implicit declaration of function 'kfree' [-Werror,-Wimplicit-function-declaration]
> >   drivers/remoteproc/qcom_q6v5_mss.c:808:2: error: implicit declaration of function 'kfree' [-Werror,-Wimplicit-function-declaration]
> >   drivers/remoteproc/qcom_q6v5_mss.c:1195:2: error: implicit declaration of function 'kfree' [-Werror,-Wimplicit-function-declaration]
> > 
> > They may have other causes as they are full linux-next builds (not just
> > after the merge of the vfs tree), but the timing is suspicious.  
> 
> OK, here's a patch for both of these together:
> 
> diff --git a/arch/s390/lib/test_unwind.c b/arch/s390/lib/test_unwind.c
> index 32b7a30b2485..eb382ceaa116 100644
> --- a/arch/s390/lib/test_unwind.c
> +++ b/arch/s390/lib/test_unwind.c
> @@ -9,6 +9,7 @@
>  #include <linux/kallsyms.h>
>  #include <linux/kthread.h>
>  #include <linux/module.h>
> +#include <linux/slab.h>
>  #include <linux/string.h>
>  #include <linux/kprobes.h>
>  #include <linux/wait.h>
> diff --git a/drivers/remoteproc/qcom_q6v5_mss.c b/drivers/remoteproc/qcom_q6v5_mss.c
> index feb70283b6a2..903b2bb97e12 100644
> --- a/drivers/remoteproc/qcom_q6v5_mss.c
> +++ b/drivers/remoteproc/qcom_q6v5_mss.c
> @@ -26,6 +26,7 @@
>  #include <linux/reset.h>
>  #include <linux/soc/qcom/mdt_loader.h>
>  #include <linux/iopoll.h>
> +#include <linux/slab.h>
>  
>  #include "remoteproc_internal.h"
>  #include "qcom_common.h"

I have applied those 2 by hand for today.

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: linux-next: build failures after merge of the vfs tree
  2020-06-18  0:08                 ` Stephen Rothwell
@ 2020-06-30  1:58                   ` Stephen Rothwell
  2020-06-30  2:14                     ` Herbert Xu
  0 siblings, 1 reply; 15+ messages in thread
From: Stephen Rothwell @ 2020-06-30  1:58 UTC (permalink / raw)
  To: Herbert Xu
  Cc: Al Viro, Linux Next Mailing List, Linux Kernel Mailing List, netdev

[-- Attachment #1: Type: text/plain, Size: 2361 bytes --]

Hi all,

On Thu, 18 Jun 2020 10:08:51 +1000 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
>
> On Wed, 17 Jun 2020 17:38:45 +1000 Herbert Xu <herbert@gondor.apana.org.au> wrote:
> >
> > On Wed, Jun 17, 2020 at 05:31:02PM +1000, Stephen Rothwell wrote:  
> > > > > 
> > > > > Presumably another include needed:
> > > > > 
> > > > > arch/s390/lib/test_unwind.c:49:2: error: implicit declaration of function 'kmalloc' [-Werror=implicit-function-declaration]
> > > > > arch/s390/lib/test_unwind.c:99:2: error: implicit declaration of function 'kfree' [-Werror=implicit-function-declaration]      
> > > 
> > > And more (these are coming from other's builds):
> > > 
> > >   drivers/remoteproc/qcom_q6v5_mss.c:772:3: error: implicit declaration of function 'kfree' [-Werror,-Wimplicit-function-declaration]
> > >   drivers/remoteproc/qcom_q6v5_mss.c:808:2: error: implicit declaration of function 'kfree' [-Werror,-Wimplicit-function-declaration]
> > >   drivers/remoteproc/qcom_q6v5_mss.c:1195:2: error: implicit declaration of function 'kfree' [-Werror,-Wimplicit-function-declaration]
> > > 
> > > They may have other causes as they are full linux-next builds (not just
> > > after the merge of the vfs tree), but the timing is suspicious.    
> > 
> > OK, here's a patch for both of these together:
> > 
> > diff --git a/arch/s390/lib/test_unwind.c b/arch/s390/lib/test_unwind.c
> > index 32b7a30b2485..eb382ceaa116 100644
> > --- a/arch/s390/lib/test_unwind.c
> > +++ b/arch/s390/lib/test_unwind.c
> > @@ -9,6 +9,7 @@
> >  #include <linux/kallsyms.h>
> >  #include <linux/kthread.h>
> >  #include <linux/module.h>
> > +#include <linux/slab.h>
> >  #include <linux/string.h>
> >  #include <linux/kprobes.h>
> >  #include <linux/wait.h>
> > diff --git a/drivers/remoteproc/qcom_q6v5_mss.c b/drivers/remoteproc/qcom_q6v5_mss.c
> > index feb70283b6a2..903b2bb97e12 100644
> > --- a/drivers/remoteproc/qcom_q6v5_mss.c
> > +++ b/drivers/remoteproc/qcom_q6v5_mss.c
> > @@ -26,6 +26,7 @@
> >  #include <linux/reset.h>
> >  #include <linux/soc/qcom/mdt_loader.h>
> >  #include <linux/iopoll.h>
> > +#include <linux/slab.h>
> >  
> >  #include "remoteproc_internal.h"
> >  #include "qcom_common.h"  
> 
> I have applied those 2 by hand for today.

I am still applying the above patch.

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: linux-next: build failures after merge of the vfs tree
  2020-06-30  1:58                   ` Stephen Rothwell
@ 2020-06-30  2:14                     ` Herbert Xu
  2020-06-30 13:36                       ` Al Viro
  0 siblings, 1 reply; 15+ messages in thread
From: Herbert Xu @ 2020-06-30  2:14 UTC (permalink / raw)
  To: Stephen Rothwell
  Cc: Al Viro, Linux Next Mailing List, Linux Kernel Mailing List, netdev

On Tue, Jun 30, 2020 at 11:58:57AM +1000, Stephen Rothwell wrote:
>
> > > OK, here's a patch for both of these together:
> > > 
> > > diff --git a/arch/s390/lib/test_unwind.c b/arch/s390/lib/test_unwind.c
> > > index 32b7a30b2485..eb382ceaa116 100644
> > > --- a/arch/s390/lib/test_unwind.c
> > > +++ b/arch/s390/lib/test_unwind.c
> > > @@ -9,6 +9,7 @@
> > >  #include <linux/kallsyms.h>
> > >  #include <linux/kthread.h>
> > >  #include <linux/module.h>
> > > +#include <linux/slab.h>
> > >  #include <linux/string.h>
> > >  #include <linux/kprobes.h>
> > >  #include <linux/wait.h>
> > > diff --git a/drivers/remoteproc/qcom_q6v5_mss.c b/drivers/remoteproc/qcom_q6v5_mss.c
> > > index feb70283b6a2..903b2bb97e12 100644
> > > --- a/drivers/remoteproc/qcom_q6v5_mss.c
> > > +++ b/drivers/remoteproc/qcom_q6v5_mss.c
> > > @@ -26,6 +26,7 @@
> > >  #include <linux/reset.h>
> > >  #include <linux/soc/qcom/mdt_loader.h>
> > >  #include <linux/iopoll.h>
> > > +#include <linux/slab.h>
> > >  
> > >  #include "remoteproc_internal.h"
> > >  #include "qcom_common.h"  
> > 
> > I have applied those 2 by hand for today.
> 
> I am still applying the above patch.

Hi Al:

Could you please fold these changes into your tree?

Thanks,
-- 
Email: Herbert Xu <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: linux-next: build failures after merge of the vfs tree
  2020-06-30  2:14                     ` Herbert Xu
@ 2020-06-30 13:36                       ` Al Viro
  0 siblings, 0 replies; 15+ messages in thread
From: Al Viro @ 2020-06-30 13:36 UTC (permalink / raw)
  To: Herbert Xu
  Cc: Stephen Rothwell, Linux Next Mailing List,
	Linux Kernel Mailing List, netdev

On Tue, Jun 30, 2020 at 12:14:01PM +1000, Herbert Xu wrote:

> Could you please fold these changes into your tree?

Done and pushed.  Sorry, had been buried in the regset mess
lately... ;-/

^ permalink raw reply	[flat|nested] 15+ messages in thread

end of thread, other threads:[~2020-06-30 13:36 UTC | newest]

Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-06-16  0:33 linux-next: build failures after merge of the vfs tree Stephen Rothwell
2020-06-16  0:34 ` Stephen Rothwell
2020-06-16  1:05   ` Herbert Xu
2020-06-16  3:38     ` Al Viro
2020-06-16 14:38       ` Herbert Xu
2020-06-16 15:14         ` Al Viro
2020-06-17  6:57         ` Stephen Rothwell
2020-06-17  7:03           ` Herbert Xu
2020-06-17  7:31             ` Stephen Rothwell
2020-06-17  7:38               ` Herbert Xu
2020-06-18  0:08                 ` Stephen Rothwell
2020-06-30  1:58                   ` Stephen Rothwell
2020-06-30  2:14                     ` Herbert Xu
2020-06-30 13:36                       ` Al Viro
2020-06-17  8:06             ` Stephen Rothwell

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).