Re: big endian builds

2016-03-10 Thread Vineet Gupta
On Friday 11 March 2016 12:18 PM, Vineet Gupta wrote: > So for big endian kernel builds, just selecting the CONFIG_CPU_BIG_ENDIAN is > not > sufficient, we also need to manually change the CROSS_COMPILE prefix to > arceb-xxx. > > @Anton, is that what you do for verification builds ! > > @Noam I p

big endian builds

2016-03-10 Thread Vineet Gupta
So for big endian kernel builds, just selecting the CONFIG_CPU_BIG_ENDIAN is not sufficient, we also need to manually change the CROSS_COMPILE prefix to arceb-xxx. @Anton, is that what you do for verification builds ! @Noam I presume your defconfig has CONFIG_CROSS_COMPILE or you have set in env

Re: [PATCH] arc: use little endian accesses

2016-03-10 Thread Vineet Gupta
On Thursday 10 March 2016 01:15 PM, Arnd Bergmann wrote: > On Thursday 10 March 2016, Vineet Gupta wrote: >> On Wednesday 09 March 2016 10:51 PM, Lada Trimasova wrote: >>> Memory access primitives should use cpu_to_le16, cpu_to_le32, le16_to_cpu >>> and le32_to_cpu because it is not really guarante

Re: [PATCH] arc: use little endian accesses

2016-03-10 Thread Noam Camus
> From: Lada Trimasova > Sent: Thursday, March 10, 2016 8:57 PM > And for example, let's see what happens when we use "readl" in function > "serial8250_early_in" in driver/tty/serial/8250. Is your DTS entry includes for serial node following entries: reg-io-width = <4> native-endian; if answe

Re: [PATCH v2 2/9] ARC: [dts] Introduce Timer bindings

2016-03-10 Thread Vineet Gupta
On Tuesday 08 March 2016 05:56 PM, Vineet Gupta wrote: > ARC Timers have historically been probed directly. > As precursor to start probing Timers thru DT introduce these bindings > Note that to keep series bisectable, these bindings are not yet used in > code. > > Cc: Daniel Lezcano > Cc: Rob Her

Re: [PATCH] arc: use little endian accesses

2016-03-10 Thread Arnd Bergmann
On Thursday 10 March 2016, Lada Trimasova wrote: > Driver is 8250, kernel is built for BE arc, nsim option in model > "nsim_isa_big_endian = 1". > > With current "readl" and "writel" implementation for ARC we read word from > memory without any endianess manipulation. So in case of little endian

Re: [PATCH] arc: use little endian accesses

2016-03-10 Thread Lada Trimasova
Hi Vineet, Alexey, Arnd, On Thu, 2016-03-10 at 09:55 +, Vineet Gupta wrote: Can you or Lada provide more details as to exactly what is not working - what driver to be precise 8250 or dw-8250. And where exactly the failure shows up. I want to understand this more Noam told me off list that he h

Re: [PATCH] arc: use little endian accesses

2016-03-10 Thread Vineet Gupta
On Thursday 10 March 2016 01:14 PM, Alexey Brodkin wrote: >>> For example, serial port driver doesn't work when kernel is build for >>> arc big endian architecture. >> Last I tested Big Endian on SDP with 8250 part + 8250 driver it was working >> fine. >> I presume this is the systemC model for de

Re: [PATCH] mm: slub: Ensure that slab_unlock() is atomic

2016-03-10 Thread Peter Zijlstra
On Thu, Mar 10, 2016 at 11:21:21AM +0530, Vineet Gupta wrote: > On Wednesday 09 March 2016 08:21 PM, Peter Zijlstra wrote: > >> But in SLUB: bit_spin_lock() + __bit_spin_unlock() is acceptable ? How so > >> (ignoring the performance thing for discussion sake, which is a side > >> effect of > >> th