From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Cooper Subject: Re: [PATCH 03/16] libelf: add `struct elf_binary*' parameter to elf_load_image Date: Wed, 5 Jun 2013 12:01:04 +0100 Message-ID: <51AF1A70.2060605@citrix.com> References: <1370368803-9436-1-git-send-email-ian.jackson@eu.citrix.com> <1370368803-9436-4-git-send-email-ian.jackson@eu.citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: George Dunlap Cc: "xen-devel@lists.xensource.com" , Ian Jackson , "mattjd@gmail.com" , "security@xen.org" List-Id: xen-devel@lists.xenproject.org On 05/06/13 11:32, George Dunlap wrote: > On Tue, Jun 4, 2013 at 6:59 PM, Ian Jackson wrote: >> The meat of this function is going to need a copy of the elf pointer, >> in forthcoming patches. >> >> No functional change in this patch. >> >> Signed-off-by: Ian Jackson >> Acked-by: Ian Campbell >> Reviewed-by: Konrad Rzeszutek Wilk > Is there any reason not to commit the patches that have been acked and > reviewed, so we can start getting stuff tested sooner (and reduce the > cognitive load on you of having to deal with 16 patches)? > > -George I would certainly appreciate a reduction of cognative load. >>From the series, I believe that patches 1-6 are sufficiently simple and obviously correct. ~Andrew