|   | 
      | 
  
  
      | 
      | 
  
 
     | 
    | 
  
  
     | 
    | 
  
  
    |   | 
      | 
  
  
    | 
         
xen-devel
[Xen-devel] Re: [patch 21/26] Xen-paravirt_ops: Use the hvc console	infr
 
| 
To:  | 
Jeremy Fitzhardinge <jeremy@xxxxxxxx> | 
 
| 
Subject:  | 
[Xen-devel] Re: [patch 21/26] Xen-paravirt_ops: Use the hvc console	infrastructure for Xen console | 
 
| 
From:  | 
Ingo Molnar <mingo@xxxxxxx> | 
 
| 
Date:  | 
Fri, 16 Mar 2007 09:54:51 +0100 | 
 
| 
Cc:  | 
Zachary Amsden <zach@xxxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxx,	virtualization@xxxxxxxxxxxxxx, Rusty Russell <rusty@xxxxxxxxxxxxxxx>,	linux-kernel@xxxxxxxxxxxxxxx, Chris Wright <chrisw@xxxxxxxxxxxx>,	Andi Kleen <ak@xxxxxx>, Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> | 
 
| 
Delivery-date:  | 
Fri, 16 Mar 2007 01:54:38 -0700 | 
 
| 
Envelope-to:  | 
www-data@xxxxxxxxxxxxxxxxxx | 
 
| 
In-reply-to:  | 
<20070301232528.926984984@xxxxxxxx> | 
 
| 
List-help:  | 
<mailto:xen-devel-request@lists.xensource.com?subject=help> | 
 
| 
List-id:  | 
Xen developer discussion <xen-devel.lists.xensource.com> | 
 
| 
List-post:  | 
<mailto:xen-devel@lists.xensource.com> | 
 
| 
List-subscribe:  | 
<http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>,	<mailto:xen-devel-request@lists.xensource.com?subject=subscribe> | 
 
| 
List-unsubscribe:  | 
<http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>,	<mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe> | 
 
| 
References:  | 
<20070301232443.195603797@xxxxxxxx>	<20070301232528.926984984@xxxxxxxx> | 
 
| 
Sender:  | 
xen-devel-bounces@xxxxxxxxxxxxxxxxxxx | 
 
| 
User-agent:  | 
Mutt/1.4.2.2i | 
 
 
 
* Jeremy Fitzhardinge <jeremy@xxxxxxxx> wrote:
> Implement a Xen back-end for hvc console.
> +     cons = intf->out_cons;
> +     prod = intf->out_prod;
> +     mb();
> +     BUG_ON((prod - cons) > sizeof(intf->out));
> +
> +     while ((sent < len) && ((prod - cons) < sizeof(intf->out)))
> +             intf->out[MASK_XENCONS_IDX(prod++, intf->out)] = data[sent++];
> +
> +     wmb();
> +     intf->out_prod = prod;
> +     prod = intf->in_prod;
> +     mb();
> +     BUG_ON((prod - cons) > sizeof(intf->in));
such mb()'s are typically a sign of "i have no clear idea what SMP 
serialization rules apply here, but something is needed because 
otherwise it breaks" ?
        Ingo
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
 |   
 
 | 
    | 
  
  
    |   | 
    |