Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1664462 > unrolled thread

linux-next: build failure after merge of the driver-core tree

Started byStephen Rothwell <sfr@canb.auug.org.au>
First post2017-06-13 08:10 +0200
Last post2017-06-13 08:40 +0200
Articles 4 — 3 participants

Back to article view | Back to linux.kernel


Contents

  linux-next: build failure after merge of the driver-core tree Stephen Rothwell <sfr@canb.auug.org.au> - 2017-06-13 08:10 +0200
    Re: linux-next: build failure after merge of the driver-core tree Stephen Rothwell <sfr@canb.auug.org.au> - 2017-06-13 08:30 +0200
    Re: linux-next: build failure after merge of the driver-core tree Greg KH <greg@kroah.com> - 2017-06-13 08:30 +0200
      Re: linux-next: build failure after merge of the driver-core tree Sergey Senozhatsky <sergey.senozhatsky.work@gmail.com> - 2017-06-13 08:40 +0200

#1664462 — linux-next: build failure after merge of the driver-core tree

FromStephen Rothwell <sfr@canb.auug.org.au>
Date2017-06-13 08:10 +0200
Subjectlinux-next: build failure after merge of the driver-core tree
Message-ID<tRNbz-4k7-5@gated-at.bofh.it>
Hi Greg,

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

drivers/block/zram/zram_drv.c:1296:28: error: expected ')' before numeric constant
 static CLASS_ATTR(hot_add, 0400, hot_add_show, NULL);
                            ^
drivers/block/zram/zram_drv.c:1330:3: error: 'class_attr_hot_add' undeclared here (not in a function)
  &class_attr_hot_add.attr,
   ^
drivers/block/zram/zram_drv.c:1282:16: warning: 'hot_add_show' defined but not used [-Wunused-function]
 static ssize_t hot_add_show(struct class *class,
                ^

Caused by commit

  27104a53d02e ("zram: use class_groups instead of class_attrs")

I have reverted that commit for today (and commit

  ecbaa83ee84c ("driver core: remove class_attrs from struct class")

temporarily so the the tree will build).

-- 
Cheers,
Stephen Rothwell

[toc] | [next] | [standalone]


#1664480

FromStephen Rothwell <sfr@canb.auug.org.au>
Date2017-06-13 08:30 +0200
Message-ID<tRNuW-4sb-1@gated-at.bofh.it>
In reply to#1664462
Hi Greg,

On Tue, 13 Jun 2017 08:20:48 +0200 Greg KH <greg@kroah.com> wrote:
>
> Odd that 0-day isn't giving me any failed build reports :(

Yeah, I wonder if 0-day is doing x86_64 allmodconfig builds regularly ...

-- 
Cheers,
Stephen Rothwell

[toc] | [prev] | [next] | [standalone]


#1664482

FromGreg KH <greg@kroah.com>
Date2017-06-13 08:30 +0200
Message-ID<tRNuW-4sb-3@gated-at.bofh.it>
In reply to#1664462
On Tue, Jun 13, 2017 at 04:04:18PM +1000, Stephen Rothwell wrote:
> Hi Greg,
> 
> After merging the driver-core tree, today's linux-next build (x86_64
> allmodconfig) failed like this:
> 
> drivers/block/zram/zram_drv.c:1296:28: error: expected ')' before numeric constant
>  static CLASS_ATTR(hot_add, 0400, hot_add_show, NULL);
>                             ^
> drivers/block/zram/zram_drv.c:1330:3: error: 'class_attr_hot_add' undeclared here (not in a function)
>   &class_attr_hot_add.attr,
>    ^
> drivers/block/zram/zram_drv.c:1282:16: warning: 'hot_add_show' defined but not used [-Wunused-function]
>  static ssize_t hot_add_show(struct class *class,
>                 ^
> 
> Caused by commit
> 
>   27104a53d02e ("zram: use class_groups instead of class_attrs")
> 
> I have reverted that commit for today (and commit
> 
>   ecbaa83ee84c ("driver core: remove class_attrs from struct class")
> 
> temporarily so the the tree will build).

Crap I missed one.  I'll go fix that up now so this should work for
tomorrows linux-next, thanks for letting me know.

Odd that 0-day isn't giving me any failed build reports :(

thanks,

greg k-h

[toc] | [prev] | [next] | [standalone]


#1664492

FromSergey Senozhatsky <sergey.senozhatsky.work@gmail.com>
Date2017-06-13 08:40 +0200
Message-ID<tRNEB-4v8-3@gated-at.bofh.it>
In reply to#1664482
On (06/13/17 08:20), Greg KH wrote:
> On Tue, Jun 13, 2017 at 04:04:18PM +1000, Stephen Rothwell wrote:
> > Hi Greg,
> > 
> > After merging the driver-core tree, today's linux-next build (x86_64
> > allmodconfig) failed like this:
> > 
> > drivers/block/zram/zram_drv.c:1296:28: error: expected ')' before numeric constant
> >  static CLASS_ATTR(hot_add, 0400, hot_add_show, NULL);
> >                             ^
> > drivers/block/zram/zram_drv.c:1330:3: error: 'class_attr_hot_add' undeclared here (not in a function)
> >   &class_attr_hot_add.attr,
> >    ^
> > drivers/block/zram/zram_drv.c:1282:16: warning: 'hot_add_show' defined but not used [-Wunused-function]
> >  static ssize_t hot_add_show(struct class *class,
> >                 ^
> > 
> > Caused by commit
> > 
> >   27104a53d02e ("zram: use class_groups instead of class_attrs")
> > 
> > I have reverted that commit for today (and commit
> > 
> >   ecbaa83ee84c ("driver core: remove class_attrs from struct class")
> > 
> > temporarily so the the tree will build).
> 
> Crap I missed one.  I'll go fix that up now so this should work for
> tomorrows linux-next, thanks for letting me know.
> 
> Odd that 0-day isn't giving me any failed build reports :(

how did I miss that... I was pretty sure I compile-tested
the patch before I replied... odd... sorry!

thanks for the report, Stephen.

	-ss

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web