diff options
| author | Michael Brown <mcb30@ipxe.org> | 2015-06-30 17:06:58 +0100 |
|---|---|---|
| committer | Michael Brown <mcb30@ipxe.org> | 2015-06-30 17:18:51 +0100 |
| commit | 211529a7fe97521acdeaf1f785e48ea3ddd26517 (patch) | |
| tree | 2f1cb9d465700d03b10846d2b88b98b8b1d9cfec | |
| parent | a0f60d26f5780196f75f4b14444dfa8d020a5e00 (diff) | |
| download | ipxe-211529a7fe97521acdeaf1f785e48ea3ddd26517.tar.gz | |
[xen] Wait for and clear XenStore event before receiving data
Older, out-of-tree Xen kernel modules (such as those provided with
SuSE Linux Enterprise Server 11) do not clear the leftover "event
pending" bit when opening an event channel. Consequently, no event is
ever delivered to indicate that there is information in the XenStore
ring buffer, and the system hangs shortly after loading the
xen-platform-pci kernel module.
Work around this problem by always waiting for the XenStore event
channel to be signalled, and clearing the event before processing the
received data.
Signed-off-by: Michael Brown <mcb30@ipxe.org>
| -rw-r--r-- | src/arch/x86/include/bits/xen.h | 19 | ||||
| -rw-r--r-- | src/interface/xen/xenstore.c | 4 |
2 files changed, 23 insertions, 0 deletions
diff --git a/src/arch/x86/include/bits/xen.h b/src/arch/x86/include/bits/xen.h index 3433cea1..fc065ea3 100644 --- a/src/arch/x86/include/bits/xen.h +++ b/src/arch/x86/include/bits/xen.h @@ -161,4 +161,23 @@ xen_hypercall_5 ( struct xen_hypervisor *xen, unsigned int hypercall, return retval; } +/** + * Test and clear pending event + * + * @v xen Xen hypervisor + * @v port Event channel port + * @ret pending Event was pending + */ +static inline __attribute__ (( always_inline )) uint8_t +xenevent_pending ( struct xen_hypervisor *xen, evtchn_port_t port ) { + uint8_t pending; + + __asm__ __volatile__ ( "lock btr %2, %0\n\t" + "setc %1\n\t" + : "+m" ( xen->shared->evtchn_pending ), + "=a" ( pending ) + : "Ir" ( port ) ); + return pending; +} + #endif /* _BITS_XEN_H */ diff --git a/src/interface/xen/xenstore.c b/src/interface/xen/xenstore.c index 731af779..23424a92 100644 --- a/src/interface/xen/xenstore.c +++ b/src/interface/xen/xenstore.c @@ -242,6 +242,10 @@ static int xenstore_response ( struct xen_hypervisor *xen, uint32_t req_id, char *string; int rc; + /* Wait for response to become available */ + while ( ! xenevent_pending ( xen, xen->store.port ) ) + cpu_nap(); + /* Receive message header */ xenstore_recv ( xen, &msg, sizeof ( msg ) ); *len = msg.len; |