On Mon, Dec 5, 2011 at 8:59 AM, Jason Wang <jasowang@xxxxxxxxxx> wrote: > +static int virtnet_set_fd(struct net_device *dev, u32 pfn) > +{ > + struct virtnet_info *vi = netdev_priv(dev); > + struct virtio_device *vdev = vi->vdev; > + > + if (virtio_has_feature(vdev, VIRTIO_NET_F_HOST_FD)) { > + vdev->config->set(vdev, > + offsetof(struct virtio_net_config_fd, addr), > + &pfn, sizeof(u32)); Please use the virtio model (i.e. virtqueues) instead of shared memory. Mapping a page breaks the virtio abstraction. Stefan -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html