All of lore.kernel.org
 help / color / mirror / Atom feed
From: Juergen Gross <jgross@suse.com>
To: Roger Pau Monne <roger.pau@citrix.com>, linux-kernel@vger.kernel.org
Cc: Boris Ostrovsky <boris.ostrovsky@oracle.com>,
	Amit Shah <aams@amazon.com>,
	stable@vger.kernel.org, xen-devel@lists.xenproject.org
Subject: Re: [PATCH] xen/pirq: fix error path cleanup when binding MSIs
Date: Wed, 28 Feb 2018 20:22:01 +0100	[thread overview]
Message-ID: <bede6e55-88d3-3f61-f196-c53212abc66a@suse.com> (raw)
In-Reply-To: <20180228091903.99889-1-roger.pau@citrix.com>

On 28/02/18 10:19, Roger Pau Monne wrote:
> Current cleanup in the error path of xen_bind_pirq_msi_to_irq is
> wrong. First of all there's an off-by-one in the cleanup loop, which
> can lead to unbinding wrong IRQs.
> 
> Secondly IRQs not bound won't be freed, thus leaking IRQ numbers.
> 
> Note that there's no need to differentiate between bound and unbound
> IRQs when freeing them, __unbind_from_irq will deal with both of them
> correctly.
> 
> Fixes: 4892c9b4ada9f9 ("xen: add support for MSI message groups")
> Reported-by: Hooman Mirhadi <mirhadih@amazon.com>
> Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>

Committed to xen/tip for-linus-4.16a


Juergen

  parent reply	other threads:[~2018-02-28 19:22 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-02-28  9:19 [PATCH] xen/pirq: fix error path cleanup when binding MSIs Roger Pau Monne
2018-02-28  9:22 ` Shah, Amit
2018-02-28 14:57   ` Boris Ostrovsky
2018-02-28 14:57   ` Boris Ostrovsky
2018-02-28  9:22 ` Shah, Amit
2018-02-28 19:22 ` Juergen Gross
2018-02-28 19:22 ` Juergen Gross [this message]
2018-03-15 17:07 ` Shah, Amit
2018-03-15 17:07 ` Shah, Amit
2018-02-28  9:19 Roger Pau Monne

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=bede6e55-88d3-3f61-f196-c53212abc66a@suse.com \
    --to=jgross@suse.com \
    --cc=aams@amazon.com \
    --cc=boris.ostrovsky@oracle.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=roger.pau@citrix.com \
    --cc=stable@vger.kernel.org \
    --cc=xen-devel@lists.xenproject.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.