Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757343Ab2BHQz2 (ORCPT ); Wed, 8 Feb 2012 11:55:28 -0500 Received: from terminus.zytor.com ([198.137.202.10]:48133 "EHLO mail.zytor.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754680Ab2BHQz1 (ORCPT ); Wed, 8 Feb 2012 11:55:27 -0500 Message-ID: <4F32A8F1.1080001@zytor.com> Date: Wed, 08 Feb 2012 08:55:13 -0800 From: "H. Peter Anvin" User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:9.0) Gecko/20111222 Thunderbird/9.0 MIME-Version: 1.0 To: Martin Schwidefsky CC: linux-arch@vger.kernel.org, linux-kernel@vger.kernel.org, Linus Torvalds , Heiko Carstens , linux390@de.ibm.com Subject: Re: [PATCH 16/21] s390: Use generic posix_types.h References: <1328677745-20121-1-git-send-email-hpa@zytor.com> <1328677745-20121-17-git-send-email-hpa@zytor.com> <20120208100445.2e527a4e@de.ibm.com> In-Reply-To: <20120208100445.2e527a4e@de.ibm.com> X-Enigmail-Version: 1.3.4 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1517 Lines: 41 On 02/08/2012 01:04 AM, Martin Schwidefsky wrote: > On Tue, 7 Feb 2012 21:09:00 -0800 > "H. Peter Anvin" wrote: > >> From: "H. Peter Anvin" >> >> Change the s390 architecture to use . >> >> Signed-off-by: H. Peter Anvin >> Cc: Martin Schwidefsky >> Cc: Heiko Carstens >> Cc: >> --- >> arch/s390/include/asm/posix_types.h | 70 +++++------------------------------ >> 1 files changed, 10 insertions(+), 60 deletions(-) > > I carefully checked the conversion and came to the conclusion that the new type > definitions are equivalent to the old ones. The code compiles fine with the > patches applied and the kernel boots. > What I don't like is the conversion of __FD_ZERO. The new definition creates > this code sequence: > It shouldn't matter: a. __FD_ZERO seems to actually be unused in the current kernel. b. The last patch in the series deletes the generic __FD_ZERO and replaces it with memset(). If there are any magic optimizations an architecture wants to do, they belong in memset(). -hpa -- H. Peter Anvin, Intel Open Source Technology Center I work for Intel. I don't speak on their behalf. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/