Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Switch to using new xendevicemodel_nr_vcpus() API
This is a stable API replacing the need to use the unstable
xc_domain_getinfo().  This in turn drops the dependency on libxenctrl.

As part of dropping the xenctrl.h include, include xen/memory.h for the
resource mapping constants, and swap xc_evtchn_port_or_error_t for
xenevtchn_port_or_error_t which is provided by xenevtchn.h.

Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
  • Loading branch information
andyhhp committed Mar 12, 2021
1 parent f8c2656 commit fde707c
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 32 deletions.
1 change: 0 additions & 1 deletion Makefile
Expand Up @@ -29,7 +29,6 @@ LDLIBS := -lutil -lrt
endif

LDLIBS += -lxenstore \
-lxenctrl \
-lxenforeignmemory \
-lxendevicemodel \
-lxenevtchn \
Expand Down
44 changes: 13 additions & 31 deletions varstored.c
Expand Up @@ -55,7 +55,7 @@

#include <locale.h>

#include <xenctrl.h>
#include <xen/memory.h>
#include <xen/hvm/ioreq.h>
#include <xenstore.h>
#include <xenevtchn.h>
Expand Down Expand Up @@ -159,7 +159,7 @@ typedef struct varstored_state {
bool ioserv_created;
xenforeignmemory_resource_handle *iores;
shared_iopage_t *iopage;
xc_evtchn_port_or_error_t *ioreq_local_port;
xenevtchn_port_or_error_t *ioreq_local_port;
} varstored_state_t;

static varstored_state_t varstored_state;
Expand Down Expand Up @@ -340,38 +340,12 @@ static bool
varstored_initialize(domid_t domid)
{
int rc, i;
xc_dominfo_t dominfo;
evtchn_port_t port;
struct xs_handle *xsh = NULL;
xc_interface *xch = NULL;
void *addr = NULL;

varstored_state.domid = domid;

xch = xc_interface_open(NULL, NULL, 0);
if (!xch) {
ERR("Failed to open xc_interface handle: %d, %s\n",
errno, strerror(errno));
goto err;
}

rc = xc_domain_getinfo(xch, domid, 1, &dominfo);
if (rc < 0) {
ERR("Failed to get domain info: %d, %s\n", errno, strerror(errno));
goto err;
}
if (dominfo.domid != domid) {
ERR("Domid %u does not match expected %u\n", dominfo.domid, domid);
goto err;
}

varstored_state.vcpus = dominfo.max_vcpu_id + 1;

INFO("%d vCPU(s)\n", varstored_state.vcpus);

xc_interface_close(xch);
xch = NULL;

varstored_state.dmod = xendevicemodel_open(NULL, 0);
if (!varstored_state.dmod) {
ERR("Failed to open xendevicemodel handle: %d, %s\n",
Expand Down Expand Up @@ -399,6 +373,15 @@ varstored_initialize(domid_t domid)
goto err;
}

rc = xendevicemodel_nr_vcpus(varstored_state.dmod, varstored_state.domid,
&varstored_state.vcpus);
if (rc < 0) {
ERR("Failed to query nr_vcpus: %d, %s\n", errno, strerror(errno));
goto err;
}

INFO("%d vCPU(s)\n", varstored_state.vcpus);

rc = xendevicemodel_create_ioreq_server(varstored_state.dmod,
varstored_state.domid, 0,
&varstored_state.ioservid);
Expand Down Expand Up @@ -436,7 +419,7 @@ varstored_initialize(domid_t domid)
goto err;
}

varstored_state.ioreq_local_port = malloc(sizeof (xc_evtchn_port_or_error_t) *
varstored_state.ioreq_local_port = malloc(sizeof (xenevtchn_port_or_error_t) *
varstored_state.vcpus);
if (!varstored_state.ioreq_local_port) {
ERR("Failed to alloc port array: %d, %s\n", errno, strerror(errno));
Expand Down Expand Up @@ -528,7 +511,6 @@ varstored_initialize(domid_t domid)
return true;

err:
xc_interface_close(xch);
xs_close(xsh);
free_auth_data();
return false;
Expand Down Expand Up @@ -560,7 +542,7 @@ varstored_poll_iopage(unsigned int i)
static void
varstored_poll_iopages(void)
{
xc_evtchn_port_or_error_t port;
xenevtchn_port_or_error_t port;
int i;

port = xenevtchn_pending(varstored_state.evth);
Expand Down

0 comments on commit fde707c

Please sign in to comment.