On 10/09/2015 19:15, Stefano Stabellini wrote:
> +
> + switch (reg->size) {
> + case 1: rc = xen_host_pci_get_byte(&s->real_device, offset, (uint8_t
> *)&val);A bit ugly, and it relies on the host being little endian. > + break; > + case 2: rc = xen_host_pci_get_word(&s->real_device, offset, > (uint16_t *)&val); Same here. > + break; > + case 4: rc = xen_host_pci_get_long(&s->real_device, offset, &val); > + break; > + default: assert(1); This should be assert(0) or, better, abort(). Paolo
