lkml.org 
[lkml]   [2010]   [Oct]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH 22/22] bitops: remove minix bitops from asm/bitops.h
Date
On Friday 15 October 2010, Akinobu Mita wrote:
> minix bit operations are only used by minix filesystem and useless
> by other modules.

Right.

> This provides new config option CONFIG_MINIX_FS_LITTLE_ENDIAN and
> CONFIG_MINIX_FS_NATIVE_ENDIAN that each architecture selects one of which.
> Then we can remove minix bit operations from asm/bitops.h from all
> architectures by making them minix filesystem local macros.

I would say that any architecture that defines minix bitops as
little-endian is broken and we should not even need the #define.

You have defined these as "native endian":

always LE:
alpha, blackfin, ia64, score, tile, x86

always BE:
h8300, microblaze, s390, sparc

configurable:
m32r, mips, sh, xtensa

The only ones among these that possibly ever cared about mounting minix
file systems on a big-endian kernel are really old sparc and mips systems,
everyone else probably never noticed their mistake.

I'd say let's define the minix bitops as always LE and be done with it.

Arnd


\
 
 \ /
  Last update: 2010-10-15 12:55    [W:0.224 / U:0.096 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site