From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mtagate7.uk.ibm.com (mtagate7.uk.ibm.com [195.212.29.140]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client CN "mtagate7.uk.ibm.com", Issuer "Equifax" (verified OK)) by ozlabs.org (Postfix) with ESMTPS id DE8A5474C1 for ; Wed, 10 Dec 2008 20:52:40 +1100 (EST) Received: from d06nrmr1407.portsmouth.uk.ibm.com (d06nrmr1407.portsmouth.uk.ibm.com [9.149.38.185]) by mtagate7.uk.ibm.com (8.13.8/8.13.8) with ESMTP id mBA9qZcr533336 for ; Wed, 10 Dec 2008 09:52:35 GMT Received: from d06av01.portsmouth.uk.ibm.com (d06av01.portsmouth.uk.ibm.com [9.149.37.212]) by d06nrmr1407.portsmouth.uk.ibm.com (8.13.8/8.13.8/NCO v9.1) with ESMTP id mBA9qZ8s2543704 for ; Wed, 10 Dec 2008 09:52:35 GMT Received: from d06av01.portsmouth.uk.ibm.com (loopback [127.0.0.1]) by d06av01.portsmouth.uk.ibm.com (8.12.11.20060308/8.13.3) with ESMTP id mBA9qZCs032061 for ; Wed, 10 Dec 2008 09:52:35 GMT Message-ID: <493F915D.9090909@gmx.de> Date: Wed, 10 Dec 2008 10:52:29 +0100 From: Mijo Safradin MIME-Version: 1.0 To: Cliff Wickman Subject: Re: [Re: PATCH] numactl: fix libnuma on big-endian 64-bit systems References: In-Reply-To: Content-Type: text/plain; charset=us-ascii Cc: Lee.Schermerhorn@hp.com, safradin@de.ibm.com, arnd@arndb.de, linuxppc-dev@ozlabs.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cliff Wickman wrote: > linux-numa@vger.kernel.org > > Arnd Bergmann and Mijo Safradin called my attention to problems with > libnuma's read_mask() on a big-endian 64-bit machine. > > In testing that function I found it to be buggy on little-endian and 32-bit > systems as well. > > I propose a function that looks like the below. The patch is farther below. > (Arnd, I did not switch to longs, as that proved problematic. The ascii hex > strings being scanned are 32-bit values.) > > I tested on a 64-bit MIPS system. Could you test on your platform? > Hi Cliff, using the current numactl-2.0.3-rc1.tar.gz, which contains your patch. numademo breaks with "mbind: Invalid argument". Mijo