Re: [PATCH 7/8] LinuxPPS: pps_gen_parport: Add polarity parameter for inverted signal.

2017-03-03 Thread Rodolfo Giometti
On 02/15/17 15:31, Andrey Drobyshev wrote: From: Alexander GQ Gerasiov On some devices it may be necessary to transmit inverted data. This commit simply adds polarity parameter to define which state represents presence of signal: it equals 0 if signal is on the low level (default), or 1 if sign

Re: [PATCH 6/8] LinuxPPS: pps_parport: Ignore interrupt invoked less than 0.5sec after previous.

2017-03-03 Thread Rodolfo Giometti
On 02/15/17 15:31, Andrey Drobyshev wrote: From: Alexander GQ Gerasiov On some devices interrupt may be invoked by parasitic assert event produced while switching from high to low. Such interrupt should be ignored. Signed-off-by: Alexander GQ Gerasiov Acked-by: Rodolfo Giometti

Crypto Fixes for 4.11

2017-03-03 Thread Herbert Xu
Hi Linus: This push fixes the following issues: - vmalloc stack regression in CCM. - Build problem in CRC32 on ARM. - Memory leak in cavium. - Missing Kconfig dependencies in atmel and mediatek. - XTS Regression on some platforms (s390 and ppc). - Memory overrun in CCM test vector. Please pull

Re: [PATCH 1/2] pps: add ioctl_compat function to correct ioctl definitions

2017-03-03 Thread Rodolfo Giometti
On 02/24/17 21:23, Matt Ranostay wrote: ioctl definitions use the pointer size of the architecture which is fine when userspace and kernel are the same bitsize. This patchset workarounds an issue with mixed bitsize kernel + userspace by rewriting the cmd to the kernelspace architecture pointer si

Re: [PATCH 5/8] LinuxPPS: pps_parport: Do not generate assert in case of lost signal.

2017-03-03 Thread Rodolfo Giometti
On 02/15/17 15:31, Andrey Drobyshev wrote: From: Alexander GQ Gerasiov Since clear timeout is non-zero, some clear event capture is requested. Therefore, if signal is lost we shouldn't generate assert event alone. Signed-off-by: Alexander GQ Gerasiov Acked-by: Rodolfo Giometti

Re: [PATCH 4/8] LinuxPPS: kapi: Unlock before waking up events queue in pps_event().

2017-03-03 Thread Rodolfo Giometti
On 02/15/17 15:31, Andrey Drobyshev wrote: From: Alexander GQ Gerasiov Otherwise we get "scheduling while atomic" problem. Signed-off-by: Alexander GQ Gerasiov Acked-by: Rodolfo Giometti

Re: [PATCH 3/8] hardpps: fix some pps_jitter issues.

2017-03-03 Thread Rodolfo Giometti
On 02/15/17 15:31, Andrey Drobyshev wrote: Handle possible overflow, implementation-defined result of signed right shift and replace unsuitable constant. Signed-off-by: Andrey Drobyshev Signed-off-by: Alexander GQ Gerasiov Acked-by: Rodolfo Giometti

Re: [PATCH 1/8] hardpps: simple fixes replacing clumsy code with abs() macro.

2017-03-03 Thread Rodolfo Giometti
On 02/15/17 15:31, Andrey Drobyshev wrote: From: Alexander GQ Gerasiov Here are some trivial fixes, including: * simplify comparisons by using abs() macro * remove unnecessary variables * remove duplicate headers * fix typos Signed-off-by: Alexander GQ Gerasiov Acked-by: Rodolfo Gio

Re: [GIT PULL] sched.h split-up

2017-03-03 Thread Ingo Molnar
* Linus Torvalds wrote: > Anyway, I fixed the semantic merge error by just including thar > file, but this is just not acceptable. > > Having to have files know that if they use the wait-event functins > (well, _some_ of them), they not only need to include , > they need to completely illogica

Re: [PATCH] staging: ks7010: clean up code

2017-03-03 Thread Joe Perches
On Sat, 2017-03-04 at 08:49 +0200, Ernestas Kulik wrote: > On Fri, 2017-03-03 at 22:37-0800, Joe Perches wrote: > > > > struct hostif_hdr.event is declared at uint16_t > > and not as __le16 so I believe this is incorrect > > and actually introduces a sparse error. > > Sure, but I change that in t

[PATCH v2] mmc: mediatek: Fixed bug where clock frequency could be set wrong

2017-03-03 Thread Yong Mao
yong mao (1): mmc: mediatek: Fixed bug where clock frequency could be set wrong drivers/mmc/host/mtk-sd.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) -- 1.8.1.1.dirty

[PATCH] mmc: mediatek: Fixed bug where clock frequency could be set wrong

2017-03-03 Thread Yong Mao
From: yong mao This patch can fix two issues: Issue 1: In previous code, div may be overflow when setting clock frequency as f_min. We can use DIV_ROUND_UP to fix this boundary related issue. Issue 2: In previous code, we can not set the correct clock frequency when div equals 0xff. Signed-off

Re: [PATCH v1] mmc: mediatek: Fixed bug where clock frequency could be set wrong

2017-03-03 Thread Yong Mao
On Thu, 2017-03-02 at 20:20 +0800, Daniel Kurtz wrote: > On Wed, Mar 1, 2017 at 5:56 PM, Yong Mao wrote: > > On Tue, 2017-02-28 at 14:56 +0800, Daniel Kurtz wrote: > >> On Fri, Feb 24, 2017 at 5:38 PM, Yong Mao wrote: > >> > From: Yong Mao > >> > To: Daniel Kurtz > >> > Subject:Re

Re: [PATCH] staging: ks7010: clean up code

2017-03-03 Thread Ernestas Kulik
On Fri, 2017-03-03 at 22:37-0800, Joe Perches wrote: > > struct hostif_hdr.event is declared at uint16_t > and not as __le16 so I believe this is incorrect > and actually introduces a sparse error. Sure, but I change that in the patch. :) On Sat, 2017-03-04 at 08:28 +0200, Ernestas Kulik wrote: >

Re: Is it really safe to use workqueues to drive expedited grace periods?

2017-03-03 Thread Paul E. McKenney
On Fri, Mar 03, 2017 at 11:30:49AM -0800, Tejun Heo wrote: > Hello, Paul. > > Sorry about the long delay. Travel + sickness. Just starting to > catch up with things now. No problem, I have been distracted by other projects in any case. ;-) > On Mon, Feb 13, 2017 at 04:16:00PM -0800, Paul E. M

Re: [PATCH] staging: ks7010: clean up code

2017-03-03 Thread Joe Perches
On Sat, 2017-03-04 at 08:28 +0200, Ernestas Kulik wrote: > On Fri, 2017-03-03 at 17:23-0800, Joe Perches wrote: > > > On Fri, 2017-03-03 at 22:58 +0200, Ernestas Kulik wrote: > > > > > This fixes type warnings generated by sparse, replaces instances of > > > ntohs() with be16_to_cpu() and removes

Re: [PATCH] staging: ks7010: clean up code

2017-03-03 Thread Ernestas Kulik
On Fri, 2017-03-03 at 17:23-0800, Joe Perches wrote: > On Fri, 2017-03-03 at 22:58 +0200, Ernestas Kulik wrote: > > > This fixes type warnings generated by sparse, replaces instances of > > ntohs() with be16_to_cpu() and removes unused fields in structs. > > There's no real need to convert ntohs

Re: [PATCH v5 3/4] iio: accel: adxl345: Split driver into core and I2C

2017-03-03 Thread Eva Rachel Retuya
On Fri, Mar 03, 2017 at 06:51:28PM +0200, Andy Shevchenko wrote: > On Tue, Feb 28, 2017 at 4:37 AM, Eva Rachel Retuya > wrote: > > Move I2C-specific code into its own file and rely on regmap to access > > registers. The core code provides access to x, y, z and scale readings. > > Portion of mino

Re: [RFC] arm64: support HAVE_ARCH_RARE_WRITE

2017-03-03 Thread Hoeun Ryu
> On Mar 4, 2017, at 5:50 AM, Andy Lutomirski wrote: > >> On Thu, Mar 2, 2017 at 7:00 AM, Hoeun Ryu wrote: >> +unsigned long __rare_write_rw_alias_start = TASK_SIZE_64 / 4; >> + >> +__always_inline unsigned long __arch_rare_write_map(void) >> +{ >> + struct mm_struct *mm = &rare_write_mm;

Re: [PATCH v5 4/4] iio: accel: adxl345: Add SPI support

2017-03-03 Thread Eva Rachel Retuya
On Fri, Mar 03, 2017 at 06:55:26PM +0200, Andy Shevchenko wrote: > On Tue, Feb 28, 2017 at 4:37 AM, Eva Rachel Retuya > wrote: > > Add SPI driver that initializes SPI regmap for the adxl345 core driver. > > The driver supports the same functionality as I2C namely the x, y, z and > > scale reading

Re: [PATCH 1/4] net: thunderx: Fix IOMMU translation faults

2017-03-03 Thread Sunil Kovvuri
On Fri, Mar 3, 2017 at 11:26 PM, David Miller wrote: > From: sunil.kovv...@gmail.com > Date: Fri, 3 Mar 2017 16:17:47 +0530 > >> @@ -1643,6 +1650,9 @@ static int nicvf_probe(struct pci_dev *pdev, const >> struct pci_device_id *ent) >> if (!pass1_silicon(nic->pdev)) >> nic->hw

Re: [RFC] arm64: support HAVE_ARCH_RARE_WRITE

2017-03-03 Thread Hoeun Ryu
> On Mar 3, 2017, at 1:02 PM, Kees Cook wrote: > >> On Thu, Mar 2, 2017 at 7:00 AM, Hoeun Ryu wrote: >> This RFC is a quick and dirty arm64 implementation for Kees Cook's RFC for >> rare_write infrastructure [1]. > > Awesome! :) > >> This implementation is based on Mark Rutland's suggestions,

Re: [PATCH v5 2/4] iio: accel: adxl345: Use I2C regmap instead of direct I2C access

2017-03-03 Thread Eva Rachel Retuya
On Fri, Mar 03, 2017 at 06:44:27PM +0200, Andy Shevchenko wrote: > On Tue, Feb 28, 2017 at 4:37 AM, Eva Rachel Retuya > wrote: > > Convert the driver to use regmap instead of I2C-specific functions. This > > is done in preparation for splitting this driver into core and > > I2C-specific code as w

[PATCH] KVM: nVMX: Reset nested_run_pending if the CPU is going to be reset

2017-03-03 Thread Wanpeng Li
Reported by syzkaller: WARNING: CPU: 1 PID: 27742 at arch/x86/kvm/vmx.c:11029 nested_vmx_vmexit+0x5c35/0x74d0 arch/x86/kvm/vmx.c:11029 CPU: 1 PID: 27742 Comm: a.out Not tainted 4.10.0+ #229 Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS Bochs 01/01/201

Re: [PATCH 1/2] xfs: allow kmem_zalloc_greedy to fail

2017-03-03 Thread Dave Chinner
On Fri, Mar 03, 2017 at 03:19:12PM -0800, Darrick J. Wong wrote: > On Sat, Mar 04, 2017 at 09:54:44AM +1100, Dave Chinner wrote: > > On Thu, Mar 02, 2017 at 04:45:40PM +0100, Michal Hocko wrote: > > > From: Michal Hocko > > > > > > Even though kmem_zalloc_greedy is documented it might fail the cu

Re: [PATCH v4 2/3] perf: kretprobes: offset from reloc_sym if kernel supports it

2017-03-03 Thread Masami Hiramatsu
On Sat, 4 Mar 2017 11:35:51 +0900 Masami Hiramatsu wrote: > On Sat, 4 Mar 2017 09:49:11 +0900 > Masami Hiramatsu wrote: > > > On Thu, 2 Mar 2017 23:25:06 +0530 > > "Naveen N. Rao" wrote: > > > > > We indicate support for accepting sym+offset with kretprobes through a > > > line in ftrace REA

Re: [v2] mmc: sdhci-cadence: add HS400 enhanced strobe support

2017-03-03 Thread Masahiro Yamada
Hi Piotr, 2017-03-03 17:31 GMT+09:00 Piotr Sroka : > Add support for HS400ES mode to Cadence SDHCI driver. > > Signed-off-by: Piotr Sroka Thanks for your patch. It looks almost good to me. My comments are about some coding style issues only. Please see below. > +static void sdhci_cdns_get

Re: [PATCH v10 1/2] usb: dwc2: assert phy reset when waking up in rk3288 platform

2017-03-03 Thread ayaka
What is wrong with this patch? I have not seen it is merged in next-20170303. On 12/11/2016 11:36 PM, Randy Li wrote: On the rk3288 USB host-only port (the one that's not the OTG-enabled port) the PHY can get into a bad state when a wakeup is asserted (not just a wakeup from full s

[git pull] vfs.git final bits and pieces

2017-03-03 Thread Al Viro
A few unrelated patches that got beating in -next. Everything else will have to go into the next window ;-/ The following changes since commit 7ce7d89f48834cefece7804d38fc5d85382edf77: Linux 4.10-rc1 (2016-12-25 16:13:08 -0800) are available in the git repository at: git://git.kern

[PATCH] mm: do not call mem_cgroup_free() from within mem_cgroup_alloc()

2017-03-03 Thread Tahsin Erdogan
mem_cgroup_free() indirectly calls wb_domain_exit() which is not prepared to deal with a struct wb_domain object that hasn't executed wb_domain_init(). For instance, the following warning message is printed by lockdep if alloc_percpu() fails in mem_cgroup_alloc(): INFO: trying to register non-st

Re: [PATCH v2 1/1] serial: 8250_dw: Allow hardware flow control to be used

2017-03-03 Thread Andy Shevchenko
On Sat, Mar 4, 2017 at 1:23 AM, James Hogan wrote: > On Fri, Mar 03, 2017 at 03:31:06PM +0200, Andy Shevchenko wrote: >> On Fri, 2017-03-03 at 00:21 +, James Hogan wrote: >> > The CONFIG_HAVE_CLK=n implementation of devm_clk_get() in particular >> > seems highly questionable to me, given that

Re: [PATCH v4 2/3] perf: kretprobes: offset from reloc_sym if kernel supports it

2017-03-03 Thread Masami Hiramatsu
On Sat, 4 Mar 2017 09:49:11 +0900 Masami Hiramatsu wrote: > On Thu, 2 Mar 2017 23:25:06 +0530 > "Naveen N. Rao" wrote: > > > We indicate support for accepting sym+offset with kretprobes through a > > line in ftrace README. Parse the same to identify support and choose the > > appropriate forma

Re: [PATCH v4 2/3] perf: kretprobes: offset from reloc_sym if kernel supports it

2017-03-03 Thread Masami Hiramatsu
On Sat, 4 Mar 2017 11:35:51 +0900 Masami Hiramatsu wrote: > On Sat, 4 Mar 2017 09:49:11 +0900 > Masami Hiramatsu wrote: > > > On Thu, 2 Mar 2017 23:25:06 +0530 > > "Naveen N. Rao" wrote: > > > > > We indicate support for accepting sym+offset with kretprobes through a > > > line in ftrace REA

[PATCH 1/3] USB: misc: ldusb: fixed decimal permission coding issue

2017-03-03 Thread Milian Reichardt
Fixed ERROR: Use 4 digit octal (0777) not decimal permissions to fulfill the current coding-style. Signed-of-by: Milian Reichardt --- drivers/usb/misc/ldusb.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/usb/misc/ldusb.c b/drivers/usb/misc/ldusb.c index 3bc

[PATCH 3/3] USB: misc: ldusb: Added Space after ',' to fit the coding style

2017-03-03 Thread Milian Reichardt
Added a Space after ',' to get rid of an error message in checkpatch.pl and improve readability Signed-of-by: Milian Reichardt --- drivers/usb/misc/ldusb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/misc/ldusb.c b/drivers/usb/misc/ldusb.c index 2c68e5153539..

[PATCH 0/3] fixed errors from checkpatch.pl

2017-03-03 Thread Milian Reichardt
Changed multiple errors output from checkpatch.pl to enhance readability Milian Reichardt (3): USB: misc: ldusb: fixed decimal permission coding issue USB: misc: ldusb: changed '*' location to fit coding Style USB: misc: ldusb: Added Space after ',' to fit the coding style drivers/usb/misc

[PATCH 2/3] USB: misc: ldusb: changed '*' location to fit coding Style

2017-03-03 Thread Milian Reichardt
Changed the location of '*' to fit the current coding style and easy readability. Signed-of-by: Milian Reichardt --- drivers/usb/misc/ldusb.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/drivers/usb/misc/ldusb.c b/drivers/usb/misc/ldusb.c index e965e

Re: [PATCH] staging, android: remove lowmemory killer from the tree

2017-03-03 Thread Tim Murray
Hi all, I mentioned before that I had some ideas to overhaul lowmemorykiller, which would have the side effect of getting it out of the kernel. I've been working through some prototypes over the past few weeks (actually started before Michal sent his patch out), and I'd appreciate some feedback on

Re: [PATCH v2 0/4] fujitsu_init() cleanup

2017-03-03 Thread Jonathan Woithe
Hi Michael On Wed, Mar 01, 2017 at 09:10:40AM +0100, Micha?? K??pie?? wrote: > These patches should make fujitsu_init() a bit more palatable. No > changes are made to platform device code yet, for clarity these will be > posted in a separate series after this one gets applied. I have a prelimina

Re: [RFC PATCH v1 11/30] fs: new API for handling i_version

2017-03-03 Thread Jeff Layton
On Sat, 2017-03-04 at 10:55 +1100, NeilBrown wrote: > On Wed, Dec 21 2016, Jeff Layton wrote: > > > We already have inode_inc_iversion. Add inode_set_iversion, > > inode_get_iversion, inode_cmp_iversion and inode_iversion_need_inc. > > This list of added interfaces is incomplete. > And some of th

[PATCH] Staging: media: platform: bcm2835 - Style fix

2017-03-03 Thread Derek Robson
Fixed style of block comments across whole driver Found using checkpatch Signed-off-by: Derek Robson --- .../media/platform/bcm2835/bcm2835-camera.c| 24 ++ drivers/staging/media/platform/bcm2835/controls.c | 22 +++- .../staging/media/platform/bcm283

[PATCH v3] blkcg: allocate struct blkcg_gq outside request queue spinlock

2017-03-03 Thread Tahsin Erdogan
blkg_conf_prep() currently calls blkg_lookup_create() while holding request queue spinlock. This means allocating memory for struct blkcg_gq has to be made non-blocking. This causes occasional -ENOMEM failures in call paths like below: pcpu_alloc+0x68f/0x710 __alloc_percpu_gfp+0xd/0x10 __per

Re: [PATCH v3 0/3] Locality support for tpm_crb

2017-03-03 Thread Jerry Snitselaar
Jarkko Sakkinen @ 2017-02-08 11:11 GMT: > The tpm_crb driver should follow the policy of reserving and > relinquishing the locality it uses when multiple localities are used, > like when TXT is another locality. > > [This is a resend of v3 as this has been asked from me for a few times > now and

Re: [PATCH v3 0/3] Add support for MyGica T230C DVB-T2 stick

2017-03-03 Thread Antti Palosaari
On 03/03/2017 08:35 PM, Brüns, Stefan wrote: On Fr, 2017-02-17 at 01:55 +0100, Stefan Brüns wrote: The required command sequence for the new tuner (Si2141) was traced from the current Windows driver and verified with a small python script/libusb. The changes to the Si2168 and dvbsky driver are m

Re: [PATCH] uapi: fix asm/signal.h userspace compilation errors

2017-03-03 Thread Carlos O'Donell
On Thu, Mar 2, 2017 at 10:48 AM, Dmitry V. Levin wrote: > On Thu, Mar 02, 2017 at 10:22:18AM -0500, Carlos O'Donell wrote: >> On Wed, Mar 1, 2017 at 11:20 AM, Arnd Bergmann wrote: >> > On Sun, Feb 26, 2017 at 2:01 AM, Dmitry V. Levin wrote: >> >> Include (guarded by #ifndef __KERNEL__) to fix a

Re: [PATCH] staging: ks7010: clean up code

2017-03-03 Thread Joe Perches
On Fri, 2017-03-03 at 22:58 +0200, Ernestas Kulik wrote: > This fixes type warnings generated by sparse, replaces instances of > ntohs() with be16_to_cpu() and removes unused fields in structs. There's no real need to convert ntohs to be16_to_cpu > diff --git a/drivers/staging/ks7010/ks7010_sdio.

[GIT PULL] final round of SCSI updates for the 4.10+ merge window

2017-03-03 Thread James Bottomley
This is the set of stuff that didn't quite make the initial pull and a set of fixes for stuff which did. The new stuff is basically lpfc (nvme), qedi and aacraid. The fixes cover a lot of previously submitted stuff, the most important of which probably covers some of the failing irq vectors alloc

Re: [RFC PATCH v1 00/30] fs: inode->i_version rework and optimization

2017-03-03 Thread Jeff Layton
On Fri, 2017-03-03 at 18:00 -0500, J. Bruce Fields wrote: > On Wed, Dec 21, 2016 at 12:03:17PM -0500, Jeff Layton wrote: > > tl;dr: I think we can greatly reduce the cost of the inode->i_version > > counter, by exploiting the fact that we don't need to increment it > > if no one is looking at it. W

Re: [PATCH v4 2/3] perf: kretprobes: offset from reloc_sym if kernel supports it

2017-03-03 Thread Masami Hiramatsu
On Thu, 2 Mar 2017 23:25:06 +0530 "Naveen N. Rao" wrote: > We indicate support for accepting sym+offset with kretprobes through a > line in ftrace README. Parse the same to identify support and choose the > appropriate format for kprobe_events. Could you give us an example of this change here?

Re: [RFC PATCH v1 29/30] fs: track whether the i_version has been queried with an i_state flag

2017-03-03 Thread Jeff Layton
On Sat, 2017-03-04 at 11:03 +1100, NeilBrown wrote: > On Wed, Dec 21 2016, Jeff Layton wrote: > > > @@ -2072,7 +2093,12 @@ inode_cmp_iversion(const struct inode *inode, const > > u64 old) > > static inline bool > > inode_iversion_need_inc(struct inode *inode) > > { > > - return true; > > +

Re: [PATCH v3 20/25] ARM: owl: Implement CPU enable-method for S500

2017-03-03 Thread Andreas Färber
Am 01.03.2017 um 11:40 schrieb Andreas Färber: > Am 01.03.2017 um 08:19 schrieb kbuild test robot: >> Hi Andreas, >> >> [auto build test ERROR on next-20170228] >> [also build test ERROR on v4.10] >> [cannot apply to robh/for-next linus/master linux/master v4.9-rc8 v4.9-rc7 >> v4.9-rc6] >> [if you

Re: [PATCH v4 3/3] perf: powerpc: choose local entry point with kretprobes

2017-03-03 Thread Masami Hiramatsu
On Thu, 2 Mar 2017 23:25:07 +0530 "Naveen N. Rao" wrote: > perf now uses an offset from _text/_stext for kretprobes if the kernel > supports it, rather than the actual function name. As such, let's choose > the LEP for powerpc ABIv2 so as to ensure the probe gets hit. Do it only > if the kernel

[PATCHv5] arm: ftrace: Adds support for CONFIG_DYNAMIC_FTRACE_WITH_REGS

2017-03-03 Thread Abel Vesa
The DYNAMIC_FTRACE_WITH_REGS configuration makes it possible for a ftrace operation to specify if registers need to saved/restored by the ftrace handler. This is needed by kgraft and possibly other ftrace-based tools, and the ARM architecture is currently lacking this feature. It would also be the

Re: [PATCH v4 14/36] [media] v4l2-mc: add a function to inherit controls from a pipeline

2017-03-03 Thread Steve Longerbeam
On 03/03/2017 03:06 PM, Russell King - ARM Linux wrote: On Thu, Mar 02, 2017 at 06:02:57PM +0200, Sakari Ailus wrote: Hi Steve, On Wed, Feb 15, 2017 at 06:19:16PM -0800, Steve Longerbeam wrote: v4l2_pipeline_inherit_controls() will add the v4l2 controls from all subdev entities in a pipeline

Re: [PATCH 0/3] ipc subsystem refcounter conversions

2017-03-03 Thread Andrew Morton
On Mon, 20 Feb 2017 13:29:46 +0200 Elena Reshetova wrote: > Now when new refcount_t type and API are finally merged > (see include/linux/refcount.h), the following > patches convert various refcounters in the ipc susystem from atomic_t > to refcount_t. By doing this we prevent intentional or acc

Re: [RFC PATCH v1 11/30] fs: new API for handling i_version

2017-03-03 Thread Jeff Layton
On Fri, 2017-03-03 at 17:36 -0500, J. Bruce Fields wrote: > The patch ordering is a little annoying as I'd like to be able to be > able to verify the implementation at the same time these new interfaces > are added, but, I don't know, I don't have a better idea. > Fair point. My thinking was "add

Re: [RFC PATCH v1 29/30] fs: track whether the i_version has been queried with an i_state flag

2017-03-03 Thread NeilBrown
On Wed, Dec 21 2016, Jeff Layton wrote: > @@ -2072,7 +2093,12 @@ inode_cmp_iversion(const struct inode *inode, const > u64 old) > static inline bool > inode_iversion_need_inc(struct inode *inode) > { > - return true; > + bool ret; > + > + spin_lock(&inode->i_lock); > + ret = in

Re: Hundreds of null PATH records for *init_module syscall audit logs

2017-03-03 Thread Paul Moore
On Fri, Mar 3, 2017 at 4:14 PM, Richard Guy Briggs wrote: > On 2017-02-28 23:15, Steve Grubb wrote: >> On Tuesday, February 28, 2017 10:37:04 PM EST Richard Guy Briggs wrote: >> > Sorry, I forgot to include Cc: in this cover letter for context to the 4 >> > alt patches. >> > >> > On 2017-02-28 22:

Re: Hundreds of null PATH records for *init_module syscall audit logs

2017-03-03 Thread Paul Moore
On Tue, Feb 28, 2017 at 10:37 PM, Richard Guy Briggs wrote: > Sorry, I forgot to include Cc: in this cover letter for context to the 4 > alt patches. > > On 2017-02-28 22:15, Richard Guy Briggs wrote: >> The background to this is: >> https://github.com/linux-audit/audit-kernel/issues/8 >> >>

Re: [PATCH 4/5] clk: sunxi-ng: Add driver for A83T CCU

2017-03-03 Thread Stephen Boyd
On 03/03, Maxime Ripard wrote: > On Wed, Mar 01, 2017 at 11:17:05AM -0800, Stephen Boyd wrote: > > > Can someone explain what the issue is? Could something like > > clk_get_phase() + clk_get_rate() tell us if we're in one mode > > vs. the other? > > So we have two modes of operation for that cloc

Re: [PATCH 3/3] cpufreq: schedutil: remove redundant code from sugov_next_freq_shared()

2017-03-03 Thread Rafael J. Wysocki
On Saturday, March 04, 2017 01:03:17 AM Rafael J. Wysocki wrote: > On Thursday, March 02, 2017 02:03:22 PM Viresh Kumar wrote: > > The same code is present both within and outside the loop and it doesn't > > look like it provides any additional benefit. > > Well, not quite. This is on purpose. >

Re: [PATCH v4 1/3] perf: probe: factor out the ftrace README scanning

2017-03-03 Thread Masami Hiramatsu
On Thu, 2 Mar 2017 23:25:05 +0530 "Naveen N. Rao" wrote: > Simplify and separate out the ftrace README scanning logic into a > separate helper. This is used subsequently to scan for all patterns of > interest and to cache the result. > > Since we are only interested in availability of probe arg

Re: [PATCH 3/5] staging: lustre: lustre: Remove unnecessary cast on void pointer

2017-03-03 Thread kbuild test robot
Hi simran, [auto build test WARNING on staging/staging-testing] [also build test WARNING on v4.10 next-20170303] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/simran-singhal/staging-nvec

Re: [PATCH 3/3] cpufreq: schedutil: remove redundant code from sugov_next_freq_shared()

2017-03-03 Thread Rafael J. Wysocki
On Thursday, March 02, 2017 02:03:22 PM Viresh Kumar wrote: > The same code is present both within and outside the loop and it doesn't > look like it provides any additional benefit. Well, not quite. This is on purpose. Note the "if (j == smp_processor_id())" condition within the loop and think

Re: [PATCH v2 1/1] serial: 8250_dw: Allow hardware flow control to be used

2017-03-03 Thread James Hogan
Hi Jason, On Fri, Mar 03, 2017 at 04:02:56PM -0800, Jason Uy wrote: > HI James, > > Maybe instead of that, we should do this instead > > If ((IS_ERR(d->clk) && PTR_ERR(d->clk)) || !old) IS_ERR(x) matches a range of non-zero negative values, so it implies PTR_ERR(x) already, so it doesn't change

Re: [PATCH V5 6/6] proc: show MADV_FREE pages info in smaps

2017-03-03 Thread Andrew Morton
On Thu, 2 Mar 2017 17:30:54 +0100 Michal Hocko wrote: > > It's not that I think you're wrong: it *is* an implementation detail. > > But we take a bit of incoherency from batching all over the place, so > > it's a little odd to take a stand over this particular instance of it > > - whether demandi

[PATCH net] rxrpc: Call state should be read with READ_ONCE() under some circumstances

2017-03-03 Thread David Howells
The call state may be changed at any time by the data-ready routine in response to received packets, so if the call state is to be read and acted upon several times in a function, READ_ONCE() must be used unless the call state lock is held. Signed-off-by: David Howells --- net/rxrpc/input.c |

RE: [PATCH v2 1/1] serial: 8250_dw: Allow hardware flow control to be used

2017-03-03 Thread Jason Uy
HI James, Maybe instead of that, we should do this instead If ((IS_ERR(d->clk) && PTR_ERR(d->clk)) || !old) Note that this is what is done in the probe function of the dw driver. Regards, Jason -Original Message- From: James Hogan [mailto:james.ho...@imgtec.com] Sent: March-03-17 3:07

Re: [PATCH 0/2] fix the traced mt-exec deadlock

2017-03-03 Thread Eric W. Biederman
ebied...@xmission.com (Eric W. Biederman) writes: > The big lesson for me, and what was not obvious from your change > description is that we are changing the user space visible semantics > of exec+ptrace and that cred_guard_mutex is not at all the problem (as > we always take cred_guard_mutex in

Re: [RFC PATCH v1 30/30] fs: convert i_version counter over to an atomic64_t

2017-03-03 Thread NeilBrown
On Wed, Dec 21 2016, Jeff Layton wrote: > > +/* > + * We borrow the top bit in the i_version to use as a flag to tell us whether > + * it has been queried since we last bumped it. If it has, then we must bump > + * it and set the flag. Note that this means that we have to handle wrapping > + * m

Re: [PATCH v3 4/5] clk: hi6220: add debug APB clock

2017-03-03 Thread Stephen Boyd
On 03/03, Leo Yan wrote: > The debug APB clock is absent in hi6220 driver, so this patch is to add > support for it. > > Signed-off-by: Leo Yan > --- Acked-by: Stephen Boyd -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project

Re: [RFC PATCH v1 11/30] fs: new API for handling i_version

2017-03-03 Thread NeilBrown
On Wed, Dec 21 2016, Jeff Layton wrote: > We already have inode_inc_iversion. Add inode_set_iversion, > inode_get_iversion, inode_cmp_iversion and inode_iversion_need_inc. This list of added interfaces is incomplete. And some of these interfaces could really use some justification up front. You

Re: [PATCH v2] clk/axs10x: introduce AXS10X pll driver

2017-03-03 Thread Stephen Boyd
On 03/03, Vlad Zakharov wrote: > Hi Michael, Stephen, > > On Tue, 2017-02-21 at 16:11 +0300, Vlad Zakharov wrote: > > AXS10X boards manages it's clocks using various PLLs. These PLL has same > > dividers and corresponding control registers mapped to different addresses. > > So we add one common dr

Re: [PATCH] Documentation/sphinx: fix primary_domain configuration

2017-03-03 Thread Jonathan Corbet
On Fri, 3 Mar 2017 12:24:05 + John Keeping wrote: > With Sphinx 1.5.3 I get the warning: > > WARNING: primary_domain 'C' not found, ignored. > > It seems that domain names in Sphinx are case-sensitive and for the C > domain the name must be lower case. Applied, thanks. jon

Re: [PATCH v2 09/17] Documentation: kernel-parameters: Document owl earlycon

2017-03-03 Thread Andreas Färber
Am 03.03.2017 um 23:46 schrieb Jonathan Corbet: > On Fri, 24 Feb 2017 04:40:47 +0100 > Andreas Färber wrote: > >> +owl, >> +Start an early, polled-mode console on an Owl serial >> +port at the specified address. The serial port must >> +

Re: subdevice config into pointer (was Re: [PATCH 1/4] v4l2: device_register_subdev_nodes: allow calling multiple times)

2017-03-03 Thread Pavel Machek
Hi! > > > > > Making the sub-device bus configuration a pointer should be in a > > > > > separate > > > > > patch. It makes sense since the entire configuration is not valid for > > > > > all > > > > > sub-devices attached to the ISP anymore. I think it originally was a > > > > > separate patch,

[Update][PATCH v2 3/3] cpufreq: intel_pstate: Avoid triggering cpu_frequency tracepoint unnecessarily

2017-03-03 Thread Rafael J. Wysocki
From: Rafael J. Wysocki In the passive mode the cpu_frequency trace event is already triggered by the cpufreq core or by scaling governors, so intel_pstate should not trigger it once again for the same P-state updates. In addition to that, the frequency returned by intel_cpufreq_fast_switch() an

Re: [PATCH v2 1/1] serial: 8250_dw: Allow hardware flow control to be used

2017-03-03 Thread James Hogan
On Fri, Mar 03, 2017 at 03:31:06PM +0200, Andy Shevchenko wrote: > On Fri, 2017-03-03 at 00:21 +, James Hogan wrote: > > The CONFIG_HAVE_CLK=n implementation of devm_clk_get() in particular > > seems highly questionable to me, given that commit 93abe8e4b13a ("clk: > > add non CONFIG_HAVE_CLK ro

Re: [PATCH 1/2] xfs: allow kmem_zalloc_greedy to fail

2017-03-03 Thread Darrick J. Wong
On Sat, Mar 04, 2017 at 09:54:44AM +1100, Dave Chinner wrote: > On Thu, Mar 02, 2017 at 04:45:40PM +0100, Michal Hocko wrote: > > From: Michal Hocko > > > > Even though kmem_zalloc_greedy is documented it might fail the current > > code doesn't really implement this properly and loops on the smal

Re: [PATCH 01/10] perf, tools, stat: Factor out callback for collecting event values

2017-03-03 Thread Andi Kleen
On Fri, Mar 03, 2017 at 11:33:03AM +0100, Jiri Olsa wrote: > On Tue, Feb 28, 2017 at 10:49:15PM -0800, Andi Kleen wrote: > > SNIP > > > +static void collect_data(struct perf_evsel *counter, > > + void (*cb)(struct perf_evsel *counter, void *data, > > +

Re: [PATCH 1/2] mm: use is_migrate_highatomic() to simplify the code

2017-03-03 Thread Andrew Morton
On Fri, 3 Mar 2017 14:18:08 +0100 Michal Hocko wrote: > On Fri 03-03-17 19:10:13, Xishi Qiu wrote: > > Introduce two helpers, is_migrate_highatomic() and > > is_migrate_highatomic_page(). > > Simplify the code, no functional changes. > > static inline helpers would be nicer than macros Always.

Re: [RFC PATCH v1 00/30] fs: inode->i_version rework and optimization

2017-03-03 Thread J. Bruce Fields
On Wed, Dec 21, 2016 at 12:03:17PM -0500, Jeff Layton wrote: > tl;dr: I think we can greatly reduce the cost of the inode->i_version > counter, by exploiting the fact that we don't need to increment it > if no one is looking at it. We can also clean up the code to prepare > to eventually expose thi

Re: [PATCH v2 1/1] serial: 8250_dw: Allow hardware flow control to be used

2017-03-03 Thread James Hogan
Hi Jason, On Fri, Mar 03, 2017 at 09:43:55AM -0800, Jason Uy wrote: > James, > > Can you verify that changing the code to the following fixes your problem? > > if (IS_ERR_OR_NULL(d->clk) || !old) > goto out; It does, however I'm not at all convinced it is correct. clk_get either returns a v

Re: [PATCH v4 14/36] [media] v4l2-mc: add a function to inherit controls from a pipeline

2017-03-03 Thread Russell King - ARM Linux
On Thu, Mar 02, 2017 at 06:02:57PM +0200, Sakari Ailus wrote: > Hi Steve, > > On Wed, Feb 15, 2017 at 06:19:16PM -0800, Steve Longerbeam wrote: > > v4l2_pipeline_inherit_controls() will add the v4l2 controls from > > all subdev entities in a pipeline to a given video device. > > > > Signed-off-by

[PATCH] hwmon: (dell-smm) Add Dell XPS 15 9560 into DMI list

2017-03-03 Thread Pali Rohár
It was reported that dell-smm-hwmon is working fine on Dell XPS 15 9560. Link: http://www.spinics.net/lists/platform-driver-x86/msg10751.html Reported-by: Vasile Dumitrescu Signed-off-by: Pali Rohár --- drivers/hwmon/dell-smm-hwmon.c |7 +++ 1 file changed, 7 insertions(+) diff --git a

Re: [PATCH] doc/ko_KR/memory-barriers: Update control-dependencies section

2017-03-03 Thread Jonathan Corbet
On Fri, 3 Mar 2017 15:44:02 +0900 SeongJae Park wrote: > This commit applies upstream change, commit c8241f8553e8 ("doc: Update > control-dependencies section of memory-barriers.txt"), to Korean > translation. Applied to the docs tree, thanks. jon

Re: [PATCH 1/2] xfs: allow kmem_zalloc_greedy to fail

2017-03-03 Thread Dave Chinner
On Thu, Mar 02, 2017 at 04:45:40PM +0100, Michal Hocko wrote: > From: Michal Hocko > > Even though kmem_zalloc_greedy is documented it might fail the current > code doesn't really implement this properly and loops on the smallest > allowed size for ever. This is a problem because vzalloc might fa

[patch v2] mm, vmstat: print non-populated zones in zoneinfo

2017-03-03 Thread David Rientjes
Initscripts can use the information (protection levels) from /proc/zoneinfo to configure vm.lowmem_reserve_ratio at boot. vm.lowmem_reserve_ratio is an array of ratios for each configured zone on the system. If a zone is not populated on an arch, /proc/zoneinfo suppresses its output. This result

Re: [PATCH] mm, swap: Fix a race in free_swap_and_cache()

2017-03-03 Thread Andrew Morton
On Wed, 1 Mar 2017 22:38:09 +0800 "Huang, Ying" wrote: > Before using cluster lock in free_swap_and_cache(), the > swap_info_struct->lock will be held during freeing the swap entry and > acquiring page lock, so the page swap count will not change when > testing page information later. But after

Re: [PATCH v4 13/36] [media] v4l2: add a frame timeout event

2017-03-03 Thread Steve Longerbeam
On 03/03/2017 03:45 AM, Sakari Ailus wrote: On Thu, Mar 02, 2017 at 03:07:21PM -0800, Steve Longerbeam wrote: On 03/02/2017 07:53 AM, Sakari Ailus wrote: Hi Steve, On Wed, Feb 15, 2017 at 06:19:15PM -0800, Steve Longerbeam wrote: Add a new FRAME_TIMEOUT event to signal that a video captur

Re: [PATCH v4 14/36] [media] v4l2-mc: add a function to inherit controls from a pipeline

2017-03-03 Thread Steve Longerbeam
On 03/03/2017 11:17 AM, Sakari Ailus wrote: Hi Steve, On Thu, Mar 02, 2017 at 06:12:43PM -0800, Steve Longerbeam wrote: On 03/02/2017 03:48 PM, Steve Longerbeam wrote: On 03/02/2017 08:02 AM, Sakari Ailus wrote: Hi Steve, On Wed, Feb 15, 2017 at 06:19:16PM -0800, Steve Longerbeam wrote

Re: [PATCH] Documentation: Update path to sysrq.txt

2017-03-03 Thread Jonathan Corbet
On Fri, 24 Feb 2017 10:42:14 +0200 Krzysztof Kozlowski wrote: > Commit 9d85025b0418 ("docs-rst: create an user's manual book") moved the > sysrq.txt leaving old paths in the kernel docs. Applied to the docs tree, thanks. jon

Re: [PATCH] PM / OPP: Documentation: Fix opp-microvolt in examples

2017-03-03 Thread Rafael J. Wysocki
On Friday, March 03, 2017 09:28:05 AM Stephen Boyd wrote: > On 03/01/2017 03:09 AM, Viresh Kumar wrote: > > The triplet present in "opp-microvolt" property should be in the order > > , while all the examples have it in the order > > . > > > > Fix it. > > > > Luckily all of the users of "opp-microvo

[PATCH] net: smsc: smc91c92_cs: use new api ethtool_{get|set}_link_ksettings

2017-03-03 Thread Philippe Reynes
The ethtool api {get|set}_settings is deprecated. We move this driver to new api {get|set}_link_ksettings. As I don't have the hardware, I'd be very pleased if someone may test this patch. Signed-off-by: Philippe Reynes --- drivers/net/ethernet/smsc/smc91c92_cs.c | 98

Re: [PATCH v2 09/17] Documentation: kernel-parameters: Document owl earlycon

2017-03-03 Thread Jonathan Corbet
On Fri, 24 Feb 2017 04:40:47 +0100 Andreas Färber wrote: > + owl, > + Start an early, polled-mode console on an Owl serial > + port at the specified address. The serial port must > + already be setup and configured. Options a

Re: [patch] mm, zoneinfo: print non-populated zones

2017-03-03 Thread David Rientjes
On Fri, 3 Mar 2017, Anshuman Khandual wrote: > > This patch shows statistics for non-populated zones in /proc/zoneinfo. > > The zones exist and hold a spot in the vm.lowmem_reserve_ratio array. > > Without this patch, it is not possible to determine which index in the > > array controls which zone

Re: [PATCH 4/4] thp: fix MADV_DONTNEED vs clear soft dirty race

2017-03-03 Thread Andrew Morton
On Thu, 2 Mar 2017 18:10:34 +0300 "Kirill A. Shutemov" wrote: > Yet another instance of the same race. > > Fix is identical to change_huge_pmd(). Nit: someone who is reading this changelog a year from now will be quite confused - how do they work out what the race was? I'll add : See "thp:

Re: [PATCH 3/5] staging: lustre: lustre: Remove unnecessary cast on void pointer

2017-03-03 Thread kbuild test robot
Hi simran, [auto build test WARNING on staging/staging-testing] [also build test WARNING on v4.10 next-20170303] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/simran-singhal/staging-nvec

[PATCH] sparc64: Fix size check in huge_pte_alloc

2017-03-03 Thread Nitin Gupta
Signed-off-by: Nitin Gupta --- arch/sparc/mm/hugetlbpage.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/sparc/mm/hugetlbpage.c b/arch/sparc/mm/hugetlbpage.c index 323bc6b..3016850 100644 --- a/arch/sparc/mm/hugetlbpage.c +++ b/arch/sparc/mm/hugetlbpage.c @@ -261,7 +261

Re: [RFC PATCH v1 11/30] fs: new API for handling i_version

2017-03-03 Thread J. Bruce Fields
The patch ordering is a little annoying as I'd like to be able to be able to verify the implementation at the same time these new interfaces are added, but, I don't know, I don't have a better idea. Anyway, various nits: On Wed, Dec 21, 2016 at 12:03:28PM -0500, Jeff Layton wrote: > We already ha

  1   2   3   4   5   6   7   8   >