Re: [PATCH 3/5] tty/8250_dw: Add support for OCTEON UARTS.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 06/19/2013 11:52 AM, Arnd Bergmann wrote:
On Wednesday 19 June 2013, David Daney wrote:
On 06/19/2013 03:01 AM, Arnd Bergmann wrote:

It's also wrong to use the
__raw_* variant, which is not guaranteed to be atomic and is not
endian-safe.

We do runtime probing and only use this function on platforms where it
is appropriate, so atomicity is not an issue.  As for endianess, I used
the __raw_ variant precisely because it is correct for both big and
little endian kernels.

You don't know what the compiler turns a __raw_writeq into, it could
always to eight byte wise stores, that's why typically writeq is
an inline assembly while __raw_writeq is just a pointer dereference.

Well, I do know that for the cases of interest, it will be a single load or store, but it is moot, as I rewrote that part.


__raw_* never do endian swaps,

Yes, I know that.

so it will be wrong on either big-endian
CPUs or on little-endian CPUs, depending on what the MMIO register
needs.

Please see the instruction set reference manual (MD00087-2B-MIPS64BIS-AFP-03.51 or similar) available at:

http://www.mips.com/products/architectures/mips64/#specifications

... for why you are mistaken. Pay particular attention to the low order address bit scrambling on narrow loads and stores and how this results in uniform (not affected by processor endian mode) load and store results for aligned 64-bit accesses. In effect, it is magic, and __raw_writeq yields correct results in both big and little endian modes of operation.

David Daney.


--
To unsubscribe from this list: send the line "unsubscribe linux-serial" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux PPP]     [Linux FS]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Linmodem]     [Device Mapper]     [Linux Kernel for ARM]

  Powered by Linux