rust-vmm / linux-loader

Linux kernel loader
Apache License 2.0
181 stars 55 forks source link

Fix cargo unaligned warnings #137

Closed dhriti-rajan closed 1 year ago

dhriti-rajan commented 1 year ago

Fixing issue https://github.com/rust-vmm/linux-loader/issues/130.

dhriti-rajan commented 1 year ago

Thanks for the tip! I've regenerated the file with a newer bindgen version, but am still working on cleaning it up since regenerating it seems to have resulted in a couple errors. I'll unmark the PR as a draft once I get it building again and make sure it passes the tests. Sorry for taking a while on this, I'll try to get it wrapped up soon.

alxiord commented 1 year ago

This PR will have to get merged first as the warnings it fixes have since been "promoted" to errors and all PRs are going to fail following the container image update included in #151 (also failing for the same reason).

andreeaflorescu commented 1 year ago

This PR needs a significant rework. I am closing it for now, but @dhriti-rajan if you want to continue working on it, feel free to reopen. Thanks!