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 for Android: free password hash cracker in your pocket
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:	Sun, 19 Aug 2012 17:23:49 +0200
From:	Andi Kleen <andi@...stfloor.org>
To:	Jan Beulich <jbeulich@...e.com>
Cc:	andi@...stfloor.org, ak@...ux.intel.com,
	linux-kernel@...r.kernel.org, x86@...nel.org,
	akpm@...ux-foundation.org, jmario@...hat.com,
	rusty@...tcorp.com.au, mmarek@...e.cz, linux-kbuild@...r.kernel.org
Subject: Re: [PATCH 59/74] lto: Handle LTO common symbols in module loader

On Sun, Aug 19, 2012 at 09:53:02AM +0100, Jan Beulich wrote:
> >>> Andi Kleen <andi@...stfloor.org> 08/19/12 4:59 AM >>>
> >@@ -1904,6 +1904,10 @@ static int simplify_symbols(struct module *mod, const struct load_info *info)
>  >
> >        switch (sym[i].st_shndx) {
> >        case SHN_COMMON:
> >+            /* Ignore common symbols */
> >+            if (!strncmp(name, "__gnu_lto", 9))
> >+                break;
> >+
> >            /* We compiled with -fno-common.  These are not
> >               supposed to happen.  */
> >            pr_debug("Common symbol: %s\n", name);
> 
> I think it is dangerous to just match the start of the symbol name here -
> this may in the future well lead to ignoring symbols we shouldn't be
> ignoring.
> 
> Also I would think the added comment ought to say "Ignore LTO symbols."
> Otherwise its sort of contradicting the purpose of the case being handled
> here.

Ok maybe should error out. This case only happens with fat LTO when
the LTO step is not actually run.

It used to happen because old versions of this patchkit 
didn't correctly LTO modules

I'll change it to error out. The reason for the prefix was that
there is a __gnu_lto_vXXX and the version number could change.

Thanks for the reviewws.

-Andi


-- 
ak@...ux.intel.com -- Speaking for myself only.
--
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/

Powered by blists - more mailing lists