Re: [PATCH v6 2/3] rust: io: mem: add a generic iomem abstraction

From: Daniel Almeida
Date: Thu Feb 06 2025 - 10:58:20 EST


Btw, Miguel & others,

IMHO, I think we should write a comment about this somewhere in the docs.

When I first came across this issue myself, it took me a while to understand that
the build_error was actually triggering.

That’s because the result is:

```
ERROR: modpost: "rust_build_error" [rust_platform_uio_driver.ko] undefined!
```

When a symbol is undefined, someone would be within their rights to assume that
something is broken in some KConfig somewhere, like this person did. It specifically
doesn’t tell them that the problem is their own code triggering a build_error because
they are misusing an API.

I know that we can’t really provide a message through build_error itself, hence my
suggestion about the docs.

I can send a patch if you agree, it will prevent this confusion from coming up in the
future.

— Daniel