From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: Re: [PATCH XEN v7 01/29] stubdom: recurse into tools/include in mk-headers-$(XEN_TARGET_ARCH) rule Date: Tue, 5 Jan 2016 14:52:53 +0000 Message-ID: <1452005573.13361.299.camel@citrix.com> References: <1450269007.4053.48.camel@citrix.com> <1450269096-27598-1-git-send-email-ian.campbell@citrix.com> <1450269096-27598-2-git-send-email-ian.campbell@citrix.com> <20151216221334.GD13723@var.home> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20151216221334.GD13723@var.home> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Samuel Thibault Cc: wei.liu2@citrix.com, ian.jackson@eu.citrix.com, xen-devel@lists.xen.org List-Id: xen-devel@lists.xenproject.org On Wed, 2015-12-16 at 23:13 +0100, Samuel Thibault wrote: > Ian Campbell, on Wed 16 Dec 2015 12:31:08 +0000, wrote: > > ... rather than in the libxc rule. > > > > This puts all the header dependencies in one place and will allow us > > to avoid races when more libraries which need these headers are > > introduced. I observed issues with the xen-foreign/tmp.size file > > getting deleted in parallel with another process trying to use it. > > > > The mini-os links are already created in the > > mk-headers-$(XEN_TARGET_ARCH) target so the other places which do so > > are redundant, in the case of polarssl and vtpmmgr indirectly through > > their eventual dependency on newlib which in turn depends on > > mk-headers-$(XEN_TARGET_ARCH). > > > > Signed-off-by: Ian Campbell > > Cc: samuel.thibault@ens-lyon.org > > Acked-by: Samuel Thibault Thanks, applied.