From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1750978AbaEWBHd (ORCPT ); Thu, 22 May 2014 21:07:33 -0400 Received: from mail.active-venture.com ([67.228.131.205]:57980 "EHLO mail.active-venture.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750808AbaEWBHc (ORCPT ); Thu, 22 May 2014 21:07:32 -0400 X-Originating-IP: 108.223.40.66 Message-ID: <537E9F47.5020809@roeck-us.net> Date: Thu, 22 May 2014 18:07:19 -0700 From: Guenter Roeck User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.5.0 MIME-Version: 1.0 To: Linus Torvalds , Miklos Szeredi , Helge Deller CC: Linux Kernel Mailing List , Parisc List , "James E.J. Bottomley" Subject: Re: Linux 3.15-rc6 References: <20140522172032.GA20975@roeck-us.net> In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 05/22/2014 05:43 PM, Linus Torvalds wrote: > Miklos, > > this is from your commit 18e480aa07f78 ("parisc: add renameat2 > syscall") which was acked by Helge Deller: > > On Thu, May 22, 2014 at 10:20 AM, Guenter Roeck wrote: >> >> Unfortunately it appears that the parisc changes cause build failures >> with parisc 64 bit builds. >> >> Building parisc:a500_defconfig ... failed >> Building parisc:generic-64bit_defconfig ... failed >> >> Error log: >> arch/parisc/kernel/built-in.o: In function `sys_call_table': >> (.rodata+0xad0): undefined reference to `compat_sys_renameat2' >> make: *** [vmlinux] Error 1 > > only s390 seems to need a compat wrapper, and s390 is kind of odd in > many respects, so I suspect renameat2 should just use ENTRY_SAME() on > parisc, but without any way to *test* it I won't apply the patch. > > Guenter, I assume that changing the > > ENTRY_COMP(renameat2) > > line in arch/parisc/kernel/syscall_table.S to use ENTRY_SAME() at > least fixes the compile error? > Yes, it fixes the build error for parisc64, and parisc (32 bit) still builds. Guenter