From mboxrd@z Thu Jan 1 00:00:00 1970 From: sboyd@codeaurora.org (Stephen Boyd) Date: Mon, 20 Jan 2014 17:56:24 -0800 Subject: [PATCH] ARM: asm: __und_usr_thumb need byteswap instructions in BE case In-Reply-To: <20140114090518.GA2222@e103592.cambridge.arm.com> References: <1389652251-18298-1-git-send-email-victor.kamensky@linaro.org> <20140114090518.GA2222@e103592.cambridge.arm.com> Message-ID: <52DDD3C8.7080306@codeaurora.org> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 01/14/14 01:05, Dave Martin wrote: > On Mon, Jan 13, 2014 at 02:30:51PM -0800, Victor Kamensky wrote: >> __und_usr_thumb function deals with thumb2 opcodes. In case of BE >> image, it needs to byteswap half word thumb2 encoded instructions >> before further processing them. >> >> Without this fix BE image user-land thread executing first VFP >> instruction encoded in thumb2 fails with SIGILL, because kernel >> does not recognize instruction and does not enable VFP. >> >> Reported-by: Corey Melton >> Signed-off-by: Victor Kamensky > Good spot. This looks like the correct fix to me. > > Acked-by: Dave Martin > Can you please send this to the patch tracker? Feel free to add Tested-by: Stephen Boyd -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation