linux-kernel - Re: [PATCH v2] ARM: Use udiv/sdiv for __aeabi

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]

Date:	Tue, 12 Nov 2013 09:32:03 -0500 (EST)
From:	Nicolas Pitre <nicolas.pitre@...aro.org>
To:	Ben Dooks <ben.dooks@...ethink.co.uk>
cc:	Russell King - ARM Linux <linux@....linux.org.uk>,
	Måns Rullgård <mans@...sr.com>,
	Stephen Boyd <sboyd@...eaurora.org>,
	linux-kernel@...r.kernel.org,
	Christopher Covington <cov@...eaurora.org>,
	Jean-Christophe PLAGNIOL-VILLARD <plagnioj@...osoft.com>,
	linux-arm-kernel@...ts.infradead.org
Subject: Re: [PATCH v2] ARM: Use udiv/sdiv for __aeabi_{u}idiv library
 functions

On Tue, 12 Nov 2013, Ben Dooks wrote:

> Given these are single instructoins for ARM, is it possible we could
> make a table of all the callers and fix them up when we initialise
> as we do for the SMP/UP case and for page-offset?

Not really.  Calls to those functions are generated by the compiler 
implicitly when a divisor operand is used and therefore we cannot 
annotate those calls.  We'd have to use special accessors everywhere to 
replace the standard division operand (like we do for 64 by 32 bit 
divisions) but I doubt that people would accept that.

You cannot just scan the binary for the appropriate branch opcode either 
as you may turn up false positives in literal pools.

Nicolas
--
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/