From mboxrd@z Thu Jan  1 00:00:00 1970
From: Julien Grall <julien.grall@linaro.org>
Subject: Re: [PATCH 5/7] mini-os: Moved arch-specific xenbus
 code under arch
Date: Thu, 05 Jun 2014 15:00:03 +0100
Message-ID: <539077E3.4020408@linaro.org>
References: <538D9161.6090402@linaro.org>	<1401897998-29569-1-git-send-email-talex5@gmail.com>
	<1401897998-29569-6-git-send-email-talex5@gmail.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.bemta4.messagelabs.com ([85.158.143.247])
	by lists.xen.org with esmtp (Exim 4.72)
	(envelope-from <julien.grall@linaro.org>) id 1WsYD9-0005v7-Lg
	for xen-devel@lists.xenproject.org; Thu, 05 Jun 2014 14:00:07 +0000
Received: by mail-we0-f182.google.com with SMTP id t60so1187881wes.27
	for <xen-devel@lists.xenproject.org>;
	Thu, 05 Jun 2014 07:00:05 -0700 (PDT)
In-Reply-To: <1401897998-29569-6-git-send-email-talex5@gmail.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: Thomas Leonard <talex5@gmail.com>, xen-devel@lists.xenproject.org
List-Id: xen-devel@lists.xenproject.org

Hi Thomas,

Thank you for the patch.

On 06/04/2014 05:06 PM, Thomas Leonard wrote:
> From: Karim Raslan <karim.allah.ahmed@gmail.com>
> 
> Signed-off-by: Karim Allah Ahmed <karim.allah.ahmed@gmail.com>
> [talex5@gmail.com: separated from big ARM commit]
> Signed-off-by: Thomas Leonard <talex5@gmail.com>
> ---
>  extras/mini-os/arch/x86/xenbus.c | 10 ++++++++++
>  extras/mini-os/xenbus/xenbus.c   | 17 +++++++++++------
>  2 files changed, 21 insertions(+), 6 deletions(-)
>  create mode 100644 extras/mini-os/arch/x86/xenbus.c
> 
> diff --git a/extras/mini-os/arch/x86/xenbus.c b/extras/mini-os/arch/x86/xenbus.c
> new file mode 100644
> index 0000000..5cda78d
> --- /dev/null
> +++ b/extras/mini-os/arch/x86/xenbus.c
> @@ -0,0 +1,10 @@
> +#include <mini-os/os.h>
> +#include <mini-os/mm.h>
> +#include <mini-os/xmalloc.h>
> +#include <xen/xen.h>
> +#include <xen/io/xs_wire.h>
> +
> +void arch_init_xenbus(struct xenstore_domain_interface **xenstore_buf, uint32_t *store_evtchn) {

The mini-os coding style requests the '{' on a newline for function
declaration.

> +    *xenstore_buf = mfn_to_virt(start_info.store_mfn);
> +    *store_evtchn = start_info.store_evtchn;
> +}
> diff --git a/extras/mini-os/xenbus/xenbus.c b/extras/mini-os/xenbus/xenbus.c
> index 934f23b..d995020 100644
> --- a/extras/mini-os/xenbus/xenbus.c
> +++ b/extras/mini-os/xenbus/xenbus.c
> @@ -27,6 +27,7 @@
>  #include <mini-os/wait.h>
>  #include <xen/io/xs_wire.h>
>  #include <mini-os/spinlock.h>
> +#include <mini-os/hypervisor.h>
>  #include <mini-os/xmalloc.h>
>  
>  #define min(x,y) ({                       \
> @@ -43,6 +44,7 @@
>  #endif
>  
>  static struct xenstore_domain_interface *xenstore_buf;
> +static uint32_t store_evtchn;
>  static DECLARE_WAIT_QUEUE_HEAD(xb_waitq);
>  DECLARE_WAIT_QUEUE_HEAD(xenbus_watch_queue);
>  
> @@ -325,20 +327,23 @@ static int allocate_xenbus_id(void)
>      return o_probe;
>  }
>  
> +void arch_init_xenbus(struct xenstore_domain_interface **xenstore_buf, uint32_t *store_evtchn);
> +

Likewise in the previous patch, this prototype should go in an header.
Maybe be include/xenbus.h?

Regards,

-- 
Julien Grall