<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<style type="text/css" style="display:none"><!--P{margin-top:0;margin-bottom:0;} --></style>
</head>
<body dir="ltr" style="font-size:12pt;color:#000000;background-color:#FFFFFF;font-family:Calibri,Arial,Helvetica,sans-serif;">
<p>Hey everyone,</p>
<p><br>
</p>
<p>There are a bunch of PRs opened that would need some attention. Could you please take some time to review them?</p>
<p>Also, if you are one of the people that have submitted a PR, please take some time to respond to comments and update your PRs based on the reviews.<br>
</p>
<p><br>
</p>
<p>In kvm-ioctls:</p>
<p>- helper functions for altering zero sized array structures (like kvm_cpuid2, kvm_msrs):
</p>
<p><a href="https://github.com/rust-vmm/kvm-ioctls/pull/6">https://github.com/rust-vmm/kvm-ioctls/pull/6</a></p>
<p><br>
</p>
<p>- high level documentation for kvm-ioctls (with an x86 example of usage): </p>
<p><a href="https://github.com/rust-vmm/kvm-ioctls/pull/26">https://github.com/rust-vmm/kvm-ioctls/pull/26</a></p>
<p><br>
</p>
<p>- helper function for creating CpuId from kvm_entries:</p>
<p><a href="https://github.com/rust-vmm/kvm-ioctls/pull/29">https://github.com/rust-vmm/kvm-ioctls/pull/29</a><br>
</p>
<p><br>
</p>
<p>In vmm-sys-utils:</p>
<p>- Initial PR that adds helpers (this is blocking for adding new crates depending on sys-util)</p>
<p><a href="https://github.com/rust-vmm/vmm-sys-util/pull/1">https://github.com/rust-vmm/vmm-sys-util/pull/1</a></p>
<p><br>
</p>
<p>In vhost:</p>
<p>- Inital PR that adds the vhost code from Firecracker:</p>
<p><a href="https://github.com/rust-vmm/vhost/pull/2">https://github.com/rust-vmm/vhost/pull/2</a></p>
<p><br>
</p>
<p>In vm-memory:</p>
<p>- Windows Support: </p>
<p><a href="https://github.com/rust-vmm/vm-memory/pull/10">https://github.com/rust-vmm/vm-memory/pull/10</a><br>
</p>
<p><br>
</p>
<p>If you want to be a reviewer for any of them please add yourself as a reviewer. If you don't have permission to do so let me know and I'll fix it.</p>
<p><br>
</p>
<p> This list contains all PRs that don't have at least 2 reviewers already. I believe we agreed that the first PR in a repository (the one that essentially adds a lot of code) will be reviewed by at least 3 people coming from 3 different projects to ensure
 that the crate is usable by all interested parties. For all the others we can fall back to 2 reviews.<br>
</p>
<p><br>
</p>
<p>What I am trying to avoid here is having same people reviewing all PRs because:</p>
<p>1. It takes some time to do reviews</p>
<p>2. Different people might have different views & find different problems in PRs.</p>
<p><br>
</p>
<p>Thanks,</p>
<p>Andreea<br>
</p>
<p></p>

<p><br>
Amazon Development Center (Romania) S.R.L. registered office: 27A Sf. Lazar Street, UBC5, floor 2, Iasi, Iasi County, 700045, Romania. Registered in Romania. Registration number J22/2621/2005.</p>
</body>
</html>