linux-kernel - Re: [REPOST][TRIVIAL][PATCH 1/1] Fix various section mismatches in 2.6.30-rc7

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <1243582585.3212.6.camel@localhost.localdomain>
Date:	Fri, 29 May 2009 13:06:25 +0530
From:	Jaswinder Singh Rajput <jaswinder@...nel.org>
To:	Pranith Kumar <bobby.prani@...il.com>
Cc:	airlied@...ux.ie, torvalds@...ux-foundation.org,
	LKML <linux-kernel@...r.kernel.org>,
	Jarod Wilson <jarod@...hat.com>, Len Brown <lenb@...nel.org>,
	Andrew Morton <akpm@...ux-foundation.org>,
	drm list <dri-devel@...ts.sf.net>, Ingo Molnar <mingo@...e.hu>,
	Eric Anholt <eric@...olt.net>, mjg@...hat.com,
	Sam Ravnborg <sam@...nborg.org>,
	x86 maintainers <x86@...nel.org>
Subject: Re: [REPOST][TRIVIAL][PATCH 1/1] Fix various section mismatches in
  2.6.30-rc7

Hi Pranith,

On Fri, 2009-05-29 at 12:54 +0530, Pranith Kumar wrote:
> Hello,
> 
> Fix the following section mismatches reported by
> 
> $make CONFIG_DEBUG_SECTION_MISMATCH=y
> 
> For the first warning, intel_lvds_init might get called after module
> has been loaded,
> intel_no_lvds shouldn't be marked __initdata.
> 
> For the others the explanation is in the below warnings.
> 
> WARNING: vmlinux.o(.text+0x25ad83): Section mismatch in reference from
> the function intel_lvds_init() to the variable
> .init.data:intel_no_lvds
> The function intel_lvds_init() references
> the variable __initdata intel_no_lvds.
> This is often because intel_lvds_init lacks a __initdata
> annotation or the annotation of intel_no_lvds is wrong.
> 
> WARNING: vmlinux.o(.text+0x26308b): Section mismatch in reference from
> the function intel_opregion_free() to the function
> .exit.text:acpi_video_exit()
> The function intel_opregion_free() references a function in an exit section.
> Often the function acpi_video_exit() has valid usage outside the exit section
> and the fix is to remove the __exit annotation of acpi_video_exit.
> 
> WARNING: vmlinux.o(__ksymtab+0x75f0): Section mismatch in reference
> from the variable __ksymtab_acpi_video_exit to the function
> .exit.text:acpi_video_exit()
> The symbol acpi_video_exit is exported and annotated __exit
> Fix this by removing the __exit annotation of acpi_video_exit or drop
> the export.
> 
> 
> 
> Thanks,
> Pranith.
> 
> Signed-off-by: D Pranith Kumar <bobby.prani@...il.com>
> Cc: Dave Airlie <airlied@...ux.ie>
> 
> diff -uprN -X linux-2.6.30-rc7-vanilla/Documentation/dontdiff
> linux-2.6.30-rc7-vanilla/drivers/acpi/video.c
> linux-2.6.30-rc7/drivers/acpi/video.c
> --- linux-2.6.30-rc7-vanilla/drivers/acpi/video.c	2009-05-26
> 13:40:14.000000000 +0530
> +++ linux-2.6.30-rc7/drivers/acpi/video.c	2009-05-26 13:27:59.000000000 +0530
> @@ -2334,7 +2334,7 @@ static int __init acpi_video_init(void)
>  	return acpi_video_register();
>  }
> 
> -void __exit acpi_video_exit(void)
> +void acpi_video_exit(void)
>  {
> 
>  	acpi_bus_unregister_driver(&acpi_video_bus);


This patch : [PATCH 1/2 -tip] drm/i915: acpi/video.c fix
section mismatch warning" is already applied in acpi tree by Len Brown.

> diff -uprN -X linux-2.6.30-rc7-vanilla/Documentation/dontdiff
> linux-2.6.30-rc7-vanilla/drivers/gpu/drm/i915/intel_lvds.c
> linux-2.6.30-rc7/drivers/gpu/drm/i915/intel_lvds.c
> --- linux-2.6.30-rc7-vanilla/drivers/gpu/drm/i915/intel_lvds.c	2009-05-26
> 13:40:14.000000000 +0530
> +++ linux-2.6.30-rc7/drivers/gpu/drm/i915/intel_lvds.c	2009-05-26
> 13:25:50.000000000 +0530
> @@ -391,7 +391,7 @@ static int __init intel_no_lvds_dmi_call
>  }
> 
>  /* These systems claim to have LVDS, but really don't */
> -static const struct dmi_system_id __initdata intel_no_lvds[] = {
> +static const struct dmi_system_id intel_no_lvds[] = {
>  	{
>  		.callback = intel_no_lvds_dmi_callback,
>  		.ident = "Apple Mac Mini (Core series)",

This patch is already in queue :

[PATCH 2/2 -tip] drm/i915: intel_lvds.c fix section mismatch warning

Thanks,
--
JSR

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/