Re: [PATCH 0/2] cramfs: Add mount option "swapendian"

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Andi Drebes <lists-receive@...>
Cc: <linux-fsdevel@...>, Christoph Hellwig <hch@...>, Andrew Morton <akpm@...>
Date: Thursday, November 15, 2007 - 4:45 pm

On Thu, 15 Nov 2007, Andi Drebes wrote:

Please don't do it this way.

It would be *much* better to just standardize on one endianness, and be 
done with it. That way there are no config options, no confusion, and the 
code is smaller, simpler, and faster. Because nn unconditional byte swap 
is generally faster than a conditional non-byte-swap!

So can you please just make it little-endian? 

There can't be that many big-endian machines that really care about old 
cramfs images..

		Linus
-
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 0/2] cramfs: support for other endianness, Andi Drebes, (Thu Nov 15, 4:29 pm)
Re: [PATCH 0/2] cramfs: support for other endianness, Andi Drebes, (Thu Nov 15, 4:43 pm)
[PATCH 0/2] cramfs: update README file, Andi Drebes, (Thu Nov 15, 4:37 pm)
[PATCH 0/2] cramfs: Add mount option "swapendian", Andi Drebes, (Thu Nov 15, 4:35 pm)
Re: [PATCH 0/2] cramfs: Add mount option "swapendian", Linus Torvalds, (Thu Nov 15, 4:45 pm)
Re: [PATCH 0/2] cramfs: Add mount option "swapendian", Andi Drebes, (Thu Nov 15, 5:15 pm)
Re: [PATCH 0/2] cramfs: Add mount option "swapendian", Linus Torvalds, (Thu Nov 15, 5:37 pm)
Re: [PATCH 0/2] cramfs: Add mount option "swapendian", Phillip Lougher, (Thu Nov 15, 6:48 pm)
Re: [PATCH 0/2] cramfs: Add mount option "swapendian", Andi Drebes, (Fri Nov 16, 6:28 am)
Re: [PATCH 0/2] cramfs: Add mount option "swapendian", Linus Torvalds, (Fri Nov 16, 11:44 am)
Re: [PATCH 0/2] cramfs: Add mount option "swapendian", Christoph Hellwig, (Thu Nov 15, 4:49 pm)
Re: [PATCH 0/2] cramfs: Add mount option "swapendian", Linus Torvalds, (Thu Nov 15, 4:57 pm)
Re: [PATCH 0/2] cramfs: Add mount option "swapendian", Linus Torvalds, (Thu Nov 15, 4:46 pm)
Re: [PATCH 0/2] cramfs: Add mount option "swapendian", Christoph Hellwig, (Thu Nov 15, 4:51 pm)