epoll: use epoll wrapper from vmm-sys-util

rust-vmm / vhost-user-backend

Deprecated repository, code now lives in: https://github.com/rust-vmm/vhost/

Apache License 2.0

18 stars 16 forks source link

epoll: use epoll wrapper from vmm-sys-util #44

Closed jiangliu closed 2 years ago

jiangliu commented 2 years ago

There's a wrapper for epoll from vmm-sys-util, so use the wrapper instead of the epoll crate directly. It may help to ease dependency management.

Signed-off-by: Liu Jiang gerry@linux.alibaba.com

jiangliu commented 2 years ago

Fix #42

slp commented 2 years ago

LGTM, just needs the usual coverage percentage adjustment.

jiangliu commented 2 years ago

LGTM.

Out of curiosity, also for my future PRs, is it ok to update the rust-vmm-ci submodule in the this commit or is it better a separate commit since it's not related to this changes (IIUC)?

I prefer including rust-vmm-ci into other MR if possible, so we could reduce meaningless rust-vmm-ci merge records.