[PATCH 1/2] ARM: copypage-fa: add kto and kfrom to input operands list
From: linux@armlinux.org.uk (Russell King - ARM Linux)
Date: 2018-10-16 08:44:36
Also in:
lkml
On Tue, Oct 16, 2018 at 10:00:19AM +0200, Linus Walleij wrote:
On Tue, Oct 16, 2018 at 12:16 AM Stefan Agner [off-list ref] wrote:quoted
When functions incoming parameters are not in input operands list gcc 4.5 does not load the parameters into registers before calling this function but the inline assembly assumes valid addresses inside this function. This breaks the code because r0 and r1 are invalid when execution enters v4wb_copy_user_page () Also the constant needs to be used as third input operand so account for that as well. This fixes copypage-fa.c what has previously done before for the other copypage implementations in commit 9a40ac86152c ("ARM: 6164/1: Add kto and kfrom to input operands list."). Signed-off-by: Stefan Agner <stefan@agner.ch>Please add: Cc: stable at vger.kernel.org
It's not obvious yet whether this is right - it contradicts the GCC manual, but then we have evidence that it's required for some GCC versions where GCC may clone the function, or if the function is used within the same file.
I am on deep waters with ARM assembly, admittedly. So I wanted to ask: OpenWRT has this cache patch: https://github.com/openwrt/openwrt/blob/master/target/linux/gemini/patches-4.14/0001-cache-patch-from-OpenWRT.patch I do not know why (sorry). Do you think that patch is actually a hack to hide the problem fixed with this patch? (OK maybe stupid question but...)
No, it looks to me like a hack to make DMA cache handling "more efficient" by cleaning/invalidating the entire cache when dealing with large streaming buffers. -- RMK's Patch system: http://www.armlinux.org.uk/developer/patches/ FTTC broadband for 0.8mile line in suburbia: sync at 12.1Mbps down 622kbps up According to speedtest.net: 11.9Mbps down 500kbps up