From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Jan Beulich" Subject: Re: [PATCH v3 03/15] xen: Relocate mem_event_op domctl and access_op memop into common. Date: Tue, 02 Sep 2014 07:53:00 +0100 Message-ID: <5405856C020000780002FAE4@mail.emea.novell.com> References: <1409581329-2607-1-git-send-email-tklengyel@sec.in.tum.de> <1409581329-2607-4-git-send-email-tklengyel@sec.in.tum.de> <5404B6D9.2020108@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Content-Disposition: inline List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Julien Grall , Tamas K Lengyel Cc: Ian Campbell , Tim Deegan , Ian Jackson , "xen-devel@lists.xen.org" , Stefano Stabellini , Andres Lagar-Cavilla , Daniel De Graaf , Tamas K Lengyel List-Id: xen-devel@lists.xenproject.org >>> On 01.09.14 at 22:51, wrote: > On Mon, Sep 1, 2014 at 8:11 PM, Julien Grall wrote: >> On 01/09/14 10:21, Tamas K Lengyel wrote: >> > diff --git a/xen/common/memory.c b/xen/common/memory.c >> > index cc8a3d0..4e530bf 100644 >> > --- a/xen/common/memory.c >> > +++ b/xen/common/memory.c >> > @@ -25,6 +25,7 @@ >> > #include >> > #include >> > #include >> > +#include >> > #include >> > > @Julien: the problem causing your compile issue is here with the order of > the inclusion of the headers. public/memory.h needs to be included before > xen/mem_access.h, I'll fix it in the next iteration.. But then please by making xen/mem_access.h include the prereq header rather than altering the order here. Jan