Re: next-20170608 build: 1 failures 4 warnings (next-20170608)

[Sebastian Reichel]

Hi,

On Thu, Jun 08, 2017 at 10:17:57AM -0400, David Miller wrote:
> > On Thu, Jun 08, 2017 at 11:32:31AM +0100, Mark Brown wrote:
> >> On Thu, Jun 08, 2017 at 10:57:50AM +0100, Build bot for Mark Brown wrote:
> >> 
> >> Today's -next fails to build an ARM allmodconfig due to:
> >> 
> >> >  arm-allmodconfig
> >> > ../drivers/hsi/clients/ssi_protocol.c:1069:5: error: 'struct net_device' 
> >> > has no member named 'destructor'
> >> 
> >> due to cf124db566e6 (net: Fix inconsistent teardown and release of
> >> private netdev state.) which missed this instance, presumably due to a
> >> combination of it not being in one of the normal networking driver
> >> directories and only being buildable on ARM.
> > 
> > I assume, that this will be fixed via the net tree.
> > If you Cc me on updated/new patch I will provide Acked-by.
> 
> It is fixed in the 'net' tree, don't worry about it :-)

https://git.kernel.org/pub/scm/linux/kernel/git/davem/net.git/commit/?id=ed66e50d9587fc0bb032e276a2563c0068a5b63a

Looks good, thanks.

-- Sebastian


signature.asc
Description: PGP signature

Re: next-20170608 build: 1 failures 4 warnings (next-20170608)

[David Miller]

From: Sebastian Reichel 
Date: Thu, 8 Jun 2017 13:33:26 +0200

> Hi,
> 
> On Thu, Jun 08, 2017 at 11:32:31AM +0100, Mark Brown wrote:
>> On Thu, Jun 08, 2017 at 10:57:50AM +0100, Build bot for Mark Brown wrote:
>> 
>> Today's -next fails to build an ARM allmodconfig due to:
>> 
>> >arm-allmodconfig
>> > ../drivers/hsi/clients/ssi_protocol.c:1069:5: error: 'struct net_device' 
>> > has no member named 'destructor'
>> 
>> due to cf124db566e6 (net: Fix inconsistent teardown and release of
>> private netdev state.) which missed this instance, presumably due to a
>> combination of it not being in one of the normal networking driver
>> directories and only being buildable on ARM.
> 
> I assume, that this will be fixed via the net tree.
> If you Cc me on updated/new patch I will provide Acked-by.

It is fixed in the 'net' tree, don't worry about it :-)

Re: next-20170608 build: 1 failures 4 warnings (next-20170608)

[David Miller]

From: Stephen Rothwell 
Date: Thu, 8 Jun 2017 20:27:04 +1000

> Hi Dave,
> 
> On Thu, 08 Jun 2017 10:57:50 +0100 Build bot for Mark Brown 
>  wrote:
>>
>> Tree/Branch: next-20170608
>> Git describe: next-20170608
>> Commit: e4689b9aad Add linux-next specific files for 20170608
>> 
>> Build Time: 0 min 12 sec
>> 
>> Passed:6 / 7   ( 85.71 %)
>> Failed:1 / 7   ( 14.29 %)
>> 
>> Errors: 1
>> Warnings: 4
>> Section Mismatches: 0
>> 
>> Failed defconfigs:
>>  arm-allmodconfig
>> 
>> Errors:
>> 
>>  arm-allmodconfig
>> ../drivers/hsi/clients/ssi_protocol.c:1069:5: error: 'struct net_device' has 
>> no member named 'destructor'
> 
> Looks like Mark has found another one.

This should fix it, pushed out to 'net'.

Thanks.


[PATCH] hsi: Fix build regression due to netdev destructor fix.

> ../drivers/hsi/clients/ssi_protocol.c:1069:5: error: 'struct net_device' has 
> no member named 'destructor'

Reported-by: Mark Brown 
Reported-by: Stephen Rothwell 
Signed-off-by: David S. Miller 
---
 drivers/hsi/clients/ssi_protocol.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/hsi/clients/ssi_protocol.c 
b/drivers/hsi/clients/ssi_protocol.c
index 26b0510..93d28c0 100644
--- a/drivers/hsi/clients/ssi_protocol.c
+++ b/drivers/hsi/clients/ssi_protocol.c
@@ -1066,7 +1066,7 @@ static void ssip_pn_setup(struct net_device *dev)
dev->addr_len   = 1;
dev->tx_queue_len   = SSIP_TXQUEUE_LEN;
 
-   dev->destructor = free_netdev;
+   dev->needs_free_netdev  = true;
dev->header_ops = _header_ops;
 }
 
-- 
2.4.11

Re: next-20170608 build: 1 failures 4 warnings (next-20170608)

[Sebastian Reichel]

Hi,

On Thu, Jun 08, 2017 at 11:32:31AM +0100, Mark Brown wrote:
> On Thu, Jun 08, 2017 at 10:57:50AM +0100, Build bot for Mark Brown wrote:
> 
> Today's -next fails to build an ARM allmodconfig due to:
> 
> > arm-allmodconfig
> > ../drivers/hsi/clients/ssi_protocol.c:1069:5: error: 'struct net_device' 
> > has no member named 'destructor'
> 
> due to cf124db566e6 (net: Fix inconsistent teardown and release of
> private netdev state.) which missed this instance, presumably due to a
> combination of it not being in one of the normal networking driver
> directories and only being buildable on ARM.

I assume, that this will be fixed via the net tree.
If you Cc me on updated/new patch I will provide Acked-by.

-- Sebastian


signature.asc
Description: PGP signature

Re: next-20170608 build: 1 failures 4 warnings (next-20170608)

[Mark Brown]

On Thu, Jun 08, 2017 at 10:57:50AM +0100, Build bot for Mark Brown wrote:

Today's -next fails to build an ARM allmodconfig due to:

>   arm-allmodconfig
> ../drivers/hsi/clients/ssi_protocol.c:1069:5: error: 'struct net_device' has 
> no member named 'destructor'

due to cf124db566e6 (net: Fix inconsistent teardown and release of
private netdev state.) which missed this instance, presumably due to a
combination of it not being in one of the normal networking driver
directories and only being buildable on ARM.


signature.asc
Description: PGP signature

Re: next-20170608 build: 1 failures 4 warnings (next-20170608)

[Stephen Rothwell]

Hi Dave,

On Thu, 08 Jun 2017 10:57:50 +0100 Build bot for Mark Brown 
 wrote:
>
> Tree/Branch: next-20170608
> Git describe: next-20170608
> Commit: e4689b9aad Add linux-next specific files for 20170608
> 
> Build Time: 0 min 12 sec
> 
> Passed:6 / 7   ( 85.71 %)
> Failed:1 / 7   ( 14.29 %)
> 
> Errors: 1
> Warnings: 4
> Section Mismatches: 0
> 
> Failed defconfigs:
>   arm-allmodconfig
> 
> Errors:
> 
>   arm-allmodconfig
> ../drivers/hsi/clients/ssi_protocol.c:1069:5: error: 'struct net_device' has 
> no member named 'destructor'

Looks like Mark has found another one.

-- 
Cheers,
Stephen Rothwell