xen-devel
Re: [Xen-devel] Re: [PATCH 2/2] xen/gnt{dev, alloc}: reserve event chann
To: |
Daniel De Graaf <dgdegra@xxxxxxxxxxxxx> |
Subject: |
Re: [Xen-devel] Re: [PATCH 2/2] xen/gnt{dev, alloc}: reserve event channels for notify |
From: |
Ian Campbell <Ian.Campbell@xxxxxxxxxx> |
Date: |
Tue, 25 Oct 2011 21:27:23 +0100 |
Cc: |
"Keir \(Xen.org\)" <keir@xxxxxxx>, "jeremy@xxxxxxxx" <jeremy@xxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>, Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>, Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx> |
Delivery-date: |
Tue, 25 Oct 2011 13:28:19 -0700 |
Envelope-to: |
www-data@xxxxxxxxxxxxxxxxxxx |
In-reply-to: |
<4EA710EB.3030809@xxxxxxxxxxxxx> |
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/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe> |
List-unsubscribe: |
<http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe> |
Organization: |
Citrix Systems, Inc. |
References: |
<1316207684-19860-1-git-send-email-dgdegra@xxxxxxxxxxxxx> <1318971851-12809-1-git-send-email-dgdegra@xxxxxxxxxxxxx> <1318971851-12809-3-git-send-email-dgdegra@xxxxxxxxxxxxx> <20111024205708.GE2441@xxxxxxxxxxxxxxxxxxx> <4EA5E024.7040708@xxxxxxxxxxxxx> <20111025190231.GD10062@xxxxxxxxxxxxxxxxxxx> <4EA710EB.3030809@xxxxxxxxxxxxx> |
Sender: |
xen-devel-bounces@xxxxxxxxxxxxxxxxxxx |
On Tue, 2011-10-25 at 20:41 +0100, Daniel De Graaf wrote:
>
> >> Hmm. Perhaps have a magic value for refcount (-1?) that indicates
> evtchn_get is not
> >> available. That would become the default value of refcnt, and
> evtchn.c would then
> >> use evtchn_make_refcounted() to change the refcount to 1 and allow
> _get/_put to work.
> >
> > How would that work when the IRQ subsystem (so everything is setup
> in the kernel)
> > gets an event? Would the refcount be for that -1.. oh. You would
> only set
> > the refcnt when the _get/_put calls are made and not when in-kernel
> calls to setup> IRQs are done?
> >
>
> Right. The reference count would be a dual-purpose field indicating if
> the event channel is kernel-internal (value -1) or userspace-visible
> (reference count > 0). New event channels would start out at -1, and
> evtchn.c would change them to 1.
Is there any way that the reference count could be made part of the
datastructures associated with the /dev/xen/evtchn driver instead of the
core evtchn.c stuff? That wouldreduce the chance of current or futures
users getting something wrong.
Ian.
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
|
<Prev in Thread] |
Current Thread |
[Next in Thread>
|
- Re: [Xen-devel] Re: [PATCH 1/2] xen/event: Add reference counting to event channel, (continued)
- [Xen-devel] [PATCH 2/2] xen/gnt{dev, alloc}: reserve event channels for notify, Daniel De Graaf
- [Xen-devel] Re: [PATCH 2/2] xen/gnt{dev, alloc}: reserve event channels for notify, Konrad Rzeszutek Wilk
- [Xen-devel] Re: [PATCH 2/2] xen/gnt{dev, alloc}: reserve event channels for notify, Daniel De Graaf
- [Xen-devel] Re: [PATCH 2/2] xen/gnt{dev, alloc}: reserve event channels for notify, Konrad Rzeszutek Wilk
- [Xen-devel] Re: [PATCH 2/2] xen/gnt{dev, alloc}: reserve event channels for notify, Daniel De Graaf
- [Xen-devel] Re: [PATCH 2/2] xen/gnt{dev, alloc}: reserve event channels for notify, Konrad Rzeszutek Wilk
- Re: [Xen-devel] Re: [PATCH 2/2] xen/gnt{dev, alloc}: reserve event channels for notify,
Ian Campbell <=
- Re: [Xen-devel] Re: [PATCH 2/2] xen/gnt{dev, alloc}: reserve event channels for notify, Daniel De Graaf
- Re: [Xen-devel] Re: [PATCH 2/2] xen/gnt{dev, alloc}: reserve event channels for notify, Ian Campbell
- Re: [Xen-devel] Re: [PATCH 2/2] xen/gnt{dev, alloc}: reserve event channels for notify, Daniel De Graaf
|
|
|