rust: replace <linux/module.h> with <linux/export.h> in rust/exports.c

Message ID 20231124142617.713096-1-masahiroy@kernel.org
State New
Headers
Series rust: replace <linux/module.h> with <linux/export.h> in rust/exports.c |

Commit Message

Masahiro Yamada Nov. 24, 2023, 2:26 p.m. UTC
  <linux/export.h> is the right header to include for using
EXPORT_SYMBOL_GPL. <linux/module.h> includes much more bloat.

Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
---

 rust/exports.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Miguel Ojeda Nov. 24, 2023, 2:36 p.m. UTC | #1
On Fri, Nov 24, 2023 at 3:26 PM Masahiro Yamada <masahiroy@kernel.org> wrote:
>
> <linux/export.h> is the right header to include for using
> EXPORT_SYMBOL_GPL. <linux/module.h> includes much more bloat.
>
> Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>

Good catch, thanks!

Cheers,
Miguel
  
Martin Rodriguez Reboredo Nov. 24, 2023, 2:54 p.m. UTC | #2
On 11/24/23 11:26, Masahiro Yamada wrote:
> <linux/export.h> is the right header to include for using
> EXPORT_SYMBOL_GPL. <linux/module.h> includes much more bloat.
> 
> Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
> ---
> [...]
>   
> -#include <linux/module.h>
> +#include <linux/export.h>
>   
> [...]

I'll use this in my crates patch. Regardless, LGTM.

Reviewed-by: Martin Rodriguez Reboredo <yakoyoku@gmail.com>
  
Miguel Ojeda Dec. 13, 2023, 6:43 p.m. UTC | #3
On Fri, Nov 24, 2023 at 3:26 PM Masahiro Yamada <masahiroy@kernel.org> wrote:
>
> <linux/export.h> is the right header to include for using
> EXPORT_SYMBOL_GPL. <linux/module.h> includes much more bloat.
>
> Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>

Applied to `rust-next`.

Thanks everyone!

Cheers,
Miguel
  

Patch

diff --git a/rust/exports.c b/rust/exports.c
index 83e2a7070cae..3803c21d1403 100644
--- a/rust/exports.c
+++ b/rust/exports.c
@@ -11,7 +11,7 @@ 
  * accidentally exposed.
  */
 
-#include <linux/module.h>
+#include <linux/export.h>
 
 #define EXPORT_SYMBOL_RUST_GPL(sym) extern int sym; EXPORT_SYMBOL_GPL(sym)