Re: [PATCH] scripts/kernel-doc: Get -export option working again
From: Jonathan Corbet
Date: Tue Dec 10 2024 - 15:58:50 EST
Akira Yokosawa <akiyks@xxxxxxxxx> writes:
> Since commit cdd30ebb1b9f ("module: Convert symbol namespace to string
> literal"), exported symbols marked by EXPORT_SYMBOL_NS(_GPL) are
> ignored by "kernel-doc -export" in fresh build of "make htmldocs".
>
> This is because regex in the perl script for those markers fails to
> match the new signatures:
>
> - EXPORT_SYMBOL_NS(symbol, "ns");
> - EXPORT_SYMBOL_NS_GPL(symbol, "ns");
>
> Update the regex so that it matches quoted string.
>
> Note: Escape sequence of \w is good for C identifiers, but can be
> too strict for quoted strings. Instead, use \S, which matches any
> non-whitespace character, for compatibility with possible extension
> of namespace convention in the future [1].
>
> Fixes: cdd30ebb1b9f ("module: Convert symbol namespace to string literal")
> Link: https://lore.kernel.org/CAK7LNATMufXP0EA6QUE9hBkZMa6vJO6ZiaYuak2AhOrd2nSVKQ@xxxxxxxxxxxxxx/ [1]
> Signed-off-by: Akira Yokosawa <akiyks@xxxxxxxxx>
> Cc: Masahiro Yamada <masahiroy@xxxxxxxxxx>
Ah ... I should have thought of that ... I'm glad you did. I've just
pushed the fix out to linux-next, will send it Linusward in the near
future. Thanks for fixing this,
jon