Re: [PATCH 2/5] powernv:idle: Change return type of pnv_probe_idle_states to int

2017-07-06 Thread Nicholas Piggin
On Wed, 5 Jul 2017 22:08:13 +0530 "Gautham R. Shenoy" wrote: > From: "Gautham R. Shenoy" > > In the current idle initialization code, if there are failures in > pnv_probe_idle_states, then no platform idle state is > enabled. However, since the error is not propagated to the top-level >

Re: [PATCH] mm: make allocation counters per-order

2017-07-06 Thread Christoph Lameter
On Thu, 6 Jul 2017, Roman Gushchin wrote: > +#define PGALLOC_EVENTS_SIZE (MAX_NR_ZONES * MAX_ORDER) > +#define PGALLOC_EVENTS_CUT_SIZE (MAX_NR_ZONES * (MAX_ORDER - 1)) > +#define PGALLOC_FIRST_ZONE (PGALLOC_NORMAL - ZONE_NORMAL) You are significantly increasing the per cpu counters (ZONES *

Re: [git pull] vfs.git part 3

2017-07-06 Thread Al Viro
On Thu, Jul 06, 2017 at 04:48:40PM +0200, Christoph Hellwig wrote: > Just did the whole batch (patch below), but it seems like using a > __bitwise type in SYSCALL_DEFINE* will always give warnings like: > > fs/read_write.c:1095:1: warning: cast to restricted __kernel_rwf_t > > which I'm not

Re: [PATCH 1/3] drm: Widen vblank count to 64 bits. Change vblank time precision to ns

2017-07-06 Thread Keith Packard
Michel Dänzer writes: > Subtle breakage here: vblwait->request.sequence must still get updated > for _DRM_VBLANK_RELATIVE, in case we're interrupted by a signal. Thanks for finding this. I think it might be better to just not modify the request.type field instead, so that on re-entry it gets

Re: [PATCH] integrity: get rid of unneeded initializations in integrity_iint_cache entries

2017-07-06 Thread Serge E. Hallyn
Quoting Jeff Layton (jlay...@kernel.org): > From: Jeff Layton > > The init_once routine memsets the whole object to 0, and then > explicitly sets some of the fields to 0 again. Just remove the explicit > initializations. > > Signed-off-by: Jeff Layton Reviewed-by: Serge Hallyn > --- >

[PATCH][V2] usb: storage: return on error to avoid a null pointer dereference

2017-07-06 Thread Colin King
From: Colin Ian King When us->extra is null the driver is not initialized, however, a later call to osd200_scsi_to_ata is made that dereferences us->extra, causing a null pointer dereference. The code currently detects and reports that the driver is not initialized; add a return to avoid the

Re: [git pull] vfs.git part 3

2017-07-06 Thread Christoph Hellwig
On Thu, Jul 06, 2017 at 04:03:30PM +0100, Al Viro wrote: > #define __SC_CAST(t, a) (__force t) a > > in syscalls.h Hmm. > > > index a2d4a8ac94ca..a04adbc70ddf 100644 > > --- a/include/uapi/linux/aio_abi.h > > +++ b/include/uapi/linux/aio_abi.h > > @@ -28,6 +28,7 @@ > > #define

Re: [PATCH 3/5] powernv:idle: Define idle init function for power8

2017-07-06 Thread Nicholas Piggin
On Wed, 5 Jul 2017 22:08:14 +0530 "Gautham R. Shenoy" wrote: > From: "Gautham R. Shenoy" > > In this patch we define a new function named pnv_power8_idle_init(). > > We move the following code from pnv_init_idle_states() into this newly > defined function. >a) That patches out

Re: [git pull] vfs.git part 3

2017-07-06 Thread Christoph Hellwig
On Thu, Jul 06, 2017 at 04:03:30PM +0100, Al Viro wrote: > On Thu, Jul 06, 2017 at 04:48:40PM +0200, Christoph Hellwig wrote: > > > Just did the whole batch (patch below), but it seems like using a > > __bitwise type in SYSCALL_DEFINE* will always give warnings like: > > > >

Re: [PATCH 1/3] drm: Widen vblank count to 64 bits. Change vblank time precision to ns

2017-07-06 Thread Keith Packard
Michel Dänzer writes: > BTW, this got me thinking that we should probably treat > _DRM_VBLANK_NEXTONMISS the same way, i.e. clear the flag after updating > vblwait->request.sequence. Otherwise there could theoretically (though > unlikely) be an infinite loop: I was thinking that we should just

Re: [PATCH v2 02/10] MIPS: ranchu: Add Goldfish RTC driver

2017-07-06 Thread Alexandre Belloni
On 06/07/2017 at 13:25:09 +, Miodrag Dinic wrote: > > > +static int goldfish_rtc_read_time(struct device *dev, struct rtc_time > > > *tm) > > > +{ > > > + u64 time; > > > + u64 time_low; > > > + u64 time_high; > > > + u64 time_high_prev; > > > + > > > + struct goldfish_rtc

Re: [PATCH 4/5] powernv:idle: Move initialization of sibling pacas to pnv_alloc_idle_core_states

2017-07-06 Thread Nicholas Piggin
On Wed, 5 Jul 2017 22:08:15 +0530 "Gautham R. Shenoy" wrote: > From: "Gautham R. Shenoy" > > On POWER9 DD1, in order to get around a hardware issue, we store in > every CPU thread's paca the paca pointers of all its siblings. > > Move this code into pnv_alloc_idle_core_states() soon after

Re: [PATCH 0/2] ARM: add missing includes to header files

2017-07-06 Thread Russell King - ARM Linux
On Tue, Jul 04, 2017 at 03:25:40PM +0200, Johan Hovold wrote: > Add two missing include directives to two header files to avoid > compile breakage when the required symbols have not already been > defined. > > The missing SCU include broke compilation of the (not-yet-merged) > suspend support for

Re: [PATCH v2 0/9] Remove spin_unlock_wait()

2017-07-06 Thread Paul E. McKenney
On Thu, Jul 06, 2017 at 02:12:24PM +, David Laight wrote: > From: Paul E. McKenney > > Sent: 06 July 2017 00:30 > > There is no agreed-upon definition of spin_unlock_wait()'s semantics, > > and it appears that all callers could do just as well with a lock/unlock > > pair. This series

Re: [PATCH 2/2] afs: Add metadata xattrs

2017-07-06 Thread Christoph Hellwig
NAK. Don't overload xattrs with magic behavior just to avoid the need to do proper syscalls or ioctls. This makes them harder to discover, audit and security fix. On Thu, Jul 06, 2017 at 03:50:27PM +0100, David Howells wrote: > Add xattrs to allow the user to get/set metadata in lieu of having

Re: [GIT PULL] USB/PHY patches for 4.13-rc1

2017-07-06 Thread Babu Moger
On 7/6/2017 9:33 AM, Greg KH wrote: On Thu, Jul 06, 2017 at 09:28:06AM -0500, Babu Moger wrote: On 7/6/2017 3:24 AM, Max Filippov wrote: Hi Babu, On Tue, Jul 4, 2017 at 10:19 AM, Babu Moger wrote: Max, Do you have any concerns about xtensa? no, not ATM. I still haven't got a chance to

Re: [RFC][PATCH] exec: Use init rlimits for setuid exec

2017-07-06 Thread Andy Lutomirski
On Thu, Jul 6, 2017 at 5:45 AM, Eric W. Biederman wrote: > > How this is handled elsewhere in the code is to put the new values in > bprm. Putting new rlimits in bprm and changing them in flush_old_exec > or or setup_new_exec seems very sensible. It also allows for them to be > accessed before

Re: [PATCH v2 6/9] clk: at91: clk-generated: make gclk determine audio_pll rate

2017-07-06 Thread Nicolas Ferre
On 04/07/2017 at 13:59, Quentin Schulz wrote: > This allows gclk to determine audio_pll rate and set the parent rate > accordingly. > > However, there are multiple children clocks that could technically > change the rate of audio_pll (via gck). With the rate locking, the first > consumer to

Re: [RFC v5 09/11] mm: Try spin lock in speculative path

2017-07-06 Thread Laurent Dufour
On 06/07/2017 16:48, Peter Zijlstra wrote: > On Thu, Jul 06, 2017 at 03:46:59PM +0200, Laurent Dufour wrote: >> On 05/07/2017 20:50, Peter Zijlstra wrote: >>> On Fri, Jun 16, 2017 at 07:52:33PM +0200, Laurent Dufour wrote: @@ -2294,8 +2295,19 @@ static bool pte_map_lock(struct vm_fault *vmf)

Re: [PATCH 5/5] powernv:idle: Disable LOSE_FULL_CONTEXT states when stop-api fails.

2017-07-06 Thread Nicholas Piggin
On Wed, 5 Jul 2017 22:08:16 +0530 "Gautham R. Shenoy" wrote: > From: "Gautham R. Shenoy" > > Currently, we use the opal call opal_slw_set_reg() to inform the that > the Sleep-Winkle Engine (SLW) to restore the contents of some of the > Hypervisor state on wakeup from deep idle states that

Re: [RFC][PATCH] exec: Use init rlimits for setuid exec

2017-07-06 Thread Andy Lutomirski
On Thu, Jul 6, 2017 at 5:38 AM, Eric W. Biederman wrote: > Kees Cook writes: > >> In an attempt to provide sensible rlimit defaults for setuid execs, this >> inherits the namespace's init rlimits: >> >> $ ulimit -s >> 8192 >> $ ulimit -s unlimited >> $ /bin/sh -c 'ulimit -s' >> unlimited >> $

Re: [PATCH 8/9] RISC-V: User-facing API

2017-07-06 Thread Dave P Martin
On Wed, Jul 05, 2017 at 09:49:36AM -0700, Palmer Dabbelt wrote: > On Mon, 03 Jul 2017 16:06:39 PDT (-0700), james.ho...@imgtec.com wrote: > > On Thu, Jun 29, 2017 at 02:42:38PM -0700, Palmer Dabbelt wrote: > >> On Wed, 28 Jun 2017 15:42:37 PDT (-0700), james.ho...@imgtec.com wrote: > >> > On Wed,

Re: [PATCH 8/9] RISC-V: User-facing API

2017-07-06 Thread Christoph Hellwig
On Thu, Jul 06, 2017 at 09:55:03AM +0100, Will Deacon wrote: > Agreed on the indirection; it feels like this is something that should be in > the vDSO, which could use the cmpxchg instruction if it's available, or > otherwise just uses plain loads and stores. Even that seems like a lot of

Re: [PATCH 2/3] drm: Reorganize drm_pending_event to support future event types

2017-07-06 Thread Keith Packard
Daniel Vetter writes: > A few nits below, but looks good otherwise. Thanks. >> static struct drm_pending_vblank_event *create_vblank_event( >> -struct drm_device *dev, uint64_t user_data) >> +struct drm_device *dev, struct drm_crtc *crtc, uint64_t >> user_data) > >

Re: [RFC PATCH 3/3] dt-bindings: i2c: eeprom: document "start-offset" binding

2017-07-06 Thread Rob Herring
On Thu, Jun 29, 2017 at 02:39:11PM +0300, Claudiu Beznea wrote: > Document "start-offset" binding that will be used by at24 > EEPROM driver. Why do we need this property? Seems like this should be handled by defining regions of the EEPROM that you care about (or perhaps are reserved and don't

Re: [PATCH 8/9] RISC-V: User-facing API

2017-07-06 Thread Will Deacon
On Thu, Jul 06, 2017 at 08:34:27AM -0700, Christoph Hellwig wrote: > On Thu, Jul 06, 2017 at 09:55:03AM +0100, Will Deacon wrote: > > Agreed on the indirection; it feels like this is something that should be in > > the vDSO, which could use the cmpxchg instruction if it's available, or > >

Re: [git pull] vfs.git part 3

2017-07-06 Thread Al Viro
On Thu, Jul 06, 2017 at 05:10:33PM +0200, Christoph Hellwig wrote: > On Thu, Jul 06, 2017 at 04:03:30PM +0100, Al Viro wrote: > > On Thu, Jul 06, 2017 at 04:48:40PM +0200, Christoph Hellwig wrote: > > > > > Just did the whole batch (patch below), but it seems like using a > > > __bitwise type in

Re: [PATCH] mm: make allocation counters per-order

2017-07-06 Thread Mel Gorman
On Thu, Jul 06, 2017 at 03:46:34PM +0100, Roman Gushchin wrote: > > The alloc counter updates are themselves a surprisingly heavy cost to > > the allocation path and this makes it worse for a debugging case that is > > relatively rare. I'm extremely reluctant for such a patch to be added > > given

Re: [PATCH][V2] usb: storage: return on error to avoid a null pointer dereference

2017-07-06 Thread Alan Stern
On Thu, 6 Jul 2017, Colin King wrote: > From: Colin Ian King > > When us->extra is null the driver is not initialized, however, a > later call to osd200_scsi_to_ata is made that dereferences > us->extra, causing a null pointer dereference. The code > currently detects and reports that the

Re: [PATCH v3] mm: Add SLUB free list pointer obfuscation

2017-07-06 Thread Kees Cook
On Thu, Jul 6, 2017 at 6:43 AM, Christoph Lameter wrote: > On Wed, 5 Jul 2017, Kees Cook wrote: > >> @@ -3536,6 +3565,9 @@ static int kmem_cache_open(struct kmem_cache *s, >> unsigned long flags) >> { >> s->flags = kmem_cache_flags(s->size, flags, s->name, s->ctor); >> s->reserved =

Re: [PATCH] apparmor: Fix shadowed local variable in unpack_trans_table()

2017-07-06 Thread John Johansen
On 07/06/2017 06:44 AM, Serge E. Hallyn wrote: > Quoting Geert Uytterhoeven (ge...@linux-m68k.org): >> with W=2: >> >> security/apparmor/policy_unpack.c: In function ‘unpack_trans_table’: >> security/apparmor/policy_unpack.c:469: warning: declaration of ‘pos’ >> shadows a previous local

Re: [git pull] vfs.git part 3

2017-07-06 Thread Al Viro
On Thu, Jul 06, 2017 at 04:46:02PM +0100, Al Viro wrote: > That - on #work.read_write, as in vfs.git at the moment... ... and for COMPAT_SYSCALL you need #define __SC_DELOUSE(t,v) ((__force t)(unsigned long)(v)) in linux/compat.h

Re: [PATCH] mm: make allocation counters per-order

2017-07-06 Thread Mel Gorman
On Thu, Jul 06, 2017 at 10:54:24AM -0400, Debabrata Banerjee wrote: > On Thu, Jul 6, 2017 at 9:19 AM, Mel Gorman > wrote: > > > The alloc counter updates are themselves a surprisingly heavy cost to > > the allocation path and this makes it worse for a debugging case that is > > relatively rare.

Re: [GIT PULL] USB/PHY patches for 4.13-rc1

2017-07-06 Thread Greg KH
On Thu, Jul 06, 2017 at 10:28:03AM -0500, Babu Moger wrote: > > On 7/6/2017 9:33 AM, Greg KH wrote: > > On Thu, Jul 06, 2017 at 09:28:06AM -0500, Babu Moger wrote: > > > On 7/6/2017 3:24 AM, Max Filippov wrote: > > > > Hi Babu, > > > > > > > > On Tue, Jul 4, 2017 at 10:19 AM, Babu Moger > > >

[PATCH v3 4/5] ARM: dts: stm32: Add I2C1 support for STM32F746 SoC

2017-07-06 Thread Pierre-Yves MORDRET
This patch adds I2C1 support for STM32F746 SoC. Signed-off-by: M'boumba Cedric Madianga Signed-off-by: Pierre-Yves MORDRET --- Version history: v3: * None v2: * Update I2C SoC device tree with latest Linux version --- --- arch/arm/boot/dts/stm32f746.dtsi | 22

[PATCH v3 1/5] dt-bindings: i2c-stm32: Document the STM32F7 I2C bindings

2017-07-06 Thread Pierre-Yves MORDRET
This patch adds the documentation of device tree bindings for STM32F7 I2C Signed-off-by: M'boumba Cedric Madianga Signed-off-by: Pierre-Yves MORDRET --- Version history: v3: * None v2: * Remove i2c-timing binding in order to use generic bindings SCL Rising and

[PATCH v3 2/5] i2c: i2c-stm32f4: use generic definition of speed enum

2017-07-06 Thread Pierre-Yves MORDRET
This patch uses a more generic definition of speed enum for i2c-stm32f4 driver. Signed-off-by: M'boumba Cedric Madianga Signed-off-by: Pierre-Yves MORDRET Reviewed-by: Ludovic BARRE --- Version history: v3: v2: * None --- --- drivers/i2c/busses/i2c-stm32.h | 20

[PATCH v3 0/5] Add support for the STM32F7 I2C

2017-07-06 Thread Pierre-Yves MORDRET
This patchset adds support for the I2C controller embedded in STM32F7xx SoC. It enables I2C transfer in interrupt mode with Standard-mode, Fast-mode and Fast-mode+ bus speed. --- Version history: v3: * Move stm32f7_i2c_match above stm32f7_i2c_driver * of_device_get_match_data

Re: [GIT PULL] USB/PHY patches for 4.13-rc1

2017-07-06 Thread Babu Moger
On 7/6/2017 10:51 AM, Greg KH wrote: On Thu, Jul 06, 2017 at 10:28:03AM -0500, Babu Moger wrote: On 7/6/2017 9:33 AM, Greg KH wrote: On Thu, Jul 06, 2017 at 09:28:06AM -0500, Babu Moger wrote: On 7/6/2017 3:24 AM, Max Filippov wrote: Hi Babu, On Tue, Jul 4, 2017 at 10:19 AM, Babu Moger

Re: [PATCH v3] mm: Add SLUB free list pointer obfuscation

2017-07-06 Thread Christoph Lameter
On Thu, 6 Jul 2017, Kees Cook wrote: > On Thu, Jul 6, 2017 at 6:43 AM, Christoph Lameter wrote: > > On Wed, 5 Jul 2017, Kees Cook wrote: > > > >> @@ -3536,6 +3565,9 @@ static int kmem_cache_open(struct kmem_cache *s, > >> unsigned long flags) > >> { > >> s->flags =

[PATCH] staging: atomisp: lm3554: constify acpi_device_id.

2017-07-06 Thread Arvind Yadav
acpi_device_id are not supposed to change at runtime. All functions working with acpi_device_id provided by work with const acpi_device_id. So mark the non-const structs as const. File size before: textdata bss dec hex filename 53471920 2472911c7b

Re: [RFC PATCH v1 0/8] SCHED_DEADLINE freq/cpu invariance and OPP selection

2017-07-06 Thread Steven Rostedt
On Wed, 5 Jul 2017 09:58:57 +0100 Juri Lelli wrote: > Hi, > > v1 of the RFC set implementing frequency/cpu invariance and OPP selection for It would be nice if you specify what "OPP" stands for. A quick google search shows "Other Peoples Privates", which isn't the type of selection I would be

[PATCH v3 3/5] i2c: i2c-stm32f7: add driver

2017-07-06 Thread Pierre-Yves MORDRET
This patch adds initial support for the STM32F7 I2C controller. Signed-off-by: M'boumba Cedric Madianga Signed-off-by: Pierre-Yves MORDRET --- Version history: v3: * Move stm32f7_i2c_match above stm32f7_i2c_driver * of_device_get_match_data instead of of_match_device

[PATCH v3 5/5] ARM: dts: stm32: Add I2C1 support for STM32F746 eval board

2017-07-06 Thread Pierre-Yves MORDRET
This patch adds I2C1 support for STM32F746 eval board Signed-off-by: M'boumba Cedric Madianga Signed-off-by: Pierre-Yves MORDRET --- Version history: v3: * None v2: * Add SCL Rising/Falling time for eval board --- --- arch/arm/boot/dts/stm32746g-eval.dts | 8

Re: [PATCH v4] PCI: Workaround wrong flags completions for IDT switch

2017-07-06 Thread James Puthukattukaran
On 07/03/2017 09:56 PM, Ethan Zhao wrote: James, On Tue, Jul 4, 2017 at 2:17 AM, james puthukattukaran wrote: Ethan - On 7/2/2017 9:55 PM, Ethan Zhao wrote: James, On Wed, Jun 28, 2017 at 5:42 AM, James Puthukattukaran wrote: From: James Puthukattukaran The IDT switch incorrectly

[PATCH] fpga manager: add notifier for manager register and unregister events

2017-07-06 Thread Anatolij Gustschin
Add API functions for registering and removing a notifier for FPGA manager register/unregister events. Notify when a new FPGA manager has been registered or when an existing manager is being removed. This will help configuration interface drivers to get the notion of low-level FPGA managers

Re: [PATCH v3] perf sort: only insert overhead && overhead_children when no overhead* field given

2017-07-06 Thread Jiri Olsa
On Wed, Jul 05, 2017 at 11:35:52AM +0800, changbin...@intel.com wrote: > From: Changbin Du > > If we always insert 'overhead' and 'overhead_children' as sort keys, > this make it impossible to sort as overhead (which displayed as Self) > first.Ths will be a problem if the data is collected with

Re: [PATCH v2 0/9] Remove spin_unlock_wait()

2017-07-06 Thread Peter Zijlstra
On Thu, Jul 06, 2017 at 02:12:24PM +, David Laight wrote: > From: Paul E. McKenney > > Sent: 06 July 2017 00:30 > > There is no agreed-upon definition of spin_unlock_wait()'s semantics, > > and it appears that all callers could do just as well with a lock/unlock > > pair. This series

[PATCH v5] PCI: Workaround wrong flags completions for IDT switch

2017-07-06 Thread James Puthukattukaran
From: James Puthukattukaran The IDT switch incorrectly flags an ACS source violation on a read config request to an end point device on the completion (IDT 89H32H8G3-YC, errata #36) even though the PCI Express spec states that completions are never affected by ACS source violation (PCI Spec

[PATCH] staging: atomisp: ov2680: constify acpi_device_id.

2017-07-06 Thread Arvind Yadav
acpi_device_id are not supposed to change at runtime. All functions working with acpi_device_id provided by work with const acpi_device_id. So mark the non-const structs as const. File size before: textdata bss dec hex filename 124663120 8 155943cea

Re: [RFC PATCH v1 0/8] SCHED_DEADLINE freq/cpu invariance and OPP selection

2017-07-06 Thread Juri Lelli
On 06/07/17 11:57, Steven Rostedt wrote: > On Wed, 5 Jul 2017 09:58:57 +0100 > Juri Lelli wrote: > > > Hi, > > > > v1 of the RFC set implementing frequency/cpu invariance and OPP selection > > for > > It would be nice if you specify what "OPP" stands for. A quick google > search shows "Other

Re: [PATCH v2 6/9] clk: at91: clk-generated: make gclk determine audio_pll rate

2017-07-06 Thread Alexandre Belloni
On 06/07/2017 at 17:29:22 +0200, Nicolas Ferre wrote: > > + /* > > +* The audio_pll rate can be modified, unlike the five others clocks > > +* that should never be altered. > > +* The audio_pll can technically be used by multiple consumers. However, > > +* with the rate locking,

Re: [PATCH v2 0/9] Remove spin_unlock_wait()

2017-07-06 Thread Peter Zijlstra
On Thu, Jul 06, 2017 at 08:21:10AM -0700, Paul E. McKenney wrote: > And yes, there are architecture-specific optimizations for an > empty spin_lock()/spin_unlock() critical section, and the current > arch_spin_unlock_wait() implementations show some of these optimizations. > But I expect that

Re: [PATCH] mm: make allocation counters per-order

2017-07-06 Thread Debabrata Banerjee
On Thu, Jul 6, 2017 at 11:51 AM, Mel Gorman wrote: > > These counters do not actually help you solve that particular problem. > Knowing how many allocations happened since the system booted doesn't tell > you much about how many failed or why they failed. You don't even know > what frequency they

Re: [RFC v5 09/11] mm: Try spin lock in speculative path

2017-07-06 Thread Peter Zijlstra
On Thu, Jul 06, 2017 at 05:29:26PM +0200, Laurent Dufour wrote: > Based on the benchmarks I run, it doesn't fail so much often, but I was > thinking about adding some counters here. The system is accounting for > major page faults and minor ones, respectively current->maj_flt and >

Re: [PATCH 2/2] afs: Add metadata xattrs

2017-07-06 Thread David Howells
Christoph Hellwig wrote: > NAK. Don't overload xattrs with magic behavior just to avoid the need > to do proper syscalls or ioctls. How? This has to work on non-files, files you can't open and mountpoints. You can't do an ioctl() on a file opened O_PATH: if (unlikely(f->f_flags &

[PATCH] staging: atomisp: ov8858: constify acpi_device_id.

2017-07-06 Thread Arvind Yadav
acpi_device_id are not supposed to change at runtime. All functions working with acpi_device_id provided by work with const acpi_device_id. So mark the non-const structs as const. File size before: textdata bss dec hex filename 238048448 0 322527dfc

Re: [RFC PATCH v1 0/8] SCHED_DEADLINE freq/cpu invariance and OPP selection

2017-07-06 Thread Peter Zijlstra
On Thu, Jul 06, 2017 at 11:57:15AM -0400, Steven Rostedt wrote: > On Wed, 5 Jul 2017 09:58:57 +0100 > Juri Lelli wrote: > > > Hi, > > > > v1 of the RFC set implementing frequency/cpu invariance and OPP selection > > for > > It would be nice if you specify what "OPP" stands for. A quick

Re: [kernel-hardening] Re: [PATCH v3] mm: Add SLUB free list pointer obfuscation

2017-07-06 Thread Daniel Micay
On Thu, 2017-07-06 at 10:55 -0500, Christoph Lameter wrote: > On Thu, 6 Jul 2017, Kees Cook wrote: > > > On Thu, Jul 6, 2017 at 6:43 AM, Christoph Lameter > > wrote: > > > On Wed, 5 Jul 2017, Kees Cook wrote: > > > > > > > @@ -3536,6 +3565,9 @@ static int kmem_cache_open(struct > > > >

[RFC PATCH 0/1] mm/mremap: add MREMAP_MIRROR flag

2017-07-06 Thread Mike Kravetz
The mremap system call has the ability to 'mirror' parts of an existing mapping. To do so, it creates a new mapping that maps the same pages as the original mapping, just at a different virtual address. This functionality has existed since at least the 2.6 kernel [1]. A comment was added to the

[RFC PATCH 1/1] mm/mremap: add MREMAP_MIRROR flag for existing mirroring functionality

2017-07-06 Thread Mike Kravetz
The mremap system call has the ability to 'mirror' parts of an existing mapping. To do so, it creates a new mapping that maps the same pages as the original mapping, just at a different virtual address. This functionality has existed since at least the 2.6 kernel. This patch simply adds a new

Re: [PATCH v2 0/9] Remove spin_unlock_wait()

2017-07-06 Thread Paul E. McKenney
On Thu, Jul 06, 2017 at 06:05:55PM +0200, Peter Zijlstra wrote: > On Thu, Jul 06, 2017 at 02:12:24PM +, David Laight wrote: > > From: Paul E. McKenney [ . . . ] > Now on the one hand I feel like Oleg that it would be a shame to loose > the optimization, OTOH this thing is really really

[PATCH] staging: atomisp: gc0310: constify acpi_device_id.

2017-07-06 Thread Arvind Yadav
acpi_device_id are not supposed to change at runtime. All functions working with acpi_device_id provided by work with const acpi_device_id. So mark the non-const structs as const. File size before: textdata bss dec hex filename 102971888 0 121852f99

[GIT PULL] Mailbox changes for v4.13

2017-07-06 Thread Jassi Brar
Hi Linus, The following changes since commit 63f700aab4c11d46626de3cd051dae56cf7e9056: Merge tag 'xtensa-20170612' of git://github.com/jcmvbkbc/linux-xtensa (2017-06-13 15:09:10 +0900) are available in the git repository at:

[PATCH] integrity: track ctime in addition to i_version for assessment

2017-07-06 Thread Jeff Layton
From: Jeff Layton The IMA assessment code tries to use the i_version counter to detect when changes to a file have occurred. Many filesystems don't increment it properly (or at all) so detecting changes with that is not always reliable. That check should really be gated on IS_I_VERSION. When

Re: [PATCH v2 0/9] Remove spin_unlock_wait()

2017-07-06 Thread Paul E. McKenney
On Thu, Jul 06, 2017 at 06:10:47PM +0200, Peter Zijlstra wrote: > On Thu, Jul 06, 2017 at 08:21:10AM -0700, Paul E. McKenney wrote: > > And yes, there are architecture-specific optimizations for an > > empty spin_lock()/spin_unlock() critical section, and the current > > arch_spin_unlock_wait()

Re: [PATCH v3 1/2] perf util: Check for fused instruction

2017-07-06 Thread Arnaldo Carvalho de Melo
Em Tue, Jun 20, 2017 at 08:22:09PM +0800, Jin Yao escreveu: > Macro fusion merges two instructions to a single micro-op. Intel > core platform performs this hardware optimization under limited > circumstances. > > For example, CMP + JCC can be "fused" and executed /retired > together. While with

Re: [PATCH 3/3] drm: Add CRTC_GET_SEQUENCE and CRTC_QUEUE_SEQUENCE ioctls

2017-07-06 Thread Keith Packard
Daniel Vetter writes: > I very much like this since the old ioctl really is a rather bad horror > show. And since it's tied in with ums drivers everything is > complicated. Thanks for your kind words. > I started a discussion a while back whether these should be restricted to > DRM_MASTER

Re: [PATCH 1/3] Protectable memory support

2017-07-06 Thread Jerome Glisse
On Wed, Jul 05, 2017 at 04:46:26PM +0300, Igor Stoppa wrote: > The MMU available in many systems running Linux can often provide R/O > protection to the memory pages it handles. > > However, the MMU-based protection works efficiently only when said pages > contain exclusively data that will not

Re: [PATCH 3/3] drm: Add CRTC_GET_SEQUENCE and CRTC_QUEUE_SEQUENCE ioctls

2017-07-06 Thread Keith Packard
Ville Syrjälä writes: > Maybe, or maybe we want to turn the interrupt on in that case? That's > what the old ioctl does. That's what I suggested in my reply to Daniel's review. Even if we add the accurate function, we'll still need the interrupt-enable case as a fallback for drivers which don't

[PATCH] staging: atomisp: ov2722: constify acpi_device_id.

2017-07-06 Thread Arvind Yadav
acpi_device_id are not supposed to change at runtime. All functions working with acpi_device_id provided by work with const acpi_device_id. So mark the non-const structs as const. File size before: textdata bss dec hex filename 147711880 0 16651410b

Re: [PATCH v2 3/4] btrfs: Add zstd support

2017-07-06 Thread Adam Borowski
On Thu, Jun 29, 2017 at 12:41:07PM -0700, Nick Terrell wrote: > Add zstd compression and decompression support to BtrFS. zstd at its > fastest level compresses almost as well as zlib, while offering much > faster compression and decompression, approaching lzo speeds. Got a reproducible crash on

Re: [RFC][PATCH] exec: Use init rlimits for setuid exec

2017-07-06 Thread Linus Torvalds
On Wed, Jul 5, 2017 at 9:32 PM, Kees Cook wrote: > In an attempt to provide sensible rlimit defaults for setuid execs, this > inherits the namespace's init rlimits: Yeah, so I have to admit to hating this patch. As already mentioned by others, it's not only not clear that we want to do this on

[PATCH v3 3/3] include: warn for inconsistent endian config definition

2017-07-06 Thread Babu Moger
We have seen some generic code use config parameter CONFIG_CPU_BIG_ENDIAN to decide the endianness. Here are the few examples. include/asm-generic/qrwlock.h drivers/of/base.c drivers/of/fdt.c drivers/tty/serial/earlycon.c drivers/tty/serial/serial_core.c Display warning if CPU_BIG_ENDIAN is not

[PATCH v3 0/3] Define CPU_BIG_ENDIAN or warn for inconsistencies

2017-07-06 Thread Babu Moger
Resending the series per Greg KH's request. Found this problem while enabling queued rwlock on SPARC. The parameter CONFIG_CPU_BIG_ENDIAN is used to clear the specific byte in qrwlock structure. Without this parameter, we clear the wrong byte. Here is the code in include/asm-generic/qrwlock.h

[PATCH v3 1/3] arch: Define CPU_BIG_ENDIAN for all fixed big endian archs

2017-07-06 Thread Babu Moger
While working on enabling queued rwlock on SPARC, found this following code in include/asm-generic/qrwlock.h which uses CONFIG_CPU_BIG_ENDIAN to clear a byte. static inline u8 *__qrwlock_write_byte(struct qrwlock *lock) { return (u8 *)lock + 3 * IS_BUILTIN(CONFIG_CPU_BIG_ENDIAN); }

[PATCH v3 2/3] arch/microblaze: Add choice for endianness and update Makefile

2017-07-06 Thread Babu Moger
microblaze architectures can be configured for either little or big endian formats. Add a choice option for the user to select the correct endian format(default to big endian). Also update the Makefile so toolchain can compile for the format it is configured for. Signed-off-by: Babu Moger

Re: [PATCH v2] irqchip/mips-gic: Add missing shared interrupt handler

2017-07-06 Thread Paul Burton
Hello, On Thursday, 6 July 2017 05:03:59 PDT Thomas Gleixner wrote: > On Thu, 6 Jul 2017, jrj...@gmail.com wrote: > > CC+ MIPS folks. There is a reason WHY I added them to my previous reply. Thanks Thomas. > > From: Jun-Ru Chang > > > > Commit b87281e7f205 ("irqchip/mips-gic: Remove device

[PATCH] staging: atomisp: ov5693: constify acpi_device_id.

2017-07-06 Thread Arvind Yadav
acpi_device_id are not supposed to change at runtime. All functions working with acpi_device_id provided by work with const acpi_device_id. So mark the non-const structs as const. File size before: textdata bss dec hex filename 207293264 0 239935db9

Re: [PATCH v4 3/3] arm64: kvm: inject SError with user space specified syndrome

2017-07-06 Thread James Morse
Hi gengdongjiu, On 05/07/17 09:14, gengdongjiu wrote: > On 2017/7/4 18:14, James Morse wrote: >> Can you give us a specific example of an error you are trying to handle? > For example: > guest OS user space accesses device type memory, but happen SError. because > the > SError is asynchronous

[PATCH] staging: atomisp: mt9m114: constify acpi_device_id.

2017-07-06 Thread Arvind Yadav
acpi_device_id are not supposed to change at runtime. All functions working with acpi_device_id provided by work with const acpi_device_id. So mark the non-const structs as const. File size before: textdata bss dec hex filename 151482640 8 177964584

Re: [PATCH v2 0/9] Remove spin_unlock_wait()

2017-07-06 Thread Peter Zijlstra
On Thu, Jul 06, 2017 at 09:24:12AM -0700, Paul E. McKenney wrote: > On Thu, Jul 06, 2017 at 06:10:47PM +0200, Peter Zijlstra wrote: > > On Thu, Jul 06, 2017 at 08:21:10AM -0700, Paul E. McKenney wrote: > > > And yes, there are architecture-specific optimizations for an > > > empty

Re: [PATCH] iio: magnetometer: Only declare ACPI table when ACPI is enable

2017-07-06 Thread Matthias Kaehlcke
El Sat, Jul 01, 2017 at 01:11:12PM +0100 Jonathan Cameron ha dit: > On Thu, 29 Jun 2017 14:51:56 +0200 > Linus Walleij wrote: > > > On Tue, Jun 27, 2017 at 2:25 AM, Matthias Kaehlcke > > wrote: > > > > > Don't inflate the kernel size with data that isn't used. The conditional > > >

Re: [PATCH] mm: make allocation counters per-order

2017-07-06 Thread Mel Gorman
On Thu, Jul 06, 2017 at 12:12:47PM -0400, Debabrata Banerjee wrote: > On Thu, Jul 6, 2017 at 11:51 AM, Mel Gorman > wrote: > > > > These counters do not actually help you solve that particular problem. > > Knowing how many allocations happened since the system booted doesn't tell > > you much

[PATCH] staging: atomisp: gc2235: constify acpi_device_id.

2017-07-06 Thread Arvind Yadav
acpi_device_id are not supposed to change at runtime. All functions working with acpi_device_id provided by work with const acpi_device_id. So mark the non-const structs as const. File size before: textdata bss dec hex filename 107541360 4 121182f56

Re: [PATCH] mm: make allocation counters per-order

2017-07-06 Thread Roman Gushchin
On Thu, Jul 06, 2017 at 04:47:05PM +0100, Mel Gorman wrote: > On Thu, Jul 06, 2017 at 03:46:34PM +0100, Roman Gushchin wrote: > > > The alloc counter updates are themselves a surprisingly heavy cost to > > > the allocation path and this makes it worse for a debugging case that is > > > relatively

Re: [PATCH v2 2/6] Documentation: devicetree: add bindings to support ARM MHU doorbells

2017-07-06 Thread Sudeep Holla
On 06/07/17 15:37, Jassi Brar wrote: > On Thu, Jul 6, 2017 at 3:03 PM, Sudeep Holla wrote: [...] >> >> I said it *may not be used*, currently it is used. >> > SCPI provides more than what SCMI currently does - dvfs, clock, sensor. Not sure what you mean by that, but that's not true. > I see

Re: [PATCH v2 0/9] Remove spin_unlock_wait()

2017-07-06 Thread Alan Stern
On Thu, 6 Jul 2017, Paul E. McKenney wrote: > On Thu, Jul 06, 2017 at 06:10:47PM +0200, Peter Zijlstra wrote: > > On Thu, Jul 06, 2017 at 08:21:10AM -0700, Paul E. McKenney wrote: > > > And yes, there are architecture-specific optimizations for an > > > empty spin_lock()/spin_unlock() critical

UBSAN: Undefined behaviour in fs/fs-writeback.c:1586:18

2017-07-06 Thread Toralf Förster
Got that at a hardened Gentoo Linux server with 4.12.0 (for the first time) at a BTRFS logical volume occuping about 4/9 of a 5 TB volume group after an uptime of about 2 days : Had issues with processes accessing files at that volume now too. Jul 6 15:33:53 mr-fox kernel: [158695.417132]

[PATCH] nfc: nxp-nci: constify acpi_device_id.

2017-07-06 Thread Arvind Yadav
acpi_device_id are not supposed to change at runtime. All functions working with acpi_device_id provided by work with const acpi_device_id. So mark the non-const structs as const. File size before: textdata bss dec hex filename 2208 400 02608 a30

Re: [RFC][PATCH] exec: Use init rlimits for setuid exec

2017-07-06 Thread Linus Torvalds
On Thu, Jul 6, 2017 at 9:34 AM, Linus Torvalds wrote: > > (a) minimal: just use our existing default stack (and stack _only_) > limit value for suid binaries that actually get extra permissions: { > _STK_LIM, RLIM_INFINITY }. > > (c) perhaps encourage people to annotate their suid binaries with

Re: [PATCH v2 0/9] Remove spin_unlock_wait()

2017-07-06 Thread Peter Zijlstra
On Thu, Jul 06, 2017 at 09:20:24AM -0700, Paul E. McKenney wrote: > On Thu, Jul 06, 2017 at 06:05:55PM +0200, Peter Zijlstra wrote: > > On Thu, Jul 06, 2017 at 02:12:24PM +, David Laight wrote: > > > From: Paul E. McKenney > > [ . . . ] > > > Now on the one hand I feel like Oleg that it

Re: [PATCH v2 0/9] Remove spin_unlock_wait()

2017-07-06 Thread Peter Zijlstra
On Thu, Jul 06, 2017 at 12:49:12PM -0400, Alan Stern wrote: > On Thu, 6 Jul 2017, Paul E. McKenney wrote: > > > On Thu, Jul 06, 2017 at 06:10:47PM +0200, Peter Zijlstra wrote: > > > On Thu, Jul 06, 2017 at 08:21:10AM -0700, Paul E. McKenney wrote: > > > > And yes, there are architecture-specific

[PATCH] extcon: int3496: constify acpi_device_id.

2017-07-06 Thread Arvind Yadav
acpi_device_id are not supposed to change at runtime. All functions working with acpi_device_id provided by work with const acpi_device_id. So mark the non-const structs as const. File size before: textdata bss dec hex filename 1733 352 02085 825

Re: [git pull] vfs.git part 3

2017-07-06 Thread Christoph Hellwig
On Thu, Jul 06, 2017 at 04:51:13PM +0100, Al Viro wrote: > On Thu, Jul 06, 2017 at 04:46:02PM +0100, Al Viro wrote: > > > That - on #work.read_write, as in vfs.git at the moment... > > ... and for COMPAT_SYSCALL you need > #define __SC_DELOUSE(t,v) ((__force t)(unsigned long)(v)) > in

Re: [PATCH v3] drivers/misc: (aspeed-lpc-snoop): Add ast2400 to compat

2017-07-06 Thread Patrick Venture
On Wed, Jul 5, 2017 at 2:51 PM, Patrick Venture wrote: > This driver can be used on the aspeed ast2400 with minor > modifications. > > Tested: ast2400 on quanta-q71l > > Signed-off-by: Patrick Venture > --- > v3: added .data object to determine behavior difference between ast2400 and >

Re: [PATCH v3] drivers/misc: (aspeed-lpc-snoop): Add ast2400 to compat

2017-07-06 Thread Patrick Venture
On Thu, Jul 6, 2017 at 10:00 AM, Patrick Venture wrote: > On Wed, Jul 5, 2017 at 2:51 PM, Patrick Venture wrote: >> This driver can be used on the aspeed ast2400 with minor >> modifications. >> >> Tested: ast2400 on quanta-q71l >> >> Signed-off-by: Patrick Venture >> --- >> v3: added .data

Re: [PATCH v2 0/9] Remove spin_unlock_wait()

2017-07-06 Thread Paul E. McKenney
On Thu, Jul 06, 2017 at 06:41:34PM +0200, Peter Zijlstra wrote: > On Thu, Jul 06, 2017 at 09:24:12AM -0700, Paul E. McKenney wrote: > > On Thu, Jul 06, 2017 at 06:10:47PM +0200, Peter Zijlstra wrote: > > > On Thu, Jul 06, 2017 at 08:21:10AM -0700, Paul E. McKenney wrote: > > > > And yes, there are

[PATCH v4] drivers/misc: (aspeed-lpc-snoop): Add ast2400 to compat

2017-07-06 Thread Patrick Venture
This driver can be used on the aspeed ast2400 with minor modifications. Tested: ast2400 on quanta-q71l Signed-off-by: Patrick Venture --- v4: fix the 2400/2500 data as they were backwards. v3: added .data object to determine behavior difference between ast2400 and ast2500. v2: added

Re: [PATCH v2 1/4] dt-bindings: pwm-backlight: add pwm-delay-us property

2017-07-06 Thread Rob Herring
On Fri, Jun 30, 2017 at 6:21 AM, Enric Balletbo i Serra wrote: > From: huang lin > > Add a pwm-delay-us property to specify the delay between setting an > initial (non-zero) PWM value and enabling the backlight, and also the > delay between disabling the backlight and setting PWM value to 0. > >

<    10   11   12   13   14   15   16   17   18   19   >