<feed xmlns='http://www.w3.org/2005/Atom'>
<title>linux.git/arch/x86/include/uapi/asm/setup_data.h, branch v6.9</title>
<subtitle>Linux kernel source tree</subtitle>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/'/>
<entry>
<title>x86/setup: Move UAPI setup structures into setup_data.h</title>
<updated>2024-01-30T14:17:06+00:00</updated>
<author>
<name>Thomas Zimmermann</name>
<email>tzimmermann@suse.de</email>
</author>
<published>2024-01-12T09:44:36+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=efd7def00406ac57400501cdc76d0d95d4691927'/>
<id>efd7def00406ac57400501cdc76d0d95d4691927</id>
<content type='text'>
The type definition of struct pci_setup_rom in &lt;asm/pci.h&gt; requires
struct setup_data from &lt;asm/bootparam.h&gt;. Many drivers include
&lt;linux/pci.h&gt;, but do not use boot parameters. Changes to bootparam.h or
its included header files could easily trigger a large, unnecessary
rebuild of the kernel.

Moving struct setup_data and related code into its own header file
avoids including &lt;asm/bootparam.h&gt; in &lt;asm/pci.h&gt;. Instead include the
new header &lt;asm/screen_data.h&gt; and remove the include statement for
x86_init.h, which is unnecessary but pulls in bootparams.h.

Suggested-by: Ard Biesheuvel &lt;ardb@kernel.org&gt;
Signed-off-by: Thomas Zimmermann &lt;tzimmermann@suse.de&gt;
Signed-off-by: Borislav Petkov (AMD) &lt;bp@alien8.de&gt;
Reviewed-by: Ard Biesheuvel &lt;ardb@kernel.org&gt;
Link: https://lore.kernel.org/r/20240112095000.8952-2-tzimmermann@suse.de
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The type definition of struct pci_setup_rom in &lt;asm/pci.h&gt; requires
struct setup_data from &lt;asm/bootparam.h&gt;. Many drivers include
&lt;linux/pci.h&gt;, but do not use boot parameters. Changes to bootparam.h or
its included header files could easily trigger a large, unnecessary
rebuild of the kernel.

Moving struct setup_data and related code into its own header file
avoids including &lt;asm/bootparam.h&gt; in &lt;asm/pci.h&gt;. Instead include the
new header &lt;asm/screen_data.h&gt; and remove the include statement for
x86_init.h, which is unnecessary but pulls in bootparams.h.

Suggested-by: Ard Biesheuvel &lt;ardb@kernel.org&gt;
Signed-off-by: Thomas Zimmermann &lt;tzimmermann@suse.de&gt;
Signed-off-by: Borislav Petkov (AMD) &lt;bp@alien8.de&gt;
Reviewed-by: Ard Biesheuvel &lt;ardb@kernel.org&gt;
Link: https://lore.kernel.org/r/20240112095000.8952-2-tzimmermann@suse.de
</pre>
</div>
</content>
</entry>
</feed>
