Re: possible circular locking dependency detected [was: linux-next: Tree for Aug 22]

2017-08-22 Thread Byungchul Park
On Tue, Aug 22, 2017 at 09:43:56PM +, Bart Van Assche wrote: > On Tue, 2017-08-22 at 19:47 +0900, Sergey Senozhatsky wrote: > > == > > WARNING: possible circular locking dependency detected > > 4.13.0-rc6-next-20170822-db

[PATCH v2 0/2] enable hires timer to timeout datagram socket

2017-08-22 Thread Vallish Vaidyeshwara
v1->v2: - Cong Wang pointed out MAX_SCHEDULE_TIMEOUT wraparound, fixed the patch 1/2 to accommodate MAX_SCHEDULE_TIMEOUT wait time - Changed format of printing total time from float to long in selftests patch 2/2 Hello Dave, Resending the patch series to include netdev mailing list with

Re: [PATCH v6 1/2] dt-bindings: misc: achc: Add device tree binding for GE ACHC

2017-08-22 Thread Rob Herring
On Fri, Aug 18, 2017 at 04:53:47PM +0100, Martyn Welch wrote: > Add Device Tree binding document for GE Healthcare USB Management > Controller (ACHC). > > Signed-off-by: Martyn Welch > --- > Documentation/devicetree/bindings/misc/ge-achc.txt | 26 >

Re: [PATCH v2 2/5] dt-bindings: add amc6821, isl1208 trivial bindings

2017-08-22 Thread Rob Herring
On Fri, Aug 18, 2017 at 09:34:42PM +0200, Klaus Goger wrote: > TI AMC6821 fan controller and Intersil ISL1208 are trivial > devices, so add them to the binding list. > > Signed-off-by: Klaus Goger > --- > > Changes in v2: None > >

Re: [PATCH 1/3] soc: qcom: smem: Support global partition

2017-08-22 Thread Bjorn Andersson
On Tue 22 Aug 17:28 PDT 2017, Chris Lew wrote: > On 8/21/2017 10:17 AM, Bjorn Andersson wrote: [..] > > > +static int qcom_smem_set_global_partition(struct qcom_smem *smem, > > > + struct smem_ptable_entry *entry) > > > +{ [..] > > > + if (header->size != entry->size) { > >

[PATCH] rtc: remove .open() and .release()

2017-08-22 Thread Alexandre Belloni
There are no driver left using .open and .release. There is no good use case for them as there is nothing the character device interface does that should not be done in the sysfs interface or in-kernel interface. Remove those callbacks now to avoid future confusion. Signed-off-by: Alexandre

Re: [PATCH 3.18 00/10] 3.18.67-stable review

2017-08-22 Thread Shuah Khan
On 08/22/2017 01:09 PM, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 3.18.67 release. > There are 10 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know. > > Responses

Re: [V2 1/4] MIPS: dts: ralink: Add Mediatek MT7628A SoC

2017-08-22 Thread Rob Herring
On Mon, Aug 21, 2017 at 11:55:29AM +0100, Harvey Hunt wrote: > The MT7628A is the successor to the MT7620 and pin compatible with the > MT7688A, although the latter supports only a 1T1R antenna rather than > a 2T2R antenna. > > This commit adds support for the following features: > > - UART > -

Re: [PATCH v2 1/9] devicetree: bindings: Add sata port phy config parameters in ahci-ceva

2017-08-22 Thread Rob Herring
On Mon, Aug 21, 2017 at 01:17:16PM +0200, Michal Simek wrote: > From: Anurag Kumar Vulisha > > This patch adds device tree bindings for sata port phy parameters > in the ahci-ceva.txt file. > > Signed-off-by: Anurag Kumar Vulisha >

Re: [PATCH 2/3] ARM: dts: sun8i: Add board dts file for Banana Pi M2 Ultra

2017-08-22 Thread Icenowy Zheng
于 2017年8月23日 GMT+08:00 上午4:10:43, Maxime Ripard 写到: >Hi, > >On Tue, Aug 22, 2017 at 02:17:41PM +0800, Icenowy Zheng wrote: >> From: Chen-Yu Tsai >> >> The Banana Pi M2 Ultra is an SBC based on the Allwinner R40 SoC. The >> form factor and

Re: [PATCH v2 3/3] cpufreq: dt: add support for rockchip rv1108

2017-08-22 Thread Rafael J. Wysocki
On Wed, Aug 23, 2017 at 2:49 AM, Andy Yan wrote: > Hi Rafael: > > > On 2017年08月22日 21:44, Rafael J. Wysocki wrote: >> >> On Tuesday, August 22, 2017 3:15:22 PM CEST Andy Yan wrote: >>> >>> Add the compatible string to support the generic device tree >>> cpufreq-dt driver

Re: Do we really need d_weak_revalidate???

2017-08-22 Thread NeilBrown
On Mon, Aug 21 2017, Ian Kent wrote: >> >> A mount isn't triggered by kern_path(pathname, 0, ). >> That '0' would need to include one of >> LOOKUP_PARENT | LOOKUP_DIRECTORY | >> LOOKUP_OPEN | LOOKUP_CREATE | LOOKUP_AUTOMOUNT >> >> to trigger an automount (otherwise you just get -EISDIR). >

Re: [RFC 2/3] rtc: Add Realtek RTD1295

2017-08-22 Thread Alexandre Belloni
On 20/08/2017 at 23:10:16 +0200, Andreas Färber wrote: > > Is RTD_RTCDATE_HIGH latched when RTD_RTCDATE_LOW is read? > > I do not have an answer to that. > > > If this is not > > the case, you probably want to handle a possible update between both > > readl_relaxed. > > Are you proposing to

Re: [PATCH 1/5] iommu/arm-smmu-v3: put off the execution of TLBI* to reduce lock confliction

2017-08-22 Thread Leizhen (ThunderTown)
On 2017/8/22 23:41, Joerg Roedel wrote: > On Mon, Jun 26, 2017 at 09:38:46PM +0800, Zhen Lei wrote: >> -static int queue_insert_raw(struct arm_smmu_queue *q, u64 *ent) >> +static int queue_insert_raw(struct arm_smmu_queue *q, u64 *ent, int >> optimize) >> { >> if (queue_full(q)) >>

[PATCH v3] irqchip: uniphier-aidet: add UniPhier AIDET irqchip driver

2017-08-22 Thread Masahiro Yamada
UniPhier SoCs contain AIDET (ARM Interrupt Detector). This is intended to provide additional features that are not covered by GIC. The main purpose is to provide logic inverter to support low level and falling edge trigger types for interrupt lines from on-board devices. Signed-off-by: Masahiro

Re: [PATCH] staging: vboxvideo: constify drm_fb_helper_funcs

2017-08-22 Thread Greg KH
On Mon, Aug 21, 2017 at 03:36:34PM +0530, Arvind Yadav wrote: > drm_fb_helper_funcs are not supposed to change at runtime. > All functions working with drm_fb_helper_funcs provided by > work with const drm_fb_helper_funcs. > So mark the non-const structs as const. > > Signed-off-by: Arvind Yadav

Re: [RFC PATCH v2 1/3] PCI: rockchip: Add support for pcie wake irq

2017-08-22 Thread jeffy
Hi Tony, On 08/23/2017 01:26 AM, Tony Lindgren wrote: OK, let's fix any wakeriq ordering issues to make it more usable. Sounds like in your case the wakeirq needs to be enabled late and disabled early, while in my test cases I can keep it enabled basically any time. yes, in my case it's a

Re: [PATCH 3/3] staging:rtl8188eu:core Fix coding style issues

2017-08-22 Thread Greg KH
On Mon, Aug 21, 2017 at 01:00:14PM +0530, Janani S wrote: > This patch solves the following warning shown by > checkpatch script > WARNING: Comparisons should place the constants > on the right side of the test Why the odd line-wrapping? And where is patch 1/3 and 2/3? thanks, greg k-h

Re: [PATCH] drivers: staging: android: Fix style/formatting issues

2017-08-22 Thread Greg KH
On Tue, Aug 22, 2017 at 09:04:50PM -0400, Nick Fox wrote: > Fix two issues reported by checkpatch. The first removes an extra > blank line, the second changes an argument to kmalloc(). Original > checkpatch output is below: That should be two different patches, please break this up. thanks,

Re: [PATCH] staging: fsl-mc: be consistent when checking strcmp() returns

2017-08-22 Thread Greg KH
On Thu, Aug 17, 2017 at 02:55:04PM +0300, laurentiu.tu...@nxp.com wrote: > From: Laurentiu Tudor > > Throughout the driver we use == 0 / != 0 to check strcmp() returns except > this place, so fix it. You will just get someone else fixing it as it is not what checkpatch

Re: [PATCH] staging: fsl-mc: move bus driver out of staging

2017-08-22 Thread Greg KH
On Sat, Aug 19, 2017 at 08:18:12PM +0300, laurentiu.tu...@nxp.com wrote: > From: Stuart Yoder > > Move the source files out of staging into their final locations: > -include files in drivers/staging/fsl-mc/include go to include/linux/fsl > -irq-gic-v3-its-fsl-mc-msi.c

Re: [PATCH] sg: recheck MMAP_IO request length with lock held

2017-08-22 Thread Martin K. Petersen
Todd, > Commit 1bc0eb044615 ("scsi: sg: protect accesses to 'reserved' page > array") adds needed concurrency protection for the "reserve" buffer. > Some checks that are initially made outside the lock are replicated once > the lock is taken to ensure the checks and resulting decisions are made

Re:Re: [PATCH] fix memory leak on kvm_vm_ioctl_create_spapr_tce

2017-08-22 Thread Nixiaoming
>On 22.08.2017 17:15, David Hildenbrand wrote: >> On 22.08.2017 16:28, nixiaoming wrote: >>> miss kfree(stt) when anon_inode_getfd return fail so add check >>> anon_inode_getfd return val, and kfree stt >>> >>> Signed-off-by: nixiaoming >>> --- >>>

Re: [PATCH] sg: protect against races between mmap() and SG_SET_RESERVED_SIZE

2017-08-22 Thread Douglas Gilbert
On 2017-08-16 01:41 AM, Todd Poynor wrote: Take f_mutex around mmap() processing to protect against races with the SG_SET_RESERVED_SIZE ioctl. Ensure the reserve buffer length remains consistent during the mapping operation, and set the "mmap called" flag to prevent further changes to the

Re: [RFC PATCH v2 1/3] PCI: rockchip: Add support for pcie wake irq

2017-08-22 Thread Brian Norris
Hi Jeffy, On Wed, Aug 23, 2017 at 09:32:39AM +0800, Jeffy Chen wrote: > and for eage irq, maybe we should enable it right after(or before) > the driver activate wake function(for example activate WOWLAN or > WOLAN), otherwise would it be possible to miss some irqs(triggered > before we actually

Re: [PATCH v3 1/3] lockdep: Make LOCKDEP_CROSSRELEASE configs all part of PROVE_LOCKING

2017-08-22 Thread Byungchul Park
On Tue, Aug 22, 2017 at 03:46:03PM +1000, Dave Chinner wrote: > On Mon, Aug 21, 2017 at 05:46:00PM +0200, Peter Zijlstra wrote: > > > > > > Booting the very latest -tip on my test machine gets me the below splat. > > > > Dave, TJ, FYI, lockdep grew annotations for completions; it remembers > >

Re: [PATCH v5 0/7] Add RAS virtualization support to SEA/SEI notification type

2017-08-22 Thread gengdongjiu
Jonathan, Thanks for the review, will correct the typo issue in the next patch version. On 2017/8/22 15:54, Jonathan Cameron wrote: > On Fri, 18 Aug 2017 22:11:50 +0800 > Dongjiu Geng wrote: > >> In the firmware-first RAS solution, corrupt data is detected in a >>

Re: possible circular locking dependency detected [was: linux-next: Tree for Aug 22]

2017-08-22 Thread Sergey Senozhatsky
On (08/23/17 09:03), Byungchul Park wrote: [..] aha, ok > The report is talking about the following lockup: > > A work in a worker A task work on exit to user > -- --- > mutex_lock(>bd_mutex) >

Re: Do we really need d_weak_revalidate???

2017-08-22 Thread Ian Kent
On 23/08/17 10:32, Ian Kent wrote: > On 23/08/17 09:06, NeilBrown wrote: >> On Mon, Aug 21 2017, Ian Kent wrote: >> A mount isn't triggered by kern_path(pathname, 0, ). That '0' would need to include one of LOOKUP_PARENT | LOOKUP_DIRECTORY | LOOKUP_OPEN | LOOKUP_CREATE

Re: [PATCH] binder: let ANDROID_BINDER_IPC_32BIT be selectable on 32bit ARM

2017-08-22 Thread Jisheng Zhang
On Tue, 22 Aug 2017 18:51:08 -0700 Greg KH wrote: > On Tue, Aug 08, 2017 at 07:03:05PM +0800, Jisheng Zhang wrote: > > As noted in commit d0bdff0db809 ("staging: Fix build issues with new > > binder API"), we can add back the choice for 32bit ARM "once a 64bit > > __get_user_asm_* implementation

Re: [PATCH net-next v7 02/10] bpf: Add eBPF program subtype and is_valid_subtype() verifier

2017-08-22 Thread Alexei Starovoitov
On Mon, Aug 21, 2017 at 02:09:25AM +0200, Mickaël Salaün wrote: > The goal of the program subtype is to be able to have different static > fine-grained verifications for a unique program type. > > The struct bpf_verifier_ops gets a new optional function: > is_valid_subtype(). This new verifier is

Re: [PATCH] binder: let ANDROID_BINDER_IPC_32BIT be selectable on 32bit ARM

2017-08-22 Thread John Stultz
On Tue, Aug 22, 2017 at 7:56 PM, John Stultz wrote: > On Tue, Aug 22, 2017 at 7:34 PM, Jisheng Zhang wrote: >> On Tue, 22 Aug 2017 18:51:08 -0700 Greg KH wrote: >> >>> On Tue, Aug 08, 2017 at 07:03:05PM +0800, Jisheng Zhang wrote: >>> > As noted in

[PATCH 2/2] staging: android: ion: Change argument to kmalloc (checkpatch)

2017-08-22 Thread Nick Fox
Change argument to kmalloc() to fix style issue, reported by checkpatch Signed-off-by: Nick Fox --- drivers/staging/android/ion/ion_cma_heap.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/android/ion/ion_cma_heap.c

Re: [PATCH v3 1/3] lockdep: Make LOCKDEP_CROSSRELEASE configs all part of PROVE_LOCKING

2017-08-22 Thread Peter Zijlstra
On Tue, Aug 22, 2017 at 12:08:40PM +0200, Peter Zijlstra wrote: > > > > I meant: > > > > > > > > mutex_lock() > > > > > > > > lockdep_map_acquire_read() > > > > mutex_lock() > > > > > > > >

Re: [PATCH v2 3/3] cpufreq: dt: add support for rockchip rv1108

2017-08-22 Thread Rafael J. Wysocki
On Tuesday, August 22, 2017 3:15:22 PM CEST Andy Yan wrote: > Add the compatible string to support the generic device tree > cpufreq-dt driver on rockchip rv1108 soc. > > Signed-off-by: Andy Yan This is a [3/3] in a series, so presumably it depends on something, so I

Re: [PATCH] staging/ccree: Declare compiled out fuctions static inline

2017-08-22 Thread Dan Carpenter
Try saving this email and then do `cat email.txt | git am`. It is corrupted. This is not how to send a v2 patch. Please read: https://kernelnewbies.org/FirstKernelPatch#head-5c81b3c517a1d0bbc24f92594cb734e155fcbbcb Look through the email archives for examples of other v2 patches. regards,

[PATCH] clk: mxs: make clk_ops const

2017-08-22 Thread Bhumika Goyal
Make these const as they are only stored in the const field of a clk_init_data structure. Signed-off-by: Bhumika Goyal --- drivers/clk/mxs/clk-div.c | 2 +- drivers/clk/mxs/clk-frac.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH] tegra114: fix to a race condition due to early registration of interrupt handler

2017-08-22 Thread Anton Volkov
The early registration of interrupt handler made possible a race condition leading to the usage of an incorrect physical address obtained by reading uninitialized tspi->tx_dma_phys. This patch moves the registration of an interrupt handler further down the code of tegra_spi_probe to make the race

[PATCH v2] ALSA: pcsp: Use common error handling code in snd_card_pcsp_probe()

2017-08-22 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 22 Aug 2017 15:50:53 +0200 Add a jump target so that a bit of exception handling can be better reused at the end of this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring

Re: [PATCH v5 2/9] ASoC: soc-core: Allow searching dai driver name in snd_soc_find_dai

2017-08-22 Thread Mark Brown
On Tue, Aug 22, 2017 at 09:24:32PM +0800, Donglin Peng wrote: > In case that dai->driver->name is NULL, strcmp(dai->driver->name, > dlc->dai_name) will cause segmentation fault. > so I think that we can change it as the follows: We should be already verifying that drivers have a name, we assume

[PATCH] clk: sirf: make clk_ops const

2017-08-22 Thread Bhumika Goyal
Make these const as they are only stored in the const field of a clk_init_data structure. Signed-off-by: Bhumika Goyal --- To compile-test clk-common.c, I compiled clk-atlas6.c and clk-prima2.c as it gets included in these two files. drivers/clk/sirf/clk-common.c | 12

RE: [PATCH] ACPI, APEI, EINJ: Subtract any matching Register Region from Trigger resources

2017-08-22 Thread Ghannam, Yazen
+EDAC, Boris and Tony for RAS comments. Any comments from ACPI folks? Thanks, Yazen > -Original Message- > From: Ghannam, Yazen > Sent: Thursday, August 10, 2017 1:58 PM > To: linux-a...@vger.kernel.org > Cc: linux-kernel@vger.kernel.org; r...@rjwysocki.net; l...@kernel.org; > Ghannam,

[PATCH] firmware: tegra: add BPMP debugfs support

2017-08-22 Thread Timo Alho
Tegra power management firmware running on co-processor (BPMP) implements a simple pseudo file system akin to debugfs. The file system can be used for debugging purposes to examine and change the status of selected resources controlled by the firmware (such as clocks, resets, voltages, powergates,

Re: [PATCH] ALSA: maestro3: Use common error handling code in two functions

2017-08-22 Thread Dan Carpenter
On Tue, Aug 22, 2017 at 03:43:03PM +0200, SF Markus Elfring wrote: > From: Markus Elfring > Date: Tue, 22 Aug 2017 15:00:27 +0200 > > Add jump targets so that a bit of exception handling can be better reused > at the end of these functions. > > This issue was

Re: [PATCH] mfd: Add support for TI LMP92001

2017-08-22 Thread Jonathan Cameron
On Tue, 22 Aug 2017 13:26:11 +0700 wrote: > From: Abhisit Sangjan > > TI LMP92001 Analog System Monitor and Controller > > 8-bit GPIOs. > 12 DACs with 12-bit resolution. > The GPIOs and DACs are shared port function with Cy function pin to > take

Re: [PATCH v4 0/5] Support for LEGO MINDSTORMS EV3 LCD display

2017-08-22 Thread Sekhar Nori
On Monday 07 August 2017 11:09 PM, David Lechner wrote: > ARM: dts: da850-lego-ev3: Add node for LCD display > ARM: davinci_all_defconfig: enable tinydrm and ST7586 Queuing these two patches (4/5 and 5/5) through davinci tree. Thanks, Sekhar

[PATCH] clk: versatile: make clk_ops const

2017-08-22 Thread Bhumika Goyal
Make this const as it is only stored in the const field of a clk_init_data structure. Signed-off-by: Bhumika Goyal --- drivers/clk/versatile/clk-vexpress-osc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/clk/versatile/clk-vexpress-osc.c

Re: [PATCH RFC v3 6/9] KVM: rework kvm_vcpu_on_spin loop

2017-08-22 Thread David Hildenbrand
On 21.08.2017 22:35, Radim Krčmář wrote: > The original code managed to obfuscate a straightforward idea: > start iterating from the selected index and reset the index to 0 when > reaching the end of online vcpus, then iterate until reaching the index > that we started at. > > The resulting code

Re: [alsa-devel] ALSA: pcsp: Use common error handling code in snd_card_pcsp_probe()

2017-08-22 Thread Takashi Iwai
On Tue, 22 Aug 2017 15:15:02 +0200, SF Markus Elfring wrote: > > >> Is it appropriate to treat non-zero values as error codes there generally? > > > > No, it can't be in general. > > I got the impression that the functions which are called at the updated places > by the function

Re: [PATCH 2/2 v7] printk: Add monotonic, boottime, and realtime timestamps

2017-08-22 Thread Prarit Bhargava
On 08/17/2017 11:30 AM, Mark Salyzyn wrote: > On 08/17/2017 06:15 AM, Prarit Bhargava wrote: >> printk.time=1/CONFIG_PRINTK_TIME=1 adds a unmodified local hardware clock >> timestamp to printk messages. The local hardware clock loses time each >> day making it difficult to determine exactly

Re: [PATCH 13/18] rpmsg: glink: Add rx done command

2017-08-22 Thread Sricharan R
Hi, >> +    /* Take it off the tree of receive intents */ >> +    if (!intent->reuse) { >> +    spin_lock(>intent_lock); >> +    idr_remove(>liids, intent->id); >> +    spin_unlock(>intent_lock); >> +    } >> + >> +    /* Schedule the sending of a rx_done indication */ >> +   

Re: [PATCH v5 2/9] ASoC: soc-core: Allow searching dai driver name in snd_soc_find_dai

2017-08-22 Thread Donglin Peng
On Tue, Aug 22, 2017 at 10:02 PM, Mark Brown wrote: > On Tue, Aug 22, 2017 at 09:24:32PM +0800, Donglin Peng wrote: > >> In case that dai->driver->name is NULL, strcmp(dai->driver->name, >> dlc->dai_name) will cause segmentation fault. >> so I think that we can change it as

Re: [PATCH 1/2] sched/wait: Break up long wake list walk

2017-08-22 Thread Linus Torvalds
On Tue, Aug 22, 2017 at 12:55 PM, Liang, Kan wrote: > >> So I propose testing the attached trivial patch. > > It doesn’t work. > The call stack is the same. So I would have expected the stack trace to be the same, and I would even expect the CPU usage to be fairly similar,

[PATCH v8 2/2] mtd: spi-nor: Add spi-nor mtd resume handler

2017-08-22 Thread Kamal Dasu
Implemented and populated spi-nor mtd PM handlers for resume ops. spi-nor resume op re-initializes spi-nor flash to its probed state by calling the newly implemented spi_nor_init() function. Signed-off-by: Kamal Dasu --- drivers/mtd/spi-nor/spi-nor.c | 14 ++ 1

Re: [PATCH v6 2/3]: perf/core: use context tstamp_data for skipped events on mux interrupt

2017-08-22 Thread Peter Zijlstra
On Thu, Aug 10, 2017 at 06:57:43PM +0300, Alexey Budankov wrote: > The key thing in the patch is explicit updating of tstamp fields for > INACTIVE events in update_event_times(). > @@ -1405,6 +1426,9 @@ static void update_event_times(struct perf_event *event) >

Re: [BUG] Deadlock due due to interactions of block, RCU, and cpu offline

2017-08-22 Thread Jeffrey Hugo
On 8/22/2017 10:12 AM, Paolo Bonzini wrote: On 20/08/2017 22:56, Paul E. McKenney wrote: KVM: async_pf: avoid async pf injection when in guest mode KVM: cpuid: Fix read/write out-of-bounds vulnerability in cpuid emulation arm: KVM: Allow unaligned accesses at HYP

Re: [PATCH 1/2] sched/wait: Break up long wake list walk

2017-08-22 Thread Peter Zijlstra
On Tue, Aug 22, 2017 at 01:42:13PM -0700, Linus Torvalds wrote: > +void wait_on_page_bit_or_yield(struct page *page, int bit_nr) > +{ > + if (PageWaiters(page)) { > + yield(); > + return; > + } > + wait_on_page_bit(page, bit_nr); > +} So _the_ problem with

[PATCH] scsi: lpfc: remove useless code in lpfc_sli4_bsg_link_diag_test

2017-08-22 Thread Gustavo A. R. Silva
Remove variable assignments. The value stored in local variable _rc_ is overwritten at line 2448:rc = lpfc_sli4_bsg_set_link_diag_state(phba, 0); before it can be used. Addresses-Coverity-ID: 1226935 Signed-off-by: Gustavo A. R. Silva --- This issue was detected by

Re: [PATCH v2 05/19] perf, tools: Support weak groups

2017-08-22 Thread Andi Kleen
On Tue, Aug 22, 2017 at 10:34:15AM +0200, Jiri Olsa wrote: > On Fri, Aug 11, 2017 at 04:26:20PM -0700, Andi Kleen wrote: > > SNIP > > > diff --git a/tools/perf/Documentation/perf-list.txt > > b/tools/perf/Documentation/perf-list.txt > > index f709de54707b..d432965d728d 100644 > > ---

Re: [PATCH] PCI: dwc: dra7xx: fix error return code in dra7xx_pcie_probe()

2017-08-22 Thread Gustavo A. R. Silva
Hi Bjorn, On 08/22/2017 03:37 PM, Bjorn Helgaas wrote: On Wed, Aug 09, 2017 at 11:16:03AM -0500, Gustavo A. R. Silva wrote: platform_get_irq() returns an error code, but the pci-dra7xx driver ignores it and always returns -EINVAL. This is not correct and, prevents -EPROBE_DEFER from being

Re: [PATCH] PCI: Fix and amend express capability sizes

2017-08-22 Thread Bjorn Helgaas
On Thu, Aug 10, 2017 at 10:54:31AM -0600, Alex Williamson wrote: > PCI_CAP_EXP_ENDPOINT_SIZEOF_V1 defines the size of the PCIe express > capability structure for v1 devices with link, but we also have a need > in the vfio code for sizing the capability for devices without link, > such as root

Re: [PATCH] net: ethernet: ax88796: make mdiobb_ops const

2017-08-22 Thread David Miller
From: Bhumika Goyal Date: Tue, 22 Aug 2017 13:41:19 +0530 > Make this const as it is only stored in a const field of a > mdiobb_ctrl structure. > > Signed-off-by: Bhumika Goyal Applied.

Re: [PATCH v2 0/3] PCI: mediatek: Add MSI support for MT2712 and MT7622

2017-08-22 Thread Bjorn Helgaas
On Mon, Aug 14, 2017 at 09:04:25PM +0800, honghui.zh...@mediatek.com wrote: > From: Honghui Zhang > > MT2712 and MT7622's PCIe host controller support MSI, but only 32bit MSI > address are supportted. It connect to GIC with the same IRQ number of INTx > IRQ, so it

Re: [PATCH] sg: recheck MMAP_IO request length with lock held

2017-08-22 Thread Douglas Gilbert
On 2017-08-16 12:48 AM, Todd Poynor wrote: Commit 1bc0eb044615 ("scsi: sg: protect accesses to 'reserved' page array") adds needed concurrency protection for the "reserve" buffer. Some checks that are initially made outside the lock are replicated once the lock is taken to ensure the checks and

Re: possible circular locking dependency detected [was: linux-next: Tree for Aug 22]

2017-08-22 Thread Bart Van Assche
On Tue, 2017-08-22 at 19:47 +0900, Sergey Senozhatsky wrote: > == > WARNING: possible circular locking dependency detected > 4.13.0-rc6-next-20170822-dbg-00020-g39758ed8aae0-dirty #1746 No

Re: [PATCH net-next] net: phy: Deal with unbound PHY driver in phy_attached_print()

2017-08-22 Thread David Miller
From: Florian Fainelli Date: Tue, 22 Aug 2017 14:45:44 -0700 > On 08/22/2017 02:44 PM, David Miller wrote: >> From: Florian Fainelli >> Date: Tue, 22 Aug 2017 14:26:47 -0700 >> >>> Priit reported that stmmac was crashing with the trace below. This is

[PATCH] apparmor: fix off-by-one comparison on MAXMAPPED_SIG

2017-08-22 Thread Colin King
From: Colin Ian King There is a an off-by-one comparision on sig against MAXMAPPED_SIG that can lead to a read outside the sig_map array if sig is MAXMAPPED_SIG. Fix this. Detected with cppcheck: "Either the condition 'sig<=35' is redundant or the array 'sig_map[35]'

Re: linux-next: Tree for Aug 22

2017-08-22 Thread Stephen Rothwell
Hi Paul, On Wed, 23 Aug 2017 07:57:05 +1000 Stephen Rothwell wrote: > > On Tue, 22 Aug 2017 12:36:20 -0700 "Paul E. McKenney" > wrote: > > > > To complete the thought, if you aren't already using it, I suggest > > applying Nick's patch: > >

Re: [PATCH v2 2/2] ASoC: rockchip: separate pinctrl pins from each other

2017-08-22 Thread Heiko Stuebner
Am Montag, 21. August 2017, 16:25:07 CEST schrieb Sugar Zhang: > pdm sdi0~3 pins are optional, for example, if 4ch required, > only sdi0~1 need to be enabled. > > Signed-off-by: Sugar Zhang Reviewed-by: Heiko Stuebner But you might want to adapt

Re: [PATCH net] ethernet: xircom: small clean up in setup_xirc2ps_cs()

2017-08-22 Thread David Miller
From: Dan Carpenter Date: Mon, 21 Aug 2017 12:47:30 +0300 > The get_options() function takes the whole ARRAY_SIZE(). It doesn't > matter here because we don't use more than 7 elements. > > Signed-off-by: Dan Carpenter Applied, thanks.

[PATCH] gpio: twl6040: remove unneeded forward declaration

2017-08-22 Thread Masahiro Yamada
There is no reference to twl6040gpo_chip before its definition. Signed-off-by: Masahiro Yamada --- drivers/gpio/gpio-twl6040.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpio/gpio-twl6040.c b/drivers/gpio/gpio-twl6040.c index

Fwd: [PATCH] x86/nmi/64: avoid passing user space rsp of pt_regs to nmi handler

2017-08-22 Thread oliver yang
Copied linux-kernel, as my email got rejected by the alias. -- Forwarded message -- From: yang oliver Date: 2017-08-23 1:09 GMT+08:00 Subject: [PATCH] x86/nmi/64: avoid passing user space rsp of pt_regs to nmi handler To: "t...@linutronix.de"

Re: [PATCH] tpm/tpm_crb: Access locality for non-ACPI and non-SMC start method

2017-08-22 Thread Jarkko Sakkinen
On Thu, Aug 17, 2017 at 11:15:36PM -0500, Jiandi An wrote: > For ARM64, the locality is handled by Trust Zone in FW. > The layout does not have crb_regs_head. It is hitting > the following line. > dev_warn(dev, FW_BUG "Bad ACPI memory layout"); > > Current code excludes CRB_FL_ACPI_START and

Re: [PATCH v2 3/5] dt-bindings: sdhci-omap: Add bindings for the sdhci-omap controller

2017-08-22 Thread Tony Lindgren
* Kishon Vijay Abraham I [170822 06:39]: > Add binding for the TI's sdhci-omap controller. This now includes only > a subset of properties documented in ti-omap-hsmmc.txt but will eventually > include all the properties. > > Signed-off-by: Kishon Vijay Abraham I >

Re: [PATCH v7 5/7] io-64-nonatomic: add io{read|write}64[be]{_lo_hi|_hi_lo} macros

2017-08-22 Thread Andy Shevchenko
On Tue, Aug 22, 2017 at 8:02 PM, Logan Gunthorpe wrote: > This patch adds generic io{read|write}64[be]{_lo_hi|_hi_lo} macros if > they are not already defined by the architecture. (As they are provided > by the generic iomap library). > > The patch also points

Re: [PATCH v2 1/3] media: V3s: Add support for Allwinner CSI.

2017-08-22 Thread Maxime Ripard
Hi Yong, On Mon, Jul 31, 2017 at 11:16:40AM +0800, Yong wrote: > > > @@ -143,6 +143,7 @@ source "drivers/media/platform/am437x/Kconfig" > > > source "drivers/media/platform/xilinx/Kconfig" > > > source "drivers/media/platform/rcar-vin/Kconfig" > > > source

Re: [PATCH v7 0/7] make io{read|write}64 globally usable

2017-08-22 Thread Andy Shevchenko
On Tue, Aug 22, 2017 at 8:02 PM, Logan Gunthorpe wrote: > This is just a resend seeing I've gotten no feedback in a couple weeks. Overall quite a good clean up! Thanks! After addressing my comments you may take my Reviewed-by: Andy Shevchenko >

[PATCH] ALSA: via82xx: Use common error handling code in snd_via82xx_create()

2017-08-22 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 22 Aug 2017 19:37:26 +0200 Add jump targets so that a bit of exception handling can be better reused at the end of this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring

Re: [RFC][PATCH] kselftests: timers: leap-a-day: Change default arguments to help test runs

2017-08-22 Thread Shuah Khan
On 08/18/2017 05:23 PM, John Stultz wrote: > Change default arguments for leap-a-day to always set the time > each iteration (rather then waiting for midnight UTC), and to > only run 10 interations (rather then infinite). > > If one wants to wait for midnight UTC, they can use the new -w > flag,

Re: [PATCH] ALSA: ctxfi: Use common error handling code in two functions

2017-08-22 Thread Takashi Iwai
On Tue, 22 Aug 2017 18:50:16 +0200, SF Markus Elfring wrote: > > From: Markus Elfring > Date: Tue, 22 Aug 2017 18:43:42 +0200 > > Add jump targets so that a bit of exception handling can be better reused > at the end of these functions. > > This issue was

Re: [PATCH v7 0/7] make io{read|write}64 globally usable

2017-08-22 Thread Logan Gunthorpe
On 22/08/17 11:43 AM, Andy Shevchenko wrote: Overall quite a good clean up! Thanks! After addressing my comments you may take my Reviewed-by: Andy Shevchenko Great, Thanks! Logan

Re: [PATCH] ALSA: via82xx: Use common error handling code in snd_via82xx_create()

2017-08-22 Thread Takashi Iwai
On Tue, 22 Aug 2017 19:43:08 +0200, SF Markus Elfring wrote: > > From: Markus Elfring > Date: Tue, 22 Aug 2017 19:37:26 +0200 > > Add jump targets so that a bit of exception handling can be better reused > at the end of this function. > > This issue was detected

Re: [PATCH V8 net-next 00/22] Huawei HiNIC Ethernet Driver

2017-08-22 Thread David Miller
From: Aviad Krawczyk Date: Mon, 21 Aug 2017 23:55:46 +0800 > The patch-set contains the support of the HiNIC Ethernet driver for > hinic family of PCIE Network Interface Cards. Series applied, thanks.

Re: [PATCH v3 3/4] net: stmmac: register parent MDIO node for sun8i-h3-emac

2017-08-22 Thread Corentin Labbe
On Tue, Aug 22, 2017 at 09:40:24AM -0700, Florian Fainelli wrote: > On 08/22/2017 08:39 AM, Chen-Yu Tsai wrote: > > On Mon, Aug 21, 2017 at 10:23 PM, Andrew Lunn wrote: > >>> All muxes are mostly always represented the same way afaik, or do you > >>> want to simply introduce a new

[PATCH] net: amd: constify zorro_device_id

2017-08-22 Thread Arvind Yadav
zorro_device_id are not supposed to change at runtime. All functions working with zorro_device_id provided by work with const zorro_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav --- drivers/net/ethernet/amd/a2065.c | 2 +-

Re: linux-next: Tree for Aug 22

2017-08-22 Thread Stephen Rothwell
Hi all, This tree fails to boot on my qemu test. 2 boot logs attached. Paul, Nick, is this the same/similar to the other RCU/lockup bug you are chasing. This is the first time I have seen this failure. This qemu boot is in full emulation mode if I add --enable-kvm to the qemu command, it does

Re: [PATCH 1/2] sched/wait: Break up long wake list walk

2017-08-22 Thread Linus Torvalds
On Tue, Aug 22, 2017 at 10:23 AM, Liang, Kan wrote: > > Although the patch doesn't trigger watchdog, the spin lock wait time > is not small (0.45s). > It may get worse again on larger systems. Yeah, I don't think Mel's patch is great - because I think we could do so much

[PATCH] video: fm2fb: constify zorro_device_id

2017-08-22 Thread Arvind Yadav
zorro_device_id are not supposed to change at runtime. All functions working with zorro_device_id provided by work with const zorro_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav --- drivers/video/fbdev/fm2fb.c | 2 +- 1 file changed,

[PATCH 1/2] net: phy: Freeze PHY polling before suspending devices

2017-08-22 Thread Geert Uytterhoeven
During system resume, the PHY state machine may be run from the workqueue while the Ethernet device (and its PHY) are still suspended, which may lead to a system crash. E.g. on sh73a0/kzm9g and r8a73a4/ape6evm, the external Ethernet chip is driven by a PM controlled clock. If the Ethernet

[PATCH 2/2] net: smsc911x: Quiten netif during suspend

2017-08-22 Thread Geert Uytterhoeven
If the network interface is kept running during suspend, the net core may call net_device_ops.ndo_start_xmit() while the Ethernet device is still suspended, which may lead to a system crash. E.g. on sh73a0/kzm9g and r8a73a4/ape6evm, the external Ethernet chip is driven by a PM controlled clock.

[PATCH v1 2/2] x86/perf: Export some PMU attributes in caps

2017-08-22 Thread Andi Kleen
From: Andi Kleen It can be difficult to figure out for user programs what features the x86 cpu pmu driver actually supports. Currently it requires grepping in dmesg, but dmesg is not always available. This adds a caps directory to /sys/devices/cpu, similar to the caps

Re: [PATCH v4 07/15] NTB: switchtec_ntb: Introduce initial NTB driver

2017-08-22 Thread Bjorn Helgaas
On Thu, Aug 03, 2017 at 12:19:46PM -0600, Logan Gunthorpe wrote: > Seeing the Switchtec NTB hardware shares the same endpoint as the > management endpoint we utilize the class_interface API to register > an NTB driver for every Switchtec device in the system that has the > NTB class code. > >

[PATCH 3.18 03/10] parisc: pci memory bar assignment fails with 64bit kernels on dino/cujo

2017-08-22 Thread Greg Kroah-Hartman
3.18-stable review patch. If anyone has any objections, please let me know. -- From: Thomas Bogendoerfer commit 4098116039911e8870d84c975e2ec22dab65a909 upstream. For 64bit kernels the lmmio_space_offset of the host bridge window isn't set correctly

Re: [RFC v2 2/4] iommu/arm-smmu-v3: Add tlbi_on_map option

2017-08-22 Thread Michael S. Tsirkin
On Fri, Aug 18, 2017 at 05:49:42AM +0300, Michael S. Tsirkin wrote: > On Thu, Aug 17, 2017 at 05:34:25PM +0100, Will Deacon wrote: > > On Fri, Aug 11, 2017 at 03:45:28PM +0200, Eric Auger wrote: > > > When running a virtual SMMU on a guest we sometimes need to trap > > > all changes to the

[PATCH 3.18 00/10] 3.18.67-stable review

2017-08-22 Thread Greg Kroah-Hartman
This is the start of the stable review cycle for the 3.18.67 release. There are 10 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know. Responses should be made by Thu Aug 24 19:08:42 UTC 2017. Anything

[PATCH 4.4 09/20] ALSA: usb-audio: Add mute TLV for playback volumes on C-Media devices

2017-08-22 Thread Greg Kroah-Hartman
4.4-stable review patch. If anyone has any objections, please let me know. -- From: Takashi Iwai commit 0f174b3525a43bd51f9397394763925e0ebe7bc7 upstream. C-Media devices (at least some models) mute the playback stream when volumes are set to the minimum value.

[PATCH 4.4 15/20] irqchip/atmel-aic: Fix unbalanced refcount in aic_common_rtc_irq_fixup()

2017-08-22 Thread Greg Kroah-Hartman
4.4-stable review patch. If anyone has any objections, please let me know. -- From: Boris Brezillon commit 277867ade8262583f4280cadbe90e0031a3706a7 upstream. of_find_compatible_node() is calling of_node_put() on its first argument thus

[PATCH 4.12 30/41] x86: Fix norandmaps/ADDR_NO_RANDOMIZE

2017-08-22 Thread Greg Kroah-Hartman
4.12-stable review patch. If anyone has any objections, please let me know. -- From: Oleg Nesterov commit 47ac5484fd961420e5ec0bb5b972fde381f57365 upstream. Documentation/admin-guide/kernel-parameters.txt says: norandmaps Don't use address space

[PATCH 4.12 41/41] usb: qmi_wwan: add D-Link DWM-222 device ID

2017-08-22 Thread Greg Kroah-Hartman
4.12-stable review patch. If anyone has any objections, please let me know. -- From: Hector Martin commit bed9ff165960921303a100228585f2d1691b42eb upstream. Signed-off-by: Hector Martin Signed-off-by: David S. Miller

[PATCH 4.12 37/41] Sanitize move_pages() permission checks

2017-08-22 Thread Greg Kroah-Hartman
4.12-stable review patch. If anyone has any objections, please let me know. -- From: Linus Torvalds commit 197e7e521384a23b9e585178f3f11c9fa08274b9 upstream. The 'move_paghes()' system call was introduced long long ago with the same permission

  1   2   3   4   5   6   7   8   9   10   >