Re: 2.6.23-rc6-mm1

[Greg KH <greg@kroah.com>]

On Tue, Sep 18, 2007 at 03:04:48PM +0530, Satyam Sharma wrote:
> 
> 
> On Tue, 18 Sep 2007, Andrew Morton wrote:
> 
> > On Tue, 18 Sep 2007 14:43:48 +0530 Kamalesh Babulal <kamalesh@linux.vnet.ibm.com> wrote:
> > 
> > > Andrew Morton wrote:
> > > > ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/patches/2.6/2.6.23-rc6/2.6.23-rc6-mm1/
> > > >
> > > > 2.6.23-rc6-mm1 is a 29MB diff against 2.6.23-rc6.
> > > >
> > > >   
> > > <snip>
> > > 
> > > Hi Andrew,
> > > 
> > > The 2.6.23-rc6-mm1build fails at
> > > 
> > >   CC      drivers/pci/hotplug/rpadlpar_core.o
> > >   CC      drivers/pci/hotplug/rpadlpar_sysfs.o
> > > drivers/pci/hotplug/rpadlpar_sysfs.c:132: error: unknown field `name' specified in initializer
> > > drivers/pci/hotplug/rpadlpar_sysfs.c: In function `dlpar_sysfs_init':
> > > drivers/pci/hotplug/rpadlpar_sysfs.c:142: error: structure has no member named `name'
> > > make[3]: *** [drivers/pci/hotplug/rpadlpar_sysfs.o] Error 1
> > > make[2]: *** [drivers/pci/hotplug] Error 2
> > > make[1]: *** [drivers/pci] Error 2
> > > make: *** [drivers] Error 2
> > > 
> > 
> > Probably this:
> > 
> > --- a/drivers/pci/hotplug/rpadlpar_sysfs.c~a
> > +++ a/drivers/pci/hotplug/rpadlpar_sysfs.c
> > @@ -129,8 +129,7 @@ struct kobj_type ktype_dlpar_io = {
> >  };
> >  
> >  struct kset dlpar_io_kset = {
> > -	.kobj = {.name = DLPAR_KOBJ_NAME,
> > -		 .ktype = &ktype_dlpar_io,
> > +	.kobj = {.ktype = &ktype_dlpar_io,
> >  		 .parent = &pci_hotplug_slots_subsys.kobj},
> >  	.ktype = &ktype_dlpar_io,
> >  };
> > _
> > 
> > But I'm fed up with fixing that patch.  It's Greg's turn.
> 
> But wait ... isn't that a statically-allocated kobject, which were
> supposed to be "naughty" in the first place?

Yes it is, if you want to dynamically create it, please do.  But there's
a lot of these in the kernel and I didn't want to break _all_ of them at
once, it's a multiple-stage set of patches to get us there (this name
array change is just the first in the set of these steps...)

thanks,

greg k-h
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/