From: Greg KH <greg@kroah.com>
To: Tomas Henzl <thenzl@redhat.com>
Cc: Vinson Lee <vlee@twopensource.com>,
Masoud Sharbiani <msharbiani@twopensource.com>,
stable@vger.kernel.org, Don Brace <Don.Brace@pmcs.com>,
JBottomley@parallels.com, linux-scsi@vger.kernel.org
Subject: Re: Backport of a fix for HPSA (Disabling a disabled device problem during kdump) driver
Date: Mon, 27 Apr 2015 14:55:29 +0200 [thread overview]
Message-ID: <20150427125529.GA18817@kroah.com> (raw)
In-Reply-To: <553E28DC.1040100@redhat.com>
On Mon, Apr 27, 2015 at 02:17:32PM +0200, Tomas Henzl wrote:
> On 04/26/2015 11:27 AM, Greg KH wrote:
> > On Mon, Apr 13, 2015 at 03:18:44PM +0200, Tomas Henzl wrote:
> >> On 04/11/2015 12:45 AM, Vinson Lee wrote:
> >>> On Tue, Jan 27, 2015 at 4:18 PM, Greg KH <greg@kroah.com> wrote:
> >>>> On Tue, Jan 06, 2015 at 05:15:19PM +0100, Tomas Henzl wrote:
> >>>>> On 01/05/2015 07:41 PM, Masoud Sharbiani wrote:
> >>>>>> Dear stable maintainers,
> >>>>>> Can you please backport commitid 132aa220b45d60e9b20def1e9d8be9422eed9616
> >>>>>> (hpsa: refine the pci enable/disable handling) to 3.10 stable (and
> >>>>>> earlier, if applicable)?
> >>>>> Please do not apply this patch isolated from his friend, the
> >>>>> 859c75aba20264d87dd026bab0d0ca3bff385955 hpsa: add missing pci_set_master in kdump path
> >>>>> needs to be applied together with the 132aa220b45d60e9b20def1e9d8be9422eed9616 .
> >>>>>
> >>>>> In addition to that, after the original issue goes away you may notice sometimes
> >>>>> an unhandled irq 16 message, to fix this a patch is posted
> >>>>> here http://www.spinics.net/lists/linux-scsi/msg80316.html
> >>>>> This patch still awaits a maintainers review though.
> >>>>>
> >>>>> Probably the best idea is to wait until the issue is solved completely.
> >>>> I'll wait, when it all gets worked out, please let stable@ know what
> >>>> patches to apply where.
> >>>>
> >>>> thanks,
> >>>>
> >>>> greg k-h
> >>>> --
> >>>> To unsubscribe from this list: send the line "unsubscribe stable" in
> >>>> the body of a message to majordomo@vger.kernel.org
> >>>> More majordomo info at http://vger.kernel.org/majordomo-info.html
> >>> Hi, Thomas.
> >>>
> >>> The unhandled irq 16 issue seems to be fixed by 4.0-rc1 commit "hpsa:
> >>> turn off interrupts when kdump starts".
> >>>
> >>> Are the following patches suitable for stable now?
> >> Yes, I believe they are, just note that
> >> 03741d9 hpsa: fix memory leak in kdump hard reset
> >> is not a part of that group we discussed before, but it may be added
> >> to stable too.
> >>
> >> Cheers,
> >> Tomas
> >>
> >>> 3b74729 hpsa: turn off interrupts when kdump starts
> >>> 03741d9 hpsa: fix memory leak in kdump hard reset
> >>> 859c75a hpsa: add missing pci_set_master in kdump path
> >>> 132aa22 hpsa: refine the pci enable/disable handling
> > So what order, and to what stable tree(s) do you want these applied to?
> > They seem to span a number of kernel versions, so I'm not quite sure
> > what you are wanting me to do.
>
> Please add it in this order
> 1. 132aa220b4 hpsa: refine the pci enable/disable handling
> 2. 859c75aba2 hpsa: add missing pci_set_master in kdump path
> The first two patches create the fix Masoud has asked for originally to port to stable.
> 3. 3b74729878 hpsa: turn off interrupts when kdump starts
> The third should be added too, it fixes an issue made visible by the first patch
>
> The fourth patch
> 03741d956e hpsa: fix memory leak in kdump hard reset
> is completely unrelated to the first group, so it may be added after or before
> or in the same sequence as it is in the mainline - that is before the third patch.
> This patch is tiny error path fix - not sure if important enough for stable.
Which stable tree(s) do you want these applied to? As these "span"
different kernel versions, I have no idea of what to do here.
Please be specific.
thanks,
greg k-h
next prev parent reply other threads:[~2015-04-27 12:55 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <CANcb+4va2cYPtfTwd-Mv2X=cG5vTy=pz_wE3S+-d1-5Z9qSFoA@mail.gmail.com>
[not found] ` <54AC0A17.8020405@redhat.com>
[not found] ` <20150128001837.GE17677@kroah.com>
[not found] ` <CAHTgTXU1GV8JszqJojk9ML7f1yEtERHY0VjrnjX+OMh4Z_=t0g@mail.gmail.com>
[not found] ` <552BC234.4020000@redhat.com>
2015-04-26 9:27 ` Backport of a fix for HPSA (Disabling a disabled device problem during kdump) driver Greg KH
2015-04-27 12:17 ` Tomas Henzl
2015-04-27 12:55 ` Greg KH [this message]
2015-04-27 13:10 ` Tomas Henzl
2015-05-02 17:56 ` Greg KH
2015-05-22 19:19 ` Vinson Lee
2015-05-26 11:02 ` Jiri Slaby
2015-06-15 3:33 ` Zefan Li
2015-06-30 0:09 ` Greg KH
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20150427125529.GA18817@kroah.com \
--to=greg@kroah.com \
--cc=Don.Brace@pmcs.com \
--cc=JBottomley@parallels.com \
--cc=linux-scsi@vger.kernel.org \
--cc=msharbiani@twopensource.com \
--cc=stable@vger.kernel.org \
--cc=thenzl@redhat.com \
--cc=vlee@twopensource.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).