Re: [stable] PROBLEM: [BISECTED] 2.6.35.5 xen domU panics just after the boot

Greg KH
2010-09-28T18:28:47Z

On Mon, Sep 27, 2010 at 11:08:43AM -0700, Jeremy Fitzhardinge wrote:
> On 09/26/2010 11:13 PM, Greg KH wrote:
> > On Sun, Sep 26, 2010 at 10:40:44PM -0400, Kyle McMartin wrote:
> >> On Thu, Sep 23, 2010 at 04:23:55PM +0100, James Dingwall wrote:
> >>>> I was experiencing the same immediate crash with a null pointer
> >>>> dereference (log below) on boot with 2.6.35.5. Reverting
> >>>> fb412a178502dc498430723b082a932f797e4763 also resolved the problem for
> >>>> me.
> >>> Checking the diff from this commit it looks like there is a typo in the patch. With the following (probably mangled by my client) I get 2.6.35.5 to boot. Suggest it should be a candidate for .32 and .35 stable trees.
> >>>
> >> Greg, a spelling mistake cropped into this patch during 2.6.35.5 that
> >> James here points out. Adding you to CC since it seems to have missed
> >> stable 2.6.35.6...
> >>
> >> Was in xen-use-percpu-interrupts-for-ipis-and-virqs.patch.
> > I don't understand, where did the error come from? Was it in a patch I
> > applied, or did I mess up some patch? Is the same problem upstream, and
> > if not, why not?
>
> I don't know. It looks like an 'x' was deleted due to an editor
> fat-finger or something. The original upstream patches were cc:d to
> stable, so I think you would have got them that way (ie, I don't think I
> sent them to you specifically). Is there an opportunity in your
> workflow where this error could have crept in? The patches should have
> been a pretty clean 'git cherry-pick' with no conflicts.

Sorry, I don't know what happened, using my scripts, it does drop to an
editor so I can add the "signed-off-by" line, so perhaps I accidentally
did this in there.

It's now in the -stable queue and I'll push out new releases in a few
hours.

thanks,

greg k-h