rust-vmm / vm-virtio

virtio implementation
Apache License 2.0
366 stars 87 forks source link

Merge virtio-bindings into vm-virtio workspace #158

Closed jiangliu closed 2 years ago

jiangliu commented 2 years ago

The vm-virtio repo has been converted into a repository, so it would be great to merge this crate into the vm-virtio repo.

andreeaflorescu commented 2 years ago

@jiangliu that is a great idea, could you want to take care of that?

lauralt commented 2 years ago

Moved the issue for visibility. After fixing this, we can also get rid of some constant definitions from https://github.com/rust-vmm/vm-virtio/blob/main/crates/virtio-queue/src/defs.rs, and use them from virtio-bindings. There are also more such definitions in the other vm-virtio crates that will no longer need to be defined there, and can be imported from virtio-bindings. We can also fix https://github.com/rust-vmm/vmm-reference/issues/43 afterwards.

drakfangg commented 2 years ago

I will work on it. Divyanshu from slack.

lauralt commented 2 years ago

Fixed in #170