From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ian Campbell <Ian.Campbell@citrix.com>
Subject: Re: [PATCH] xen/arm: domain_vgic_init: Check xzalloc_*
	return
Date: Fri, 21 Mar 2014 12:37:44 +0000
Message-ID: <1395405464.19839.27.camel@kazak.uk.xensource.com>
References: <1395323486-9891-1-git-send-email-julien.grall@linaro.org>
	<1395404371.19839.14.camel@kazak.uk.xensource.com>
	<532C3EAB02000078001268B2@nat28.tlf.novell.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
Received: from mail6.bemta3.messagelabs.com ([195.245.230.39])
	by lists.xen.org with esmtp (Exim 4.72)
	(envelope-from <Ian.Campbell@citrix.com>) id 1WQyhp-00058c-H7
	for xen-devel@lists.xenproject.org; Fri, 21 Mar 2014 12:37:49 +0000
In-Reply-To: <532C3EAB02000078001268B2@nat28.tlf.novell.com>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Jan Beulich <JBeulich@suse.com>, Ian Jackson <Ian.Jackson@eu.citrix.com>
Cc: xen-devel@lists.xenproject.org, Julien Grall <julien.grall@linaro.org>, stefano.stabellini@citrix.com, tim@xen.org
List-Id: xen-devel@lists.xenproject.org

On Fri, 2014-03-21 at 12:29 +0000, Jan Beulich wrote:
> >>> On 21.03.14 at 13:19, Ian Campbell <Ian.Campbell@citrix.com> wrote:
> > On Thu, 2014-03-20 at 13:51 +0000, Julien Grall wrote:
> >> The allocations for shared_irqs and pending_irqs are not checked and use
> >> later. This may lead to a Xen segfault if the hypervisor run out of memory.
> >> 
> >> Signed-off-by: Julien Grall <julien.grall@linaro.org>
> > 
> > Acked + applied.
> > 
> >> ---
> >>     This patch should be backported to Xen 4.4
> > 
> > I've added it to my notes, but also CCing Jan since I guess he will be
> > the one to actually do it.
> 
> That's something we need to discuss I think:

I suspected it might be ;-)

>  Since I'm not actively
> using the ARM port, I'm not sure we wouldn't introduce unnecessary
> risks if I did the backports. Therefore I'd like to put  up for discussion
> whether one of the ARM maintainers wouldn't be in a better position
> to do this now that we need to do backports there - both in terms of
> tracking what needs backporting as well as actually applying them.

I'd be happy to do both of these things.

Ian, For ARM only toolstack patches how would you prefer to proceed? I'm
happy either way.

Ian.