Re: mmotm 2008-09-18-14-56 uploaded (olpc_battery build errors)

Re: mmotm 2008-09-18-14-56 uploaded (olpc_battery build errors)

[Randy.Dunlap]

On Thu, 18 Sep 2008, akpm@linux-foundation.org wrote:

> The mm-of-the-moment snapshot 2008-09-18-14-56 has been uploaded to
> 
>    http://userweb.kernel.org/~akpm/mmotm/
> 
> It contains the following patches against 2.6.27-rc6:

olpc_battery build fails with

mmotm-2008-0918-1456/drivers/power/olpc_battery.c: At top level:
mmotm-2008-0918-1456/drivers/power/olpc_battery.c:363: error: unknown field 'owner' specified in initializer
mmotm-2008-0918-1456/drivers/power/olpc_battery.c:363: warning: excess elements in struct initializer
mmotm-2008-0918-1456/drivers/power/olpc_battery.c:363: warning: (near initialization for 'olpc_bat_eeprom.attr')
make[3]: *** [drivers/power/olpc_battery.o] Error 1


-- 
~Randy

Re: mmotm 2008-09-18-14-56 uploaded (olpc_battery build errors)

[Andrew Morton]

On Thu, 18 Sep 2008 15:42:18 -0700 (PDT)
"Randy.Dunlap" <rdunlap@xenotime.net> wrote:

> On Thu, 18 Sep 2008, akpm@linux-foundation.org wrote:
> 
> > The mm-of-the-moment snapshot 2008-09-18-14-56 has been uploaded to
> > 
> >    http://userweb.kernel.org/~akpm/mmotm/
> > 
> > It contains the following patches against 2.6.27-rc6:
> 
> olpc_battery build fails with
> 
> mmotm-2008-0918-1456/drivers/power/olpc_battery.c: At top level:
> mmotm-2008-0918-1456/drivers/power/olpc_battery.c:363: error: unknown field 'owner' specified in initializer
> mmotm-2008-0918-1456/drivers/power/olpc_battery.c:363: warning: excess elements in struct initializer
> mmotm-2008-0918-1456/drivers/power/olpc_battery.c:363: warning: (near initialization for 'olpc_bat_eeprom.attr')
> make[3]: *** [drivers/power/olpc_battery.o] Error 1
> 

OK, x86-sysfs-kill-owner-field-from-attribute.patch is being a pain. 
The problem is that it tries to kill all `owner' instances and then
remove the `owner' field.

But of course new code keeps on turning up doing the wrong thing and
the build breaks and I hae to keep fixing it.

I'm going to do this:

--- a/include/linux/sysfs.h~x86-sysfs-kill-owner-field-from-attribute-fix-3
+++ a/include/linux/sysfs.h
@@ -27,9 +27,7 @@ struct module;
  */
 struct attribute {
 	const char		*name;
-#ifndef CONFIG_X86
 	struct module		*owner;
-#endif
 	mode_t			mode;
 };
 

Parag, once that patch is merged into mainline can you please send a second
one which cleans up any newly-added occurrences and which also nukes that field.

Re: mmotm 2008-09-18-14-56 uploaded (nilfs2 build error)

[Randy.Dunlap]

On Thu, 18 Sep 2008, akpm@linux-foundation.org wrote:

> The mm-of-the-moment snapshot 2008-09-18-14-56 has been uploaded to
> 
>    http://userweb.kernel.org/~akpm/mmotm/
> 
> It contains the following patches against 2.6.27-rc6:

nilfs2 build fails with

mmotm-2008-0918-1456/fs/nilfs2/inode.c: In function 'nilfs_new_inode':
mmotm-2008-0918-1456/fs/nilfs2/inode.c:283: error: 'struct task_struct' has no member named 'fsuid'
mmotm-2008-0918-1456/fs/nilfs2/inode.c:289: error: 'struct task_struct' has no member named 'fsgid'
make[3]: *** [fs/nilfs2/inode.o] Error 1

-- 
~Randy

Re: mmotm 2008-09-18-14-56 uploaded

[Randy.Dunlap]

On Thu, 18 Sep 2008, akpm@linux-foundation.org wrote:

> The mm-of-the-moment snapshot 2008-09-18-14-56 has been uploaded to
> 
>    http://userweb.kernel.org/~akpm/mmotm/
> 
> It contains the following patches against 2.6.27-rc6:

It would really help (me) if MAINTAINERS was updated to contain
entries for (e.g., these would have been useful today):

nilfs2
olpc

but I expect that there are others as well.

-- 
~Randy