From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mtagate3.de.ibm.com ([195.212.29.152]:37229 "EHLO mtagate3.de.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1761193AbXGKUaD (ORCPT ); Wed, 11 Jul 2007 16:30:03 -0400 Subject: Re: fallocate-implementation-on-i86-x86_64-and-powerpc.patch From: Martin Schwidefsky Reply-To: schwidefsky@de.ibm.com In-Reply-To: <20070711184701.GA9035@osiris.ibm.com> References: <20070710013152.ef2cd200.akpm@linux-foundation.org> <20070710115251.GG2343@thunk.org> <20070710101558.cea7aab1.akpm@linux-foundation.org> <4693C580.2000506@garzik.org> <18068.5612.100418.782729@cargo.ozlabs.ibm.com> <20070710171650.642881db.akpm@linux-foundation.org> <18068.10601.792839.787025@cargo.ozlabs.ibm.com> <20070711153939.GA19456@thunk.org> <20070711184701.GA9035@osiris.ibm.com> Content-Type: text/plain Date: Wed, 11 Jul 2007 22:32:36 +0200 Message-Id: <1184185957.6471.2.camel@localhost> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-arch-owner@vger.kernel.org To: Heiko Carstens Cc: Theodore Tso , Paul Mackerras , Andrew Morton , Jeff Garzik , linux-kernel@vger.kernel.org, Amit Arora , Andi Kleen , Benjamin Herrenschmidt , Arnd Bergmann , "Luck, Tony" , Mark Fasheh , linux-arch@vger.kernel.org List-ID: On Wed, 2007-07-11 at 20:47 +0200, Heiko Carstens wrote: > > There were a number of folks who preferred having int fd first, and I > > *thought* Amit had gotten agreement from either Martin or Heiko that > > it was ok to do this as an exception, even though it was extra work > > for that arch. But if not, we can try going back the second > > alternative, or even the 6 32-bits args (off_high, off_low, len_high, > > len_low) approach, but I think that drew even more fire. > > The second approach would work for all architectures.. but some people > didn't like (no technical reason) not having fd as first argument. For s390 we would have liked the second approach with the two int's as last arguments since it would avoid the wrapper in the kernel. It does not avoid the wrapper in user space since the call uses 6 register on 31 bit. So the fallocate call need special treatement in glibc so I don't mind that it needs another wrapper in the kernel. > Just go ahead with the current approach. s390 seems to be the only > architecture which suffers from this and I wouldn't like to start this > discussion again. Yes, don't worry about s390 for fallocate, the patch that had been in -mm only had an incorrect system call number. The wrapper is fine. -- blue skies, Martin. "Reality continues to ruin my life." - Calvin.