Re: help installing the correct version of bindgen

[Miguel Ojeda <miguel.ojeda.sandonis@gmail.com>]

On Wed, May 25, 2022 at 12:28 AM Nick Desaulniers
<ndesaulniers@google.com> wrote:
>
> There is a little bit of error handling; for my issue most of the
> error message gets swallowed as shown above.  With the below diff:

Yeah, that is what I meant by having a message, or as you did, just
call it and let it print its own error. Thanks!

> So it looks like the error is not fixed in 1.2.0 release of clang-sys.
> Left more comments in
> https://github.com/KyleMayes/clang-sys/issues/138.

Thanks for following it up.

> I'd like to find out why dependencies are borked first, before
> resorting to env vars.

Definitely, but I suggested because it is good to know that solves it
so that there is not something else going on.

> does seem to work though.

Great -- thanks for trying it.

> error[E0063]: missing field `uring_cmd` in initializer of
> `bindings_raw::file_operations`
> ...
> better than nothing. :) (I just reapplied your series on top of
> a6b450573b912316ad36262bfc70e7c3870c56d1)

Yeah, a few hours ago ee692a21e9bf ("fs,io_uring: add infrastructure
for uring-cmd") got merged, which added a field to `struct
file_operations` so we need changes on the Rust side too.

If you try on top of v5.18 it should work, or if you apply a diff like
(may be wrapped):

```diff
diff --git a/rust/kernel/file.rs b/rust/kernel/file.rs
index e1b3b324bb3d..4c6513702cfa 100644
--- a/rust/kernel/file.rs
+++ b/rust/kernel/file.rs
@@ -534,6 +534,7 @@ impl<A: OpenAdapter<T::OpenData>, T: Operations>
OperationsVtable<A, T> {
         } else {
             None
         },
+        uring_cmd: None,
         write_iter: if T::TO_USE.write_iter {
             Some(Self::write_iter_callback)
         } else {
```

Cheers,
Miguel