RE: 2.6.22.14 oops msg with commvault galaxy ?

From: Fortier,Vincent [Montreal]
Date: Tue Dec 11 2007 - 12:48:20 EST


> -----Message d'origine-----
> De : linux-kernel-owner@xxxxxxxxxxxxxxx
> [mailto:linux-kernel-owner@xxxxxxxxxxxxxxx] De la part de Dhaval Giani
>
> On Tue, Dec 11, 2007 at 09:04:00AM -0800, Greg KH wrote:
> > On Tue, Dec 11, 2007 at 10:13:19PM +0530, Dhaval Giani wrote:
> > > On Tue, Dec 11, 2007 at 08:24:37PM +0530, Dhaval Giani wrote:
> > > > On Mon, Dec 10, 2007 at 09:15:01AM -0800, Randy Dunlap wrote:
> > > > > On Mon, 10 Dec 2007 09:03:17 -0500 Fortier,Vincent wrote:
> > > > >
> > > > > Ingo, can you look at this, please?
> > > > > Vincent is getting oopses on 2.6.22.14-cfs-etch.
> > > > >
> > > >
> > > > Hi,
> > > >
> > > > We are looking into this bug now. I believe that the patch at
> > > > http://marc.info/?l=linux-kernel&m=119404922603293 should help.
> > > >
> > > > I am working with Kay to get this ported.
> > > >
> > >
> > > Hi Vincent,
> > >
> > > Does the following patch help?
> > >
> > > Kay/Greg, could you please review and add your Signed-off-by(s) as

> > > required?
> >
> > Um, why? What is this patch for? Where is it to be sent, to Linus
> > for 2.6.24-final? Or to the -stable tree?
> >
>
> Hi Greg,
>
> This is for 2.26.24-final, since Fair User scheduling is not
> yet there in stable.
>
> > > This is basically a port of the patch at
> > > http://marc.info/?l=linux-kernel&m=119404922603293
> >
> > Yeah, but that patch needs some other core kobject changes, right?
> >
>
> Yep, there are some other changes that patch needed. We have
> worked around them by using the existing functions in the
> current Linus tree.
>
> > What exactly are you trying to fix here, the fact that this code
never
> > even worked?
> >
>
> The code was not using the kobject API. Its been cleaned up now (I
> hope!)

It refused to apply cleanly on a 2.6.22.14 + CFS v24, only one failure
occured. So I resolved it manually and attached the resulting diff.

My tests with Galaxy 5.9 shows that it still does not work. Although,
the error seems to have changed a bit (see attached dmesg)

> > And, please, we need some documentation for Documenatation/ABI/ on
> > exactly what these sysfs files and tree is for. Please add that now

> > for Linus's tree.
>
> On to it, will send the patch asap.
>
> > confused,
>
> hope i helped (in clearing it :) )

Should this patch eventually be included in?
2.6.25 ?
2.6.24 ?
(-stable 2.6.23 & 2.6.22) || backport CFS v24 -> v25 ?

Thnx,

- vin

Attachment: dmesg.2.6.22.14-CFSv24-FairUserInterfaceBUGfix
Description: dmesg.2.6.22.14-CFSv24-FairUserInterfaceBUGfix

Attachment: FairUserInterface-BugFix.patch
Description: FairUserInterface-BugFix.patch