Re: [PATCH v2 3/3] kvm: svm: Use the hardware provided GPA instead of page walk

2016-12-14 Thread Brijesh Singh
Hi Paolo, On 12/13/2016 11:09 AM, Paolo Bonzini wrote: On 12/12/2016 18:51, Brijesh Singh wrote: As per the AMD BKDG [1] Section 2.7.1, we should not be using any of these instruction for MMIO access, the behavior is undefined. The question is, do we really need to add logic to detect the cr

Re: [RFC] block: check partition alignment

2016-12-14 Thread Christoph Hellwig
> To prevent partitions that are not aligned to the physical blocksize > of a device check for the alignment in the blkpg_ioctl. We'd also need to reject this when reading partitions from disk, right? > + /* check if partition is aligned to blocksize */ > +

Re: [PATCH] arch: x86: kernel: fixed unused label issue

2016-12-14 Thread Peter Zijlstra
On Wed, Dec 14, 2016 at 11:15:13AM +, Piotr Gregor wrote: > The patch_default label is only used from within > case PARAVIRT_PATCH(pv_lock_ops.queued_spin_unlock) > and > case PARAVIRT_PATCH(pv_lock_ops.vcpu_is_preempted) > i.e. when #if defined(CONFIG_PARAVIRT_SPINLOCKS) is true. >

Re: [PATCH v3 3/3] perf tool: add cgroup identifier entry in perf report

2016-12-14 Thread Hari Bathini
On Wednesday 14 December 2016 09:22 PM, Eric W. Biederman wrote: Peter Zijlstra writes: On Wed, Dec 14, 2016 at 08:56:43AM +1300, Eric W. Biederman wrote: I would just make the identifier a structure containing the device number and the inode number. It didn't look like perf required the i

Re: [PATCH v2 3/3] power: supply: bq24735-charger: allow chargers to share the ac-detect gpio

2016-12-14 Thread Sebastian Reichel
[of course I forgot to actually add gpio people, let's try again] On Wed, Dec 14, 2016 at 05:59:21PM +0100, Sebastian Reichel wrote: > Hi, > > On Wed, Dec 14, 2016 at 12:56:45AM +0100, Peter Rosin wrote: > > If several parallel bq24735 chargers have their ac-detect gpios wired > > together (or if

Re: [PATCH v2 3/3] power: supply: bq24735-charger: allow chargers to share the ac-detect gpio

2016-12-14 Thread Sebastian Reichel
Hi, On Wed, Dec 14, 2016 at 12:56:45AM +0100, Peter Rosin wrote: > If several parallel bq24735 chargers have their ac-detect gpios wired > together (or if only one of the parallel bq24735 chargers have its > ac-detect pin wired to a gpio, and the others are assumed to react the > same), then all d

Re: [PATCH] keys/encrypted: Fix two crypto-on-the-stack bugs

2016-12-14 Thread Joerg Roedel
On Tue, Dec 13, 2016 at 08:40:00AM -0800, Andy Lutomirski wrote: > But I think this is rather silly. Joerg, Linus, etc: would it be okay > to change lib/dma-debug.c to allow DMA *from* rodata? Yeah, this would be fine for DMA_TO_DEVICE mappings. At least I can't think of a reason right now to not

Re: [PATCH v2 2/4] KVM: x86: replace kvm_apic_id with kvm_{x,x2}apic_id

2016-12-14 Thread Paolo Bonzini
On 14/12/2016 17:15, David Hildenbrand wrote: > >> kvm_for_each_vcpu(i, vcpu, kvm) >> if (kvm_apic_present(vcpu)) >> -max_id = max(max_id, kvm_apic_id(vcpu->arch.apic)); >> +max_id = max(max_id, kvm_x2apic_id(vcpu->arch.apic)); >> >> new = kvm_kvzalloc(

Re: [PATCH v13 1/3] drm: Move locking into drm_debugfs_crtc_crc_add

2016-12-14 Thread Emil Velikov
On 14 December 2016 at 13:34, Tomeu Vizoso wrote: > There's no reason any more for callers of this function to take the lock > themselves, so just move the lock to the function to avoid confusion and > bugs when more callers are contributed. > > Signed-off-by: Tomeu Vizoso Reviewed-by: Emil Velik

Re: [PATCH 1/2] devicetree: power: add bindings for GPIO-driven power switches

2016-12-14 Thread Bartosz Golaszewski
2016-12-13 20:27 GMT+01:00 Rob Herring : > On Sun, Dec 11, 2016 at 11:21:44PM +0100, Bartosz Golaszewski wrote: >> Some boards are equipped with simple, GPIO-driven power load switches. >> An example of such ICs is the TI tps229* series. > > How is this different than a GPIO regulator? The input an

[RFC] perf/x86/intel: Account interrupts for PEBS errors

2016-12-14 Thread Jiri Olsa
hi, I'm hitting soft lockup generated by fuzzer, where the perf hangs in remote_install path like: NMI watchdog: BUG: soft lockup - CPU#22 stuck for 22s! [perf_fuzzer:5816] task: 880273148000 task.stack: c90002d58000 RIP: 0010:[] [] smp_call_function_single+0xe2/0x140 RSP: 0018:

Re: [PATCH] usb: musb: debugfs: allow forcing host mode together with speed in testmode

2016-12-14 Thread Greg Kroah-Hartman
On Wed, Dec 14, 2016 at 03:47:57PM +0100, Pali Rohár wrote: > Signed-off-by: Pali Rohár > --- > drivers/usb/musb/musb_debugfs.c | 44 > +-- > 1 file changed, 28 insertions(+), 16 deletions(-) I don't accept patches without any changelog information, nor wou

RE: [PATCH 2/2] FPGA: Add TS-7300 FPGA manager

2016-12-14 Thread Hartley Sweeten
On Monday, December 12, 2016 9:02 AM, Alan Tull wrote: > On Sun, 11 Dec 2016, Florian Fainelli wrote: >> Add support for loading bitstreams on the Altera Cyclone II FPGA >> populated on the TS-7300 board. This is done through the configuration >> and data registers offered through a memory interfac

Re: [PATCH linux v1 0/4] Seven segment display support

2016-12-14 Thread Greg KH
On Wed, Dec 14, 2016 at 02:12:41PM +0100, Neil Armstrong wrote: > On 12/14/2016 01:56 PM, Greg KH wrote: > > On Wed, Dec 14, 2016 at 01:45:30PM +0100, Thomas Petazzoni wrote: > >> Hello, > >> > >> On Tue, 13 Dec 2016 23:55:00 -0800, Jaghathiswari Rankappagounder > >> Natarajan wrote: > >> > >>> Doc

Re: [PATCH] HID: add Asus macrokey support for Asus "Republic of Gamers" laptop

2016-12-14 Thread Chris Chiu
On Wed, Dec 14, 2016 at 6:20 PM, Jiri Kosina wrote: > On Mon, 12 Dec 2016, Chris Chiu wrote: > > [ ... snip ... ] >> +static const struct hid_device_id asus_rog_devices[] = { >> + { HID_USB_DEVICE(USB_VENDOR_ID_ASUSTEK, >> USB_DEVICE_ID_ASUSTEK_ROG_MACROKEY1) }, >> + { HID_USB_DEVICE(USB_

Re: [PATCH] MIPS: NI 169445 board support

2016-12-14 Thread Rob Herring
On Wed, Dec 14, 2016 at 10:32 AM, Nathan Sullivan wrote: > On Fri, Dec 09, 2016 at 03:18:28PM -0600, Rob Herring wrote: >> On Fri, Dec 02, 2016 at 09:42:09AM -0600, Nathan Sullivan wrote: >> > Support the National Instruments 169445 board. >> > >> > Signed-off-by: Nathan Sullivan >> > --- >> > "g

Re: Fw: [lkp-developer] [sched,rcu] cf7a2dca60: [No primary change] +186% will-it-scale.time.involuntary_context_switches

2016-12-14 Thread Paul E. McKenney
On Wed, Dec 14, 2016 at 05:15:41PM +0100, Michal Hocko wrote: > On Wed 14-12-16 03:06:09, Paul E. McKenney wrote: > > On Wed, Dec 14, 2016 at 10:54:25AM +0100, Michal Hocko wrote: > > > On Tue 13-12-16 07:14:08, Paul E. McKenney wrote: > > > > Just FYI for the moment... > > > > > > > > So even wit

RE: [PATCH] drivers: staging: comedi: fix function prototypes

2016-12-14 Thread Hartley Sweeten
On December 14, 2016 6:42 AM, Piotr Gregor wrote: > Add names of parameters to function prototypes in comedi PCI. > Checkpatch reports now no errors. > > Signed-off-by: Piotr Gregor > --- > drivers/staging/comedi/comedi_pci.h | 18 ++ > 1 file changed, 10 insertions(+), 8 deletion

Re: [PATCH v3 1/4] pinctrl: aspeed: Read and write bits in LPC and GFX controllers

2016-12-14 Thread Rob Herring
On Tue, Dec 13, 2016 at 12:12 AM, Andrew Jeffery wrote: > On Mon, 2016-12-12 at 10:27 -0600, Rob Herring wrote: >> On Tue, Dec 06, 2016 at 02:11:49PM +1100, Andrew Jeffery wrote: >> > The System Control Unit IP block in the Aspeed SoCs is typically where >> > the pinmux configuration is found, but

[RFC] block: check partition alignment

2016-12-14 Thread Stefan Haberland
Partitions that are not aligned to the blocksize of a device may cause invalid I/O requests because the blocklayer cares only about alignment within the partition when building requests on partitions. device |4096|4096|4096| partition offset 512byte

Re: [PATCH 0/2] Add maintainers to the admin guide

2016-12-14 Thread Mauro Carvalho Chehab
Em Wed, 14 Dec 2016 08:14:44 -0800 Joe Perches escreveu: > On Tue, 2016-12-13 at 07:38 -0200, Mauro Carvalho Chehab wrote: > > Em Mon, 12 Dec 2016 12:56:50 -0800 > > Joe Perches escreveu: > > > Does the boxing with the === blocks align properly? > > > It it really useful? Is there another

Re: [PATCH v2 1/3] power: supply: bq24735-charger: simplify register update to stop charging

2016-12-14 Thread Sebastian Reichel
Hi, On Wed, Dec 14, 2016 at 12:56:43AM +0100, Peter Rosin wrote: > Providing value bits outside of the mask is pointless. > > Signed-off-by: Peter Rosin > --- > drivers/power/supply/bq24735-charger.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/power/suppl

Re: [RFC 07/10] kmod: use simplified rate limit printk

2016-12-14 Thread Joe Perches
On Wed, 2016-12-14 at 17:23 +0100, Petr Mladek wrote: > On Thu 2016-12-08 11:49:01, Luis R. Rodriguez wrote: > > Just use the simplified rate limit printk when the max modprobe > > limit is reached, while at it throw out a bone should the error > > be triggered. [] > > diff --git a/kernel/kmod.c b/

Re: usb/gadget: warning in ep_write_iter/__alloc_pages_nodemask

2016-12-14 Thread Alan Stern
On Wed, 14 Dec 2016, Michal Hocko wrote: > On Tue 13-12-16 08:33:34, Alan Stern wrote: > > On Tue, 13 Dec 2016, Michal Hocko wrote: > > > > > > > That being said, what ep_write_iter does sounds quite stupit. It just > > > > > allocates a large continuous buffer which seems to be under user > > >

RE: [PATCH v3 1/2] ARM: ep93xx: Register ts73xx-fpga manager driver for TS-7300

2016-12-14 Thread Hartley Sweeten
On Tuesday, December 13, 2016 7:36 PM, Florian Fainelli wrote: > > Register the TS-7300 FPGA manager device drivers which allows us to load > bitstreams into the on-board Altera Cyclone II FPGA. > > Signed-off-by: Florian Fainelli > --- > arch/arm/mach-ep93xx/ts72xx.c | 26 +++

Re: [PATCH 4/3] random: use siphash24 instead of md5 for get_random_int/long

2016-12-14 Thread Theodore Ts'o
On Wed, Dec 14, 2016 at 04:10:37AM +0100, Jason A. Donenfeld wrote: > This duplicates the current algorithm for get_random_int/long, but uses > siphash24 instead. This comes with several benefits. It's certainly > faster and more cryptographically secure than MD5. This patch also > hashes the pid,

Re: [PATCH 1/1] arm64: Correcting format specifier for printing 64 bit addresses

2016-12-14 Thread Paolo Bonzini
On 06/12/2016 17:11, Christoffer Dall wrote: > + kvm_err("Unsupported guest CP%d access at: %08lx\n", > + cp, *vcpu_pc(vcpu)); > + else > + kvm_err("Unsupported guest CP%d access at: %016lx\n", > + cp, *vcpu_pc(vcpu)); > > It fe

Re: [PATCH] MIPS: NI 169445 board support

2016-12-14 Thread Nathan Sullivan
On Fri, Dec 09, 2016 at 03:18:28PM -0600, Rob Herring wrote: > On Fri, Dec 02, 2016 at 09:42:09AM -0600, Nathan Sullivan wrote: > > Support the National Instruments 169445 board. > > > > Signed-off-by: Nathan Sullivan > > --- > > "gpio: mmio: add support for NI 169445 NAND GPIO" and > > "devicet

Re: [v1] net:ethernet:cavium:octeon:octeon_mgmt: Handle return NULL error from devm_ioremap

2016-12-14 Thread arvind Yadav
Sorry for build failure. I have send new changes. Which does not this failure. Thanks -Arvind On Wednesday 14 December 2016 08:10 PM, kbuild test robot wrote: Hi Arvind, [auto build test ERROR on net-next/master] [also build test ERROR on v4.9 next-20161214] [if your patch is applied to the

Re: linux-next: Tree for Dec 14

2016-12-14 Thread Paul Gortmaker
On Tue, Dec 13, 2016 at 11:22 PM, Stephen Rothwell wrote: > Hi all, > > Please do not add any material for v4.11 to your linux-next included > branches until after v4.10-rc1 has been released. > > Changes since 20161213: > > The vfs-miklos tree gained a conflict against the ubifs tree. > > The ak

[v2] net:ethernet:cavium:octeon:octeon_mgmt: Handle return NULL error from devm_ioremap

2016-12-14 Thread Arvind Yadav
Here, If devm_ioremap will fail. It will return NULL. Kernel can run into a NULL-pointer dereference. This error check will avoid NULL pointer dereference. Signed-off-by: Arvind Yadav --- drivers/net/ethernet/cavium/octeon/octeon_mgmt.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/d

Re: [PATCH] qedr: return -EINVAL if pd is null and avoid null ptr dereference

2016-12-14 Thread Doug Ledford
On 10/18/2016 2:39 PM, Colin King wrote: > From: Colin Ian King > > Currently, if pd is null then we hit a null pointer derference > on accessing pd->pd_id. Instead of just printing an error message > we should also return -EINVAL immediately. > > Signed-off-by: Colin Ian King Applied, thanks

Re: [PATCH v2] keys/encrypted: Fix two crypto-on-the-stack bugs

2016-12-14 Thread Andy Lutomirski
On Wed, Dec 14, 2016 at 12:37 AM, David Howells wrote: > Andy Lutomirski wrote: > >> > - sg_set_buf(&sg_out[1], pad, sizeof pad); >> > + sg_set_buf(&sg_out[1], empty_zero_page, 16); >> >> My fix here is obviously bogus (I meant to use ZERO_PAGE(0)), but what >> exactly is the code try

Re: [PATCH v2 04/11] Documentation: DT: binding: axp20x_usb_power: add axp223 compatible

2016-12-14 Thread Chen-Yu Tsai
On Fri, Dec 9, 2016 at 7:04 PM, Quentin Schulz wrote: > This adds the "x-powers,axp223-usb-power-supply" to the list of > compatibles for AXP20X VBUS power supply driver. > > Signed-off-by: Quentin Schulz > --- > > v2: > - adding small explanation on AXP223 variation compared to the AXP221, > >

Re: [RFC 07/10] kmod: use simplified rate limit printk

2016-12-14 Thread Petr Mladek
On Thu 2016-12-08 11:49:01, Luis R. Rodriguez wrote: > Just use the simplified rate limit printk when the max modprobe > limit is reached, while at it throw out a bone should the error > be triggered. > > Signed-off-by: Luis R. Rodriguez > --- > kernel/kmod.c | 10 ++ > 1 file changed, 2

[PATCH V2 2/4] powerpc: add helper to check if offset is within rel branch range

2016-12-14 Thread Anju T Sudhakar
From: "Naveen N. Rao" Signed-off-by: Naveen N. Rao Signed-off-by: Anju T Sudhakar --- arch/powerpc/include/asm/code-patching.h | 1 + arch/powerpc/lib/code-patching.c | 24 +++- 2 files changed, 24 insertions(+), 1 deletion(-) diff --git a/arch/powerpc/include/asm

[PATCH V2 3/4] arch/powerpc: Implement Optprobes

2016-12-14 Thread Anju T Sudhakar
Detour buffer contains instructions to create an in memory pt_regs. After the execution of the pre-handler, a call is made for instruction emulation. The NIP is determined in advanced through dummy instruction emulation and a branch instruction is created to the NIP at the end of the trampoline.

[PATCH V2 1/4] powerpc: asm/ppc-opcode.h: introduce __PPC_SH64()

2016-12-14 Thread Anju T Sudhakar
From: "Naveen N. Rao" Introduce __PPC_SH64() as a 64-bit variant to encode shift field in some of the shift and rotate instructions operating on double-words. Convert some of the BPF instruction macros to use the same. Signed-off-by: Naveen N. Rao --- arch/powerpc/include/asm/ppc-opcode.h | 1

[PATCH V2 0/4] OPTPROBES for powerpc

2016-12-14 Thread Anju T Sudhakar
This is the V2 patchset of the kprobes jump optimization (a.k.a OPTPROBES)for powerpc. Kprobe being an inevitable tool for kernel developers, enhancing the performance of kprobe has got much importance. Currently kprobes inserts a trap instruction to probe a running kernel. Jump optimization allow

[PATCH V2 4/4] arch/powerpc: Optimize kprobe in kretprobe_trampoline

2016-12-14 Thread Anju T Sudhakar
Kprobe placed on the kretprobe_trampoline during boot time can be optimized, since the instruction at probe point is a 'nop'. Signed-off-by: Anju T Sudhakar --- arch/powerpc/kernel/kprobes.c | 8 arch/powerpc/kernel/optprobes.c | 7 +++ 2 files changed, 11 insertions(+), 4 delet

Re: usb/gadget: warning in ep_write_iter/__alloc_pages_nodemask

2016-12-14 Thread Michal Hocko
On Wed 14-12-16 11:13:11, Alan Stern wrote: > On Wed, 14 Dec 2016, Michal Hocko wrote: > > > On Tue 13-12-16 08:33:34, Alan Stern wrote: > > > On Tue, 13 Dec 2016, Michal Hocko wrote: [...] > > > > Well, my point was that it is not really hard to imagine to deplete > > > > larger contiguous memory

[PATCH v3 2/2] iio: adc: hx711: Add IIO driver for AVIA HX711

2016-12-14 Thread Andreas Klinger
This is the IIO driver for AVIA HX711 ADC which ist mostly used in weighting cells. The protocol is quite simple and using GPIOs: One GPIO is used as clock (SCK) while another GPIO is read (DOUT) The raw value read from the chip is delivered. To get a weight one needs to subtract the zero offset

Re: [PATCH v2 2/4] KVM: x86: replace kvm_apic_id with kvm_{x,x2}apic_id

2016-12-14 Thread David Hildenbrand
kvm_for_each_vcpu(i, vcpu, kvm) if (kvm_apic_present(vcpu)) - max_id = max(max_id, kvm_apic_id(vcpu->arch.apic)); + max_id = max(max_id, kvm_x2apic_id(vcpu->arch.apic)); new = kvm_kvzalloc(sizeof(struct kvm_apic_map) +

Re: [PATCH] spi: SPI_FSL_DSPI should depend on HAS_DMA

2016-12-14 Thread Mark Brown
On Wed, Dec 14, 2016 at 03:40:43PM +0100, Geert Uytterhoeven wrote: > On Wed, Dec 14, 2016 at 3:37 PM, Mark Brown wrote: > > On Wed, Dec 14, 2016 at 01:28:05PM +0100, Geert Uytterhoeven wrote: > > Honestly I think we should just fix the architectures that don't support > > DMA to provide compile

Re: Fw: [lkp-developer] [sched,rcu] cf7a2dca60: [No primary change] +186% will-it-scale.time.involuntary_context_switches

2016-12-14 Thread Michal Hocko
On Wed 14-12-16 03:06:09, Paul E. McKenney wrote: > On Wed, Dec 14, 2016 at 10:54:25AM +0100, Michal Hocko wrote: > > On Tue 13-12-16 07:14:08, Paul E. McKenney wrote: > > > Just FYI for the moment... > > > > > > So even with the slowed-down checking, making cond_resched() do what > > > cond_resch

[PATCH v3 1/2] iio: adc: hx711: Add DT binding for avia,hx711

2016-12-14 Thread Andreas Klinger
Add DT bindings for avia,hx711 Add vendor avia to vendor list Signed-off-by: Andreas Klinger --- Documentation/devicetree/bindings/iio/adc/avia-hx711.txt | 16 Documentation/devicetree/bindings/vendor-prefixes.txt| 1 + 2 files changed, 17 insertions(+) create mode 100644

Re: [RFC/PATCH] tracing: show total NMI time in hwlat tracer

2016-12-14 Thread Steven Rostedt
[ added the forgotten LKML ;-) ] On Wed, 14 Dec 2016 09:53:01 -0500 Steven Rostedt wrote: > On Wed, 14 Dec 2016 16:13:51 +0900 > Namhyung Kim wrote: > > > Hi Steve, > > > > While looking at the code I found the below. I'm not sure the current > > code is correct but it seems that nmi_total_

[PATCH v3 0/2] iio: adc: hx711: Add IIO driver for AVIA HX711 ADC

2016-12-14 Thread Andreas Klinger
This series adds IIO driver support for the AVIA HX711 ADC which is mostly used in weighting cells. The first patch adds the new DT binding for which a new vendor avia was also added. The second patch is the simple IIO driver implemented as ADC. The protocol is specific to this device and implem

Re: [PATCH 0/2] Add maintainers to the admin guide

2016-12-14 Thread Joe Perches
On Tue, 2016-12-13 at 07:38 -0200, Mauro Carvalho Chehab wrote: > Em Mon, 12 Dec 2016 12:56:50 -0800 > Joe Perches escreveu: > > Does the boxing with the === blocks align properly? > > It it really useful? Is there another/better way? > > Do you mean those? > > ===

Re: [PATCH] ARM: dts: Add missing CPU frequencies for Exynos5422/5800

2016-12-14 Thread Bartlomiej Zolnierkiewicz
Hi, On Wednesday, December 14, 2016 11:40:08 AM Javier Martinez Canillas wrote: > Hello Bartlomiej, > > On 12/14/2016 11:25 AM, Bartlomiej Zolnierkiewicz wrote: > > > > Hi, > > > > On Wednesday, December 14, 2016 11:06:45 AM Javier Martinez Canillas wrote: > >> > >> Hello Bartlomiej, > >> > >>

Re: partial bluetooth success on n900 [was Re: bluetooth/uart timeout handling]

2016-12-14 Thread Sebastian Reichel
Hi Tony, On Wed, Dec 14, 2016 at 07:10:56AM -0800, Tony Lindgren wrote: > * Sebastian Reichel [161214 05:32]: > > Hi Pali & Pavel, > > > > On Wed, Dec 14, 2016 at 01:53:23PM +0100, Pavel Machek wrote: > > > > > [ 220.248596] tty ttyO1: Radio packet sent > > > > > [ 220.249328] Bluetooth: hci0:

Re: [RFC 06/10] kmod: provide sanity check on kmod_concurrent access

2016-12-14 Thread Petr Mladek
On Thu 2016-12-08 11:48:50, Luis R. Rodriguez wrote: > Only decrement *iff* we're possitive. Warn if we've hit > a situation where the counter is already 0 after we're done > with a modprobe call, this would tell us we have an unaccounted > counter access -- this in theory should not be possible as

Re: [PATCH] tools: usb: usbip: Update README

2016-12-14 Thread Shuah Khan
Hi Krzysztof, Thanks for the patch. On 12/13/2016 12:52 PM, Krzysztof Opasiak wrote: > Update README file: > - remove outdated parts > - clarify terminology and general structure > - add some description of vUDC > > Signed-off-by: Krzysztof Opasiak > --- > tools/usb/usbip/README | 56 > ++

Re: [PATCH 5/8] dax: refactor locking out of size calculation routines

2016-12-14 Thread Dan Williams
On Wed, Dec 14, 2016 at 7:01 AM, Johannes Thumshirn wrote: > Hi Dan, > > On Sat, Dec 10, 2016 at 10:28:51PM -0800, Dan Williams wrote: >> In preparation for other callers of these routines make the locking the >> responsibility of the caller. >> >> Signed-off-by: Dan Williams >> --- >> drivers/d

Re: partial bluetooth success on n900 [was Re: bluetooth/uart timeout handling]

2016-12-14 Thread Tony Lindgren
* Sebastian Reichel [161214 07:52]: > On Wed, Dec 14, 2016 at 07:10:56AM -0800, Tony Lindgren wrote: > > Maybe send it so we can merge it as a fix during the early -rc > > cycle? > > Sorry if I was not clear enough: mainline does *not* contain > incorrect DT information. My bluetooth RFC patches

Re: [PATCH] perf/x86: Fix overlap counter scheduling bug

2016-12-14 Thread Jiri Olsa
On Wed, Nov 09, 2016 at 04:51:53PM +0100, Peter Zijlstra wrote: SNIP > > As per a prior mail, the masks on the PMU in question are: > > 0x01 - 0001 > 0x03 - 0011 > 0x0e - 1110 > 0x0c - 1100 > > But since all the masks that have overlap (0xe -> {0xc,0x3}) and (0x3 -> > 0x1) are of heavier w

Re: [PATCH 1/8] dax: add region-available-size attribute

2016-12-14 Thread Dan Williams
On Wed, Dec 14, 2016 at 6:38 AM, Johannes Thumshirn wrote: > Hi Dan, > > On Sat, Dec 10, 2016 at 10:28:30PM -0800, Dan Williams wrote: >> In preparation for a facility that enables dax regions to be >> sub-divided, introduce a 'dax/available_size' attribute. This attribute >> appears under the pa

Re: [RFC 03/10] kmod: add dynamic max concurrent thread count

2016-12-14 Thread Petr Mladek
On Thu 2016-12-08 11:48:14, Luis R. Rodriguez wrote: > We currently statically limit the number of modprobe threads which > we allow to run concurrently to 50. As per Keith Owens, this was a > completely arbitrary value, and it was set in the 2.3.38 days [0] > over 16 years ago in year 2000. > > A

Re: [PATCH v5] media: Driver for Toshiba et8ek8 5MP sensor

2016-12-14 Thread Ivaylo Dimitrov
Hi On 14.12.2016 15:03, Pali Rohár wrote: Hi! See inlined some my notes. + +#ifdef USE_CRC + rval = et8ek8_i2c_read_reg(client, ET8EK8_REG_8BIT, 0x1263, &val); + if (rval) + goto out; +#if USE_CRC /* TODO get crc setting from DT */ + val |= BIT(4); +#else +

Re: [PATCH v2 2/4] KVM: x86: replace kvm_apic_id with kvm_{x,x2}apic_id

2016-12-14 Thread Paolo Bonzini
On 14/12/2016 14:41, Radim Krčmář wrote: > 2016-12-14 13:39+0100, David Hildenbrand: >> Am 13.12.2016 um 17:29 schrieb Radim Krčmář: >>> There were three calls sites: >>> - recalculate_apic_map and kvm_apic_match_physical_addr, where it would >>>only complicate implementation of x2APIC hotpl

Re: [PATCH] usb: musb: debugfs: allow forcing host mode together with speed in testmode

2016-12-14 Thread Tony Lindgren
* Pali Rohár [161214 07:40]: > On Wednesday 14 December 2016 16:34:31 Tony Lindgren wrote: > > * Pali Rohár [161214 06:48]: > > > > We need this because and this allows debugging problems related > > to ...? > > Hi! We have already discussion about it, search for older email with > Messag

Re: [PATCH v9 3/3] fpga: Add support for Lattice iCE40 FPGAs

2016-12-14 Thread Alan Tull
On Fri, 9 Dec 2016, Moritz Fischer wrote: > Joel, > > A couple of minor nits below (none of them are real blockers), other > than that looks good. > > On Thu, Dec 8, 2016 at 9:35 PM, Joel Holdsworth > wrote: > > The Lattice iCE40 is a family of FPGAs with a minimalistic architecture > > and ver

Re: [PATCH v2 01/11] power: supply: axp20x_usb_power: use of_device_id data field instead of device_is_compatible

2016-12-14 Thread Chen-Yu Tsai
On Fri, Dec 9, 2016 at 7:04 PM, Quentin Schulz wrote: > This replaces calls to of_device_is_compatible to check data field of > of_device_id matched when probing the driver. > > Signed-off-by: Quentin Schulz Acked-by: Chen-Yu Tsai

Re: [PATCH v2 03/11] power: supply: axp20x_usb_power: set min voltage and max current from sysfs

2016-12-14 Thread Chen-Yu Tsai
On Fri, Dec 9, 2016 at 7:04 PM, Quentin Schulz wrote: > AXP20X and AXP22X PMICs allow setting the min voltage and max current of > VBUS power supply. This adds entries in sysfs to allow to do so. > > Signed-off-by: Quentin Schulz Acked-by: Chen-Yu Tsai

Re: [PATCH v2 02/11] mfd: axp20x: add volatile and writeable reg ranges for VBUS power supply driver

2016-12-14 Thread Chen-Yu Tsai
On Fri, Dec 9, 2016 at 7:04 PM, Quentin Schulz wrote: > The X-Powers AXP20X and AXP22X PMICs allow to choose the maximum voltage > and minimum current delivered by the VBUS power supply. > > This adds the register used by the VBUS power supply driver to the range > of volatile and writeable regs r

Re: [PATCH] usb: musb: debugfs: allow forcing host mode together with speed in testmode

2016-12-14 Thread Pali Rohár
On Wednesday 14 December 2016 16:34:31 Tony Lindgren wrote: > * Pali Rohár [161214 06:48]: > > We need this because and this allows debugging problems related > to ...? Hi! We have already discussion about it, search for older email with Message-Id: <201601231357.32629@pali> After that in

Re: [Qemu-devel] [PATCH v7 1/1] crypto: add virtio-crypto driver

2016-12-14 Thread Halil Pasic
On 12/14/2016 12:50 PM, Gonglei wrote: > diff --git a/drivers/crypto/virtio/virtio_crypto_core.c > b/drivers/crypto/virtio/virtio_crypto_core.c > new file mode 100644 > index 000..c0854a1 > --- /dev/null > +++ b/drivers/crypto/virtio/virtio_crypto_core.c > @@ -0,0 +1,474 @@ [..] > + > +stati

Re: [PATCH] usb: musb: debugfs: allow forcing host mode together with speed in testmode

2016-12-14 Thread Tony Lindgren
* Pali Rohár [161214 06:48]: We need this because and this allows debugging problems related to ...? Regards, Tony

[PATCH driver-core] debugfs: improve formatting of debugfs_real_fops()

2016-12-14 Thread Jakub Kicinski
Type of debugfs_real_fops() is longer than parameters and the name, so there is no way to break the declaration nicely. We have to go over 80 characters. Signed-off-by: Jakub Kicinski --- include/linux/debugfs.h | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/include/linux/

Re: [OpenRISC] [PATCH] openrisc: Add _text symbol to fix ksym build error

2016-12-14 Thread Geert Uytterhoeven
Hi Stafford, On Wed, Dec 14, 2016 at 1:55 PM, Stafford Horne wrote: > The build report reports: > >.tmp_kallsyms1.o: In function `kallsyms_relative_base': >>> (.rodata+0x8a18): undefined reference to `_text' > > This is when using 'make alldefconfig'. Adding this _text symbol to mark > the st

Moratorium on coding style patches (was Re: [PATCH] include/linux/kernel.h: fixed coding style issues)

2016-12-14 Thread Alexey Dobriyan
OK, someone needs to say it. These type of patches are advertised by some people as a good way to enter Linux kernel development. You know, to learn how the process works, how to setup email client pipeline, to get initial feedback. And it is true. What those people aren't saying is that the above

Re: [PATCH v2] ARM: dts: sunxi: Change node name for pwrseq pin on Olinuxino-lime2-emmc

2016-12-14 Thread Maxime Ripard
On Wed, Dec 14, 2016 at 03:57:24PM +0100, Emmanuel Vadot wrote: > The node name for the power seq pin is mmc2@0 like the mmc2_pins_a one. > This makes the original node (mmc2_pins_a) scrapped out of the dtb and > result in a unusable eMMC if U-Boot didn't configured the pins to the > correct functi

Re: linux-next: build warning after merge of the btrfs-kdave tree

2016-12-14 Thread David Sterba
On Wed, Dec 07, 2016 at 09:38:19AM +1100, Stephen Rothwell wrote: > Hi David, > > On Mon, 28 Nov 2016 11:02:30 +1100 Stephen Rothwell > wrote: > > > > After merging the btrfs-kdave tree, today's linux-next build (powerpc > > ppc64_defconfig) produced this warning: > > > > fs/btrfs/inode.c: In f

Re: [PATCH v2 2/3] power: supply: bq24735-charger: optionally poll the ac-detect gpio

2016-12-14 Thread Sebastian Reichel
Hi, On Wed, Dec 14, 2016 at 12:56:44AM +0100, Peter Rosin wrote: > If the ac-detect gpio does not support interrupts, provide a fallback > to poll the gpio at a configurable interval. > > Signed-off-by: Peter Rosin > --- > > [...] > > + } else if (charger->status_gpio) { > + ret

[PATCH 1/1] remoteproc: fix vdev reference management

2016-12-14 Thread Loic Pallardy
Commit 2b45cef5868a ("remoteproc: Further extend the vdev life cycle") extends kref support for vdev management. It introduces a regression when following sequence is executed: rproc_boot --> rproc_shutdown --> rproc_boot Second rproc_boot call crashes on register_virtio_device as device is already

Re: Build failures due to missing 'posix_timer_event'

2016-12-14 Thread Nicolas Pitre
On Wed, 14 Dec 2016, Guenter Roeck wrote: > avr32:allnoconfig: > > kernel/built-in.o: In function `do_adjtimex': > (.text+0x1d748): undefined reference to `posix_timer_event' > make[1]: *** [vmlinux] Error 1 > > metag:allnoconfig: > > kernel/built-in.o: In function `alarm_handle_timer': > alarm

Re: [PATCH] x86/platform/intel-mid: switch MPU3050 driver to IIO

2016-12-14 Thread Andy Shevchenko
On Wed, 2016-12-14 at 14:39 +0100, Linus Walleij wrote: > The Intel Mid goes in and creates a I2C device for the > MPU3050 if the input driver for MPU-3050 is activated. > As of commit 3904b28efb2c > ("iio: gyro: Add driver for the MPU-3050 gyroscope") > there is a proper and fully featured IIO dri

Re: [PATCH v2 1/4] siphash: add cryptographically secure hashtable function

2016-12-14 Thread Hannes Frederic Sowa
Hello, On 14.12.2016 14:10, Jason A. Donenfeld wrote: > On Wed, Dec 14, 2016 at 12:21 PM, Hannes Frederic Sowa > wrote: >> Can you show or cite benchmarks in comparison with jhash? Last time I >> looked, especially for short inputs, siphash didn't beat jhash (also on >> all the 32 bit devices etc

Re: partial bluetooth success on n900 [was Re: bluetooth/uart timeout handling]

2016-12-14 Thread Tony Lindgren
* Sebastian Reichel [161214 05:32]: > Hi Pali & Pavel, > > On Wed, Dec 14, 2016 at 01:53:23PM +0100, Pavel Machek wrote: > > > > [ 220.248596] tty ttyO1: Radio packet sent > > > > [ 220.249328] Bluetooth: hci0: Frame reassembly failed (-84) > > > > [ 220.272949] tty ttyO1: wakeup received: 1 -

[PATCH 1/2] MIPS: Kconfig: Set default MIPS system type as generic

2016-12-14 Thread Matt Redfearn
The generic MIPS system type allows building a board agnostic kernel and should be the default starting point for users, so set it as the default system type in Kconfig. Since ip22 is no longer the default, update ip22_defconfig to select CONFIG_SGI_IP22. Signed-off-by: Matt Redfearn --- This p

Re: [PATCHv6] support for AD5820 camera auto-focus coil

2016-12-14 Thread Tony Lindgren
* Pali Rohár [161214 05:38]: > On Monday 08 August 2016 23:41:32 Pavel Machek wrote: > > On Mon 2016-08-08 11:09:56, Sakari Ailus wrote: > > > On Fri, Aug 05, 2016 at 12:26:11PM +0200, Pavel Machek wrote: > > > > This adds support for AD5820 autofocus coil, found for example in > > > > Nokia N900

[PATCH 2/2] MIPS: Set defconfig target to a generic system for 32r2el

2016-12-14 Thread Matt Redfearn
The default system type should be a MIPS generic kernel. In order to include some level of board support, select a 32r2el generic defconfig by default. The alternative would be to use "generic_defconfig" but rather unintuitvely that is a bare bones configuration with no platform support so is not u

[PATCH] mm: consolidate GFP_NOFAIL checks in the allocator slowpath

2016-12-14 Thread Michal Hocko
From: Michal Hocko Tetsuo Handa has pointed out that 0a0337e0d1d1 ("mm, oom: rework oom detection") has subtly changed semantic for costly high order requests with __GFP_NOFAIL and withtout __GFP_REPEAT and those can fail right now. My code inspection didn't reveal any such users in the tree but

Re: [PATCH 5/7] blk-mq-sched: add framework for MQ capable IO schedulers

2016-12-14 Thread Jens Axboe
On 12/14/2016 03:31 AM, Bart Van Assche wrote: > On 12/13/2016 04:14 PM, Jens Axboe wrote: >> On 12/13/2016 06:56 AM, Bart Van Assche wrote: >>> On 12/08/2016 09:13 PM, Jens Axboe wrote: +struct request *blk_mq_sched_alloc_shadow_request(struct request_queue *q, +

Re: [RESEND PATCH 1/2] arm64: change from CONT_PMD_SHIFT to CONT_PTE_SHIFT

2016-12-14 Thread Ard Biesheuvel
On 14 December 2016 at 14:19, zhongjiang wrote: > From: zhong jiang > > I think that CONT_PTE_SHIFT is more reasonable even if they are some > value. and the patch is not any functional change. > This may be the case for 64k pages, but not for 16k pages, and I actually think add_default_hugepage

Re: [PATCH 6/7] mq-deadline: add blk-mq adaptation of the deadline IO scheduler

2016-12-14 Thread Jens Axboe
On 12/14/2016 01:09 AM, Bart Van Assche wrote: > On 12/08/2016 09:13 PM, Jens Axboe wrote: >> +static inline bool dd_rq_is_shadow(struct request *rq) >> +{ >> +return rq->rq_flags & RQF_ALLOCED; >> +} > > Hello Jens, > > Something minor: because req_flags_t has been defined using __bitwise >

Re: [PATCH 5/8] dax: refactor locking out of size calculation routines

2016-12-14 Thread Johannes Thumshirn
Hi Dan, On Sat, Dec 10, 2016 at 10:28:51PM -0800, Dan Williams wrote: > In preparation for other callers of these routines make the locking the > responsibility of the caller. > > Signed-off-by: Dan Williams > --- > drivers/dax/dax.c | 30 -- > 1 file changed, 24 i

[char-misc 2/2] mei: move write cb to completion on credentials failures

2016-12-14 Thread Tomas Winkler
From: Alexander Usyskin The credentials handling was pushed to the write handlers but error handling wasn't done properly. Move write callbacks to completion queue to destroy them and to notify a blocked writer about the failure Fixes: 136698e535cd1 (mei: push credentials inside the irq write ha

[char-misc 1/2] mei: bus: fix mei_cldev_enable KDoc

2016-12-14 Thread Tomas Winkler
From: Alexander Usyskin Adjust function name in KDoc. Fixes: d49dc5e76fc9 (mei: bus: use mei_cldev_ prefix for the API functions) Signed-off-by: Alexander Usyskin Signed-off-by: Tomas Winkler --- drivers/misc/mei/bus.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/driver

Re: [PATCH 3/3] crypto: brcm: Add Broadcom SPU driver DT entry.

2016-12-14 Thread Rob (William) Rice
I will rebase to Herbert's latest when I submit v3 to address Mark Rutland's DT comments (and any others that come in). Rob On 12/10/2016 7:14 PM, kbuild test robot wrote: Hi Rob, [auto build test ERROR on cryptodev/master] [also build test ERROR on v4.9-rc8] [cannot apply to next-20161209]

Question about regulator API

2016-12-14 Thread Harald Geyer
Hi all! I have a quite complex situation with regulator consumers, which I'm not sure how to handle best. Suppose there is some quirky HW that after some random time crashes - i.e. it doesn't respond to requests anymore at all. The driver can detect this situation and the only way to fix it is to

[PATCH v2] ARM: dts: sunxi: Change node name for pwrseq pin on Olinuxino-lime2-emmc

2016-12-14 Thread Emmanuel Vadot
The node name for the power seq pin is mmc2@0 like the mmc2_pins_a one. This makes the original node (mmc2_pins_a) scrapped out of the dtb and result in a unusable eMMC if U-Boot didn't configured the pins to the correct functions. Changes since v1: * Rename the node mmc2-rst-pin Signed-off-by:

Re: [PATCHv3 perf/core 0/7] Reuse libbpf from samples/bpf

2016-12-14 Thread Arnaldo Carvalho de Melo
Em Wed, Dec 14, 2016 at 10:25:01AM -0300, Arnaldo Carvalho de Melo escreveu: > Em Fri, Dec 09, 2016 at 04:30:54PM +0100, Daniel Borkmann escreveu: > > On 12/09/2016 04:09 PM, Arnaldo Carvalho de Melo wrote: > > > > v3: Add ack for first patch. > > > > Split out second patch from v2 into separa

Re: [PATCH] rcu: shift by 1UL rather than 1 to fix sign extension error

2016-12-14 Thread Colin Ian King
On 14/12/16 14:42, Boqun Feng wrote: > On Tue, Dec 13, 2016 at 11:33:19AM +, Colin Ian King wrote: >> On 13/12/16 11:21, Boqun Feng wrote: >>> On Tue, Dec 13, 2016 at 10:56:46AM +, Colin King wrote: From: Colin Ian King mask and bit are unsigned longs, so if bit is 31 we end

[PATCH 2/3] oom, trace: Add oom detection tracepoints

2016-12-14 Thread Michal Hocko
From: Michal Hocko should_reclaim_retry is the central decision point for declaring the OOM. It might be really useful to expose data used for this decision making when debugging an unexpected oom situations. Say we have an OOM report: [ 52.264001] mem_eater invoked oom-killer: gfp_mask=0x242

[PATCH 0/3] mm, oom: add oom detection tracepoints

2016-12-14 Thread Michal Hocko
Hi, this is a long overdue and I am really sorry about that. I just didn't get to sit and come up with this earlier as there was always some going on which preempted it. This patchset adds two tracepoints which should help us to debug oom decision making. The first one is placed in should_reclaim_r

[PATCH 1/3] mm, trace: extract COMPACTION_STATUS and ZONE_TYPE to a common header

2016-12-14 Thread Michal Hocko
From: Michal Hocko COMPACTION_STATUS resp. ZONE_TYPE are currently used to translate enum compact_result resp. struct zone index into their symbolic names for an easier post processing. The follow up patch would like to reuse this as well. The code involves some preprocessor black magic which is

[PATCH 3/3] oom, trace: add compaction retry tracepoint

2016-12-14 Thread Michal Hocko
From: Michal Hocko Higher order requests oom debugging is currently quite hard. We do have some compaction points which can tell us how the compaction is operating but there is no trace point to tell us about compaction retry logic. This patch adds a one which will have the following format

[PATCH] usb: musb: debugfs: allow forcing host mode together with speed in testmode

2016-12-14 Thread Pali Rohár
Signed-off-by: Pali Rohár --- drivers/usb/musb/musb_debugfs.c | 44 +-- 1 file changed, 28 insertions(+), 16 deletions(-) diff --git a/drivers/usb/musb/musb_debugfs.c b/drivers/usb/musb/musb_debugfs.c index 9b22d94..62c13a2 100644 --- a/drivers/usb/musb/musb

RE: [PATCH v2 3/4] secure_seq: use siphash24 instead of md5_transform

2016-12-14 Thread David Laight
From: Jason A. Donenfeld > Sent: 14 December 2016 13:44 > To: Hannes Frederic Sowa > > __packed not only removes all padding of the struct but also changes the > > alignment assumptions for the whole struct itself. The rule, the struct > > is aligned by its maximum alignment of a member is no longe

<    1   2   3   4   5   6   7   8   9   >