[Top] [All Lists]

Re: [PATCH v2 22/22] bitops: remove minix bitops from asm/bitops.h

Subject: Re: [PATCH v2 22/22] bitops: remove minix bitops from asm/bitops.h
From: Akinobu Mita <>
Date: Fri, 22 Oct 2010 16:55:10 +0900
Cc:,, Arnd Bergmann <>, Christoph Hellwig <>, Andrew Morton <>, Greg Ungerer <>, Geert Uytterhoeven <>, Roman Zippel <>, Andreas Schwab <>,, Martin Schwidefsky <>, Heiko Carstens <>,,, Yoshinori Sato <>,, "David S. Miller" <>,, Hirokazu Takata <>,, Ralf Baechle <>,, Paul Mundt <>,, Chris Zankel <>
Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed;; s=gamma; h=domainkey-signature:mime-version:received:received:in-reply-to :references:date:message-id:subject:from:to:cc:content-type :content-transfer-encoding; bh=ms+39lDW7E7Y+Ds0br5rl1DzwcayAzOLRLu/wAELGJE=; b=v1yA0FAVkR2fj8qsNR4MXC18QPAAoBMFo6GF/gDFVh8vofzb3ZLc5jz599efBrrZwu 1S0+mHm2nJretIRQI7A3YnJGcT+17nAcwps3SwiwOrCodg6hTaaaMFVsYCfUAJc7zUyQ DQmG9jO1vjJduTl9KrLM+4jnud+wHFu+cOLTI=
Domainkey-signature: a=rsa-sha1; c=nofws;; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type:content-transfer-encoding; b=mpMmBnSKstx9DLRcTFkn7l5n9pUyXstHmlym37NJ+jQYNOxjCIg+A2Yr72JW50XNt5 Q83T6T8CZZNxSWB/bBJ/YtOVBxD1eny+9lyVPQHv344mXS8RmbSVReoMwTI+9vk1T/2c 1WVHn7Rd+eVEUwOlv7mRJCMdl9uCi2aFUGYZ4=
In-reply-to: <>
Original-recipient: rfc822;
References: <> <> <>
2010/10/22 Michal Simek <>:
> Akinobu Mita wrote:
>> minix bit operations are only used by minix filesystem and useless
>> by other modules. Because byte order of inode and block bitmaps is
>> defferent on each architecture like below:
>> m68k:
>>       big-endian 16bit indexed bitmaps
>> h8300, microblaze, s390, sparc, m68knommu:
>>       big-endian 32 or 64bit indexed bitmaps
> Just one small fix microblaze little endian support is ready for merging
> to mainline which means that microblaze is
> big-endian 32bit  and  little-endian 32bit
>> m32r, mips, sh, xtensa:
>>       big-endian 32 or 64bit indexed bitmaps for big-endian mode
>>       little-endian bitmaps for little-endian mode
>> Others:
>>       little-endian bitmaps
>> In order to move minix bit operations from asm/bitops.h to
>> architecture independent code in minix file system, this provides two
>> config options.
>> CONFIG_MINIX_FS_BIG_ENDIAN_16BIT_INDEXED is only selected by m68k.
>> CONFIG_MINIX_FS_NATIVE_ENDIAN is selected by the architectures which
>> use native byte order bitmaps (h8300, microblaze, s390, sparc,
>> m68knommu, m32r, mips, sh, xtensa).
>> The architectures which always use little-endian bitmaps do not select
>> these options.
> I haven't created any Kconfig option for little/big endian microblaze
> but there should be a little bit different handling for MINIX_FS_NATIVE_ENDIAN
> as you are describing above.
> Anyway I think you don't need to reflect this in your patch because
> we are not using that filesystem and I will write it to my to-do list and
> will fix it later.

If upcomming microblade little-endian mode will use little-endian
bitmaps for minixfs, microblade can continue to select
CONFIG_MINIX_FS_NATIVE_ENDIAN and you don't need to change it.

But if it will use big-endian bitmaps, it may need some extra work
to support it. Becuase there is no little-endian architecture
which uses bit-endian bitmaps for minixfs.

<Prev in Thread] Current Thread [Next in Thread>