Re: [PATCH net-next 2/2] net: dsa: lan9303: Add basic offloading of unicast traffic

2017-09-21 Thread Andrew Lunn
Hi Egil > +static void lan9303_bridge_ports(struct lan9303 *chip) > +{ > + /* ports bridged: remove mirroring */ > + lan9303_write_switch_reg(chip, LAN9303_SWE_PORT_MIRROR, 0); > +} Could you replace the 0 with something symbolic which makes this easier to understand. #define LAN9303_SWE

Re: RTL8192EE PCIe Wireless Network Adapter crashed with linux-4.13

2017-09-21 Thread Larry Finger
On 09/21/2017 06:37 AM, Zwindl wrote: Hi, I've reported to archlinux's bugzilla, and finally found out the flag which caused that issue, it's the `CONFIG_INTEL_IOMMU_DEFAULT_ON=y` flag, I think may this is a kernel bug, more details at https://bugs.archlinux.org/task/55665 My standard kernel h

Re: [RFC PATCH v5 6/6] i2c: dev: mark RDWR buffers as DMA_SAFE

2017-09-21 Thread Wolfram Sang
On Thu, Sep 21, 2017 at 03:17:44PM +0100, Jonathan Cameron wrote: > On Wed, 20 Sep 2017 20:59:56 +0200 > Wolfram Sang wrote: > > > Signed-off-by: Wolfram Sang > > Makes sense as do the other drivers. > > Feel free to add > > Reviewed-by: Jonathan Cameron > > to all of them (though they hard

Re: [PATCH net 1/3] net: mvpp2: fix the dma_mask and coherent_dma_mask settings for PPv2.2

2017-09-21 Thread Antoine Tenart
Hi David, On Mon, Sep 18, 2017 at 05:18:58PM -0700, David Miller wrote: > From: Antoine Tenart > Date: Mon, 18 Sep 2017 15:04:06 +0200 > > > The dev->dma_mask usually points to dev->coherent_dma_mask. This is an > > issue as setting both of them will override the other. This is > > problematic h

Re: [PATCH 3/4] iommu/arm-smmu-v3: Use NUMA memory allocations for stream tables and comamnd queues

2017-09-21 Thread Christoph Hellwig
On Thu, Sep 21, 2017 at 12:58:04PM +0100, Robin Murphy wrote: > Christoph, Marek; how reasonable do you think it is to expect > dma_alloc_coherent() to be inherently NUMA-aware on NUMA-capable > systems? SWIOTLB looks fairly straightforward to fix up (for the simple > allocation case; I'm not sure

Re: [v5 02/12] soc/fsl/qbman: Use shared-dma-pool for BMan private memory allocations

2017-09-21 Thread Catalin Marinas
On Mon, Sep 18, 2017 at 04:39:37PM -0400, Roy Pledge wrote: > Use the shared-memory-pool mechanism for free buffer proxy record > area allocation. > > Signed-off-by: Roy Pledge Reviewed-by: Catalin Marinas

Re: [PATCH net-next 2/2] net: dsa: lan9303: Add basic offloading of unicast traffic

2017-09-21 Thread Vivien Didelot
Hi Egil, Egil Hjelmeland writes: > When both user ports are joined to the same bridge, the normal > HW MAC learning is enabled. This means that unicast traffic is forwarded > in HW. > > If one of the user ports leave the bridge, > the ports goes back to the initial separated operation. > > Port

Re: [v5 01/12] soc/fsl/qbman: Add common routine for QBMan private allocations

2017-09-21 Thread Catalin Marinas
On Mon, Sep 18, 2017 at 04:39:36PM -0400, Roy Pledge wrote: > --- /dev/null > +++ b/drivers/soc/fsl/qbman/dpaa_sys.c > @@ -0,0 +1,78 @@ > +/* Copyright 2017 NXP Semiconductor, Inc. > + * > + * Redistribution and use in source and binary forms, with or without > + * modification, are permitted provi

Re: [v5 03/12] soc/fsl/qbman: Use shared-dma-pool for QMan private memory allocations

2017-09-21 Thread Catalin Marinas
On Mon, Sep 18, 2017 at 04:39:38PM -0400, Roy Pledge wrote: > Use the shared-memory-pool mechanism for frame queue descriptor and > packed frame descriptor record area allocations. > > Signed-off-by: Roy Pledge Reviewed-by: Catalin Marinas

Re: [v5 05/12] soc/fsl/qbman: Drop set/clear_bits usage

2017-09-21 Thread Catalin Marinas
On Mon, Sep 18, 2017 at 04:39:40PM -0400, Roy Pledge wrote: > From: Madalin Bucur > > Replace PPC specific set/clear_bits API with standard > bit twiddling so driver is portalable outside PPC. > > Signed-off-by: Madalin Bucur > Signed-off-by: Claudiu Manoil > Signed-off-by: Roy Pledge Review

Re: [v5 07/12] soc/fsl/qbman: Fix ARM32 typo

2017-09-21 Thread Catalin Marinas
On Mon, Sep 18, 2017 at 04:39:42PM -0400, Roy Pledge wrote: > From: Valentin Rothberg > > The Kconfig symbol for 32bit ARM is 'ARM', not 'ARM32'. > > Signed-off-by: Valentin Rothberg > Signed-off-by: Claudiu Manoil > Signed-off-by: Roy Pledge Reviewed-by: Catalin Marinas

Re: [v5 06/12] soc/fsl/qbman: Drop L1_CACHE_BYTES compile time check

2017-09-21 Thread Catalin Marinas
On Mon, Sep 18, 2017 at 04:39:41PM -0400, Roy Pledge wrote: > From: Claudiu Manoil > > Not relevant and arch dependent. Overkill for PPC. > > Signed-off-by: Claudiu Manoil > Signed-off-by: Roy Pledge Reviewed-by: Catalin Marinas

Re: [PATCH][drm-next] drm/i915/gvt: ensure -ve return value is handled correctly

2017-09-21 Thread Joonas Lahtinen
On Thu, 2017-09-21 at 06:44 +0800, Zhenyu Wang wrote: > On 2017.09.19 19:35:23 -0700, Joe Perches wrote: > > On Wed, 2017-09-20 at 05:46 +0800, Zhenyu Wang wrote: > > > On 2017.09.19 16:55:34 +0100, Colin King wrote: > > > > From: Colin Ian King > > > > > > > > An earlier fix changed the return t

Re: [v5 08/12] soc/fsl/qbman: Rework portal mapping calls for ARM/PPC

2017-09-21 Thread Catalin Marinas
On Mon, Sep 18, 2017 at 04:39:43PM -0400, Roy Pledge wrote: > Rework portal mapping for PPC and ARM. The PPC devices require a > cacheable coherent mapping while ARM will work with a non-cachable/write > combine mapping. This also eliminates the need for manual cache > flushes on ARM. This also fix

Re: [PATCH] PM: Document rules on using pm_runtime_resume() in system suspend callbacks

2017-09-21 Thread Alan Stern
On Thu, 21 Sep 2017, Johannes Stezenbach wrote: > On Thu, Sep 21, 2017 at 02:39:30AM +0200, Rafael J. Wysocki wrote: > > On Wed, Sep 20, 2017 at 6:27 PM, Johannes Stezenbach wrote: > > > > > > E.g. an audio codec could keep running > > > while the i2c bus used to program its registers can be run

Re: [PATCH V6 2/3] dma-mapping: Rework dma_get_cache_alignment()function

2017-09-21 Thread Christoph Hellwig
On Thu, Sep 21, 2017 at 12:28:25PM +0800, 陈华才 wrote: > Hi, Christoph, > > I have changed dma_get_cache_alignment's return value, and I don't know > whether those drivers want to return ARCH_DMA_MINALIGN unconditionally. So I > pass a NULL for those drivers, in order to keep their old behavior.

Re: [RFC PATCH v5 2/6] i2c: add helpers to ease DMA handling

2017-09-21 Thread Jonathan Cameron
On Thu, 21 Sep 2017 16:15:28 +0200 Wolfram Sang wrote: > > > > +/** > > > > + * i2c_release_dma_safe_msg_buf - release DMA safe buffer and sync > > > > with i2c_msg > > > > + * @msg: the message to be synced with > > > > + * @buf: the buffer obtained from i2c_get_dma_safe_msg_buf(). May be > >

Re: [PATCH] PM: Document rules on using pm_runtime_resume() in system suspend callbacks

2017-09-21 Thread Rafael J. Wysocki
On Thursday, September 21, 2017 11:27:13 AM CEST Johannes Stezenbach wrote: > On Thu, Sep 21, 2017 at 02:39:30AM +0200, Rafael J. Wysocki wrote: > > On Wed, Sep 20, 2017 at 6:27 PM, Johannes Stezenbach wrote: > > > > > > E.g. an audio codec could keep running > > > while the i2c bus used to progr

Re: [v5 00/12] soc/fsl/qbman: Enable QBMan on ARM Platforms

2017-09-21 Thread Catalin Marinas
On Mon, Sep 18, 2017 at 04:39:35PM -0400, Roy Pledge wrote: > Madalin Bucur (4): > soc/fsl/qbman: Drop set/clear_bits usage > soc/fsl/qbman: add QMAN_REV32 > soc/fsl/qbman: different register offsets on ARM > soc/fsl/qbman: Enable FSL_LAYERSCAPE config on ARM > > Roy Pledge (5): > soc/fs

Re: Worker threads in D state since c5a94a618e7ac86 (workqueue: Use TASK_IDLE)

2017-09-21 Thread Markus Trippelsdorf
On 2017.09.21 at 14:30 +0200, Peter Zijlstra wrote: > On Thu, Sep 21, 2017 at 01:08:42PM +0200, Markus Trippelsdorf wrote: > > On 2017.09.11 at 16:21 +0200, Markus Trippelsdorf wrote: > > > On 2017.09.11 at 06:11 -0700, Tejun Heo wrote: > > > > Hello, > > > > > > > > On Sun, Sep 10, 2017 at 09:36:5

Re: [PATCH] xen: support 52 bit physical addresses in pv guests

2017-09-21 Thread Juergen Gross
On 21/09/17 16:14, Boris Ostrovsky wrote: > > > On 09/21/2017 04:01 AM, Juergen Gross wrote: >> Physical addresses on processors supporting 5 level paging can be up to >> 52 bits wide. For a Xen pv guest running on such a machine those >> physical addresses have to be supported in order to be abl

Re: [PATCH] fpga: make xlnx_pr_decoupler_br_ops const

2017-09-21 Thread Alan Tull
On Tue, Aug 29, 2017 at 11:09 AM, Moritz Fischer wrote: > On Tue, Aug 29, 2017 at 12:00 AM, Michal Simek > wrote: >> On 28.8.2017 19:32, Bhumika Goyal wrote: >>> Make this const as it is only passed to a const argument of the function >>> fpga_bridge_register. >>> >>> Signed-off-by: Bhumika Goya

Re: [PATCH net-next 1/2] net: dsa: lan9303: Move tag setup to new lan9303_setup_tagging

2017-09-21 Thread Vivien Didelot
Hi Egil, Egil Hjelmeland writes: > Prepare for next patch: > Move tag setup from lan9303_separate_ports() to new function > lan9303_setup_tagging() > > Signed-off-by: Egil Hjelmeland Minor styling issues, otherwise LGTM: Reviewed-by: Vivien Didelot > +/* forward special tagged packets from

Re: [PATCH] ARM: davinci_all_defconfig: enable CMA options

2017-09-21 Thread Sekhar Nori
On Wednesday 20 September 2017 11:31 PM, Suman Anna wrote: > Enable the CMA and DMA_CMA Kconfig options by default for > Davinci platforms. Davinci remoteproc driver is one of the > modules that depends on these options, and this allows the > driver to be made visible for selection with menuconfig.

Re: [RFC PATCH 0/7] AXP803 AC/Battery support

2017-09-21 Thread Jonathan Cameron
On Wed, 20 Sep 2017 23:18:07 +0800 Icenowy Zheng wrote: > The AXP803 PMIC, used by most Allwinner A64 boards, features 3 power inputs: > AC, USB and Battery. > > This patchset adds support for the AC and Battery supplies, which is useful > for the boards from Pine64 (Pine64, SoPine w/ baseboard

Re: [PATCH 1/2] ARM: smccc-call: Use r12 to route secure monitor calls on TI platforms

2017-09-21 Thread Mark Rutland
On Tue, Sep 19, 2017 at 10:07:39AM -0500, Andrew F. Davis wrote: > On 09/19/2017 09:19 AM, Mark Rutland wrote: > > On Mon, Sep 18, 2017 at 03:50:04PM -0500, Andrew F. Davis wrote: > >> We need a way to signal that a call is for the OP-TEE SM and not for > >> the ROM SM in a way that is safe for the

[PATCH] block: drop "sending ioctl to a partition" message

2017-09-21 Thread Paolo Bonzini
After the first few months, the message has not led to many bug reports. It's been almost five years now, and in practice the main source of it seems to be MTIOCGET that someone is using to detect tape devices. While we could whitelist it just like CDROM_GET_CAPABILITY, this patch just removes the

Re: [PATCH v2] vfs: introduce UMOUNT_WAIT which waits for umount completion

2017-09-21 Thread Theodore Ts'o
On Wed, Sep 20, 2017 at 10:02:37PM -0700, Jaegeuk Kim wrote: > > Yes, and actually, android retries umount(2) for several seconds, if it gets > failure. So, first I thought it'd be better to make umount() more > deterministic. > > I'm not sure how many times we can retry and wait for this. IMHO,

Re: RTL8192EE PCIe Wireless Network Adapter crashed with linux-4.13

2017-09-21 Thread Kalle Valo
Larry Finger writes: > On 09/21/2017 06:37 AM, Zwindl wrote: >> Hi, I've reported to archlinux's bugzilla, and finally found out the >> flag which caused that issue, it's the >> `CONFIG_INTEL_IOMMU_DEFAULT_ON=y` flag, I think may this is a kernel >> bug, more details at https://bugs.archlinux.org

Re: [PATCH 9/9 v2] usb: usb251xb: Use GPIO descriptor consumer interface

2017-09-21 Thread Serge Semin
On Thu, Sep 21, 2017 at 10:23:38AM +0200, Greg KH wrote: > On Sat, Sep 16, 2017 at 01:42:20PM +0300, Serge Semin wrote: > > diff --git a/drivers/usb/misc/usb251xb.c b/drivers/usb/misc/usb251xb.c > > index 71994b883..c2dd9742f 100644 > > --- a/drivers/usb/misc/usb251xb.c > > +++ b/drivers/usb/misc

Re: [PATCH] block: drop "sending ioctl to a partition" message

2017-09-21 Thread Christoph Hellwig
This looks ok to me, but do we even need to keep the special cases above? Is there anything relying on the safe but not very useful ioctls? Condensing the thing down to: int scsi_verify_blk_ioctl(struct block_device *bd, unsigned int cmd) { if (bd && bd == bd->bd_contains)

[PATCH 1/1] fpga: make xlnx_pr_decoupler_br_ops const

2017-09-21 Thread Alan Tull
From: Bhumika Goyal Make this const as it is only passed to a const argument of the function fpga_bridge_register. Signed-off-by: Bhumika Goyal Acked-by: Michal Simek Acked-by: Moritz Fischer Signed-off-by: Alan Tull --- drivers/fpga/xilinx-pr-decoupler.c | 2 +- 1 file changed, 1 insertion

Re: [PATCH] PM: Document rules on using pm_runtime_resume() in system suspend callbacks

2017-09-21 Thread Rafael J. Wysocki
On Thursday, September 21, 2017 4:36:30 PM CEST Alan Stern wrote: > On Thu, 21 Sep 2017, Johannes Stezenbach wrote: > > > On Thu, Sep 21, 2017 at 02:39:30AM +0200, Rafael J. Wysocki wrote: > > > On Wed, Sep 20, 2017 at 6:27 PM, Johannes Stezenbach > > > wrote: > > > > > > > > E.g. an audio code

[PATCH 0/1] patch for fpga

2017-09-21 Thread Alan Tull
Hi Greg, Please take this one small fix for fpga. Thanks! Alan Bhumika Goyal (1): fpga: make xlnx_pr_decoupler_br_ops const drivers/fpga/xilinx-pr-decoupler.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- 2.11.0

Re: [PATCH 1/1] wireless: broadcom: brcm80211: use setup_timer() helper

2017-09-21 Thread Kalle Valo
Allen Pais writes: > Use setup_timer function instead of initializing timer with the > function and data fields. > > Signed-off-by: Allen Pais The commit log is weirdly indented and no need to have "wireless: broadcom:" in the title. I can fix both of those. -- Kalle Valo

Re: [PATCH 1/7] buffer: cleanup free_more_memory() flusher wakeup

2017-09-21 Thread Christoph Hellwig
Looks fine, Reviewed-by: Christoph Hellwig

Re: [PATCH 2/7] fs: kill 'nr_pages' argument from wakeup_flusher_threads()

2017-09-21 Thread Christoph Hellwig
Looks fine, Reviewed-by: Christoph Hellwig

Re: [PATCH 3/7] fs-writeback: provide a wakeup_flusher_threads_bdi()

2017-09-21 Thread Christoph Hellwig
Looks fine, Reviewed-by: Christoph Hellwig

Re: [PATCH ALT4 V3 1/2] audit: show fstype:pathname for entries with anonymous parents

2017-09-21 Thread Richard Guy Briggs
On 2017-09-20 12:52, Paul Moore wrote: > On Wed, Aug 23, 2017 at 7:03 AM, Richard Guy Briggs wrote: > > Tracefs or debugfs were causing hundreds to thousands of null PATH > > records to be associated with the init_module and finit_module SYSCALL > > records on a few modules when the following rule

Re: [PATCH 4/7] page-writeback: pass in '0' for nr_pages writeback in laptop mode

2017-09-21 Thread Christoph Hellwig
On Wed, Sep 20, 2017 at 09:32:59AM -0600, Jens Axboe wrote: > Laptop mode really wants to writeback the number of dirty > pages and inodes. Instead of calculating this in the caller, > just pass in 0 and let wakeup_flusher_threads() handle it. > > Use the new wakeup_flusher_threads_bdi() instead o

Re: [PATCH] block: drop "sending ioctl to a partition" message

2017-09-21 Thread Paolo Bonzini
On 21/09/2017 16:53, Christoph Hellwig wrote: > This looks ok to me, but do we even need to keep the special > cases above? Is there anything relying on the safe but not very > useful ioctls? No idea, I stuck to the usual "don't break userspace" rule. Honestly I doubt anything is using most of t

Re: [PATCH 0/8] pinctrl: meson: clean pin offsets

2017-09-21 Thread Jerome Brunet
On Thu, 2017-09-21 at 14:21 +0200, Linus Walleij wrote: > On Wed, Sep 20, 2017 at 3:39 PM, Jerome Brunet wrote: > > > The initial goal of this series was move to TEST_N pin from the EE > > controller to AO controller, where it belongs. This meant modify the > > EE_OFF value. > > > > This offset

Re: [PATCH 4/7] page-writeback: pass in '0' for nr_pages writeback in laptop mode

2017-09-21 Thread Christoph Hellwig
On Wed, Sep 20, 2017 at 09:32:59AM -0600, Jens Axboe wrote: > Use the new wakeup_flusher_threads_bdi() instead of rolling > our own. This changes the writeback to not be range cyclic, > but that should not matter for laptop mode flush-all > semantics. Oh btw - I think this actually is the more imp

Re: [PATCH 5/7] fs-writeback: make wb_start_writeback() static

2017-09-21 Thread Christoph Hellwig
Looks fine, Reviewed-by: Christoph Hellwig

Re: [PATCH] tool/perf: fix perf compiling errors

2017-09-21 Thread Arnaldo Carvalho de Melo
Em Thu, Sep 21, 2017 at 02:01:13PM +0800, Li Zhijian escreveu: > since f045b8c, we failed to build perf with LIBCLANGLLVM=1, this patch is to > fix the following compiling errors > -- > lizhijian@haswell-OptiPlex-9020:~/lkp/linux/tools/perf$ make > LLVM_CONFIG=/usr/bin/llvm-config-3.9

Re: [PATCH 6/7] fs-writeback: move nr_pages == 0 logic to one location

2017-09-21 Thread Christoph Hellwig
Looks fine, Reviewed-by: Christoph Hellwig

Re: [PATCH 7/7] fs-writeback: only allow one inflight and pending full flush

2017-09-21 Thread Christoph Hellwig
On Wed, Sep 20, 2017 at 09:33:02AM -0600, Jens Axboe wrote: > When someone calls wakeup_flusher_threads() or > wakeup_flusher_threads_bdi(), they schedule writeback of all dirty > pages in the system (or on that bdi). If we are tight on memory, we > can get tons of these queued from kswapd/vmscan.

Re: [PATCH] ath10: mark PM functions as __maybe_unused

2017-09-21 Thread Kalle Valo
Arnd Bergmann writes: > When CONFIG_PM_SLEEP is disabled, we get a compile-time > warning: > > drivers/net/wireless/ath/ath10k/pci.c:3417:12: error: 'ath10k_pci_pm_resume' > defined but not used [-Werror=unused-function] > static int ath10k_pci_pm_resume(struct device *dev) > ^~

[PATCH 0/4] [media] usbvision-core: Fine-tuning for some function implementations

2017-09-21 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 21 Sep 2017 17:00:17 +0200 A few update suggestions were taken into account from static source code analysis. Markus Elfring (4): Use common error handling code in usbvision_set_input() Use common error handling code in usbvision_set_compress_params() Delete

[PATCH 1/4] [media] usbvision-core: Use common error handling code in usbvision_set_input()

2017-09-21 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 21 Sep 2017 11:50:54 +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. * Replace the local variable "proc" by the identifier "__func__". *

Re: [PATCH 9/9 v2] usb: usb251xb: Use GPIO descriptor consumer interface

2017-09-21 Thread Greg KH
On Thu, Sep 21, 2017 at 05:51:29PM +0300, Serge Semin wrote: > On Thu, Sep 21, 2017 at 10:23:38AM +0200, Greg KH > wrote: > > On Sat, Sep 16, 2017 at 01:42:20PM +0300, Serge Semin wrote: > > > diff --git a/drivers/usb/misc/usb251xb.c b/drivers/usb/misc/usb251xb.c > > > index 71994b883..c2dd9742f

[PATCH 2/4] [media] usbvision-core: Use common error handling code in usbvision_set_compress_params()

2017-09-21 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 21 Sep 2017 12:45:49 +0200 * Add a jump target so that a bit of exception handling can be better reused at the end of this function. * Replace the local variable "proc" by the identifier "__func__". * Use the interface "dev_err" instead of "printk". Signed-off

[PATCH 3/4] [media] usbvision-core: Delete unnecessary braces in 11 functions

2017-09-21 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 21 Sep 2017 16:24:20 +0200 Do not use curly brackets at some source code places where a single statement should be sufficient. Signed-off-by: Markus Elfring --- drivers/media/usb/usbvision/usbvision-core.c | 71 1 file changed, 31 in

[PATCH 4/4] [media] usbvision-core: Replace four printk() calls by dev_err()

2017-09-21 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 21 Sep 2017 16:47:28 +0200 * Replace the local variable "proc" by the identifier "__func__". * Use the interface "dev_err" instead of "printk" in these functions. Signed-off-by: Markus Elfring --- drivers/media/usb/usbvision/usbvision-core.c | 18 +++---

Re: [PATCH] iommu/amd: Use raw_cpu_ptr() instead of get_cpu_ptr() for ->flush_queue

2017-09-21 Thread Sebastian Andrzej Siewior
On 2017-09-11 22:22:11 [-0400], Vinod Adhikary wrote: > Dear all, Hi, > Thank you for the great community support and support from Sebastian to > provide me this patch. I wanted to send this email to inform you and > perhaps get some information on how I could keep myself updated on updates > in r

Re: [RFC PATCH v2 1/2] membarrier: Provide register expedited private command

2017-09-21 Thread Mathieu Desnoyers
- On Sep 21, 2017, at 8:13 AM, Peter Zijlstra pet...@infradead.org wrote: > On Mon, Sep 18, 2017 at 06:36:47PM -0400, Mathieu Desnoyers wrote: >> @@ -3373,6 +3362,7 @@ static void __sched notrace __schedule(bool preempt) >> >> /* Also unlocks the rq: */ >> rq = cont

Re: [PATCH] ARM: davinci_all_defconfig: enable CMA options

2017-09-21 Thread Suman Anna
On 09/21/2017 09:43 AM, Sekhar Nori wrote: > On Wednesday 20 September 2017 11:31 PM, Suman Anna wrote: >> Enable the CMA and DMA_CMA Kconfig options by default for >> Davinci platforms. Davinci remoteproc driver is one of the >> modules that depends on these options, and this allows the >> driver

Re: [PATCH v3 0/5] Call GetEventLog before ExitBootServices

2017-09-21 Thread Jarkko Sakkinen
On Wed, Sep 20, 2017 at 10:13:35AM +0200, Thiebaud Weksteen wrote: > With TPM 1.2, the ACPI table ("TCPA") has two fields to recover the Event > Log Area (LAML and LASA). These logs are useful to understand and rebuild > the final values of PCRs. > > With TPM 2.0, the ACPI table ("TPM2") does not

Re: [PATCH 2/2] mfd: intel-lpss: switch to suspend_late()/resume_early()

2017-09-21 Thread Lee Jones
On Thu, 21 Sep 2017, Rafael J. Wysocki wrote: > On Thursday, September 21, 2017 10:41:33 AM CEST Lee Jones wrote: > > On Thu, 21 Sep 2017, Rafael J. Wysocki wrote: > > > > > On Thu, Sep 21, 2017 at 12:31 AM, Rajat Jain wrote: > > > > Ref: https://lkml.org/lkml/2017/9/19/649 > > > > > > > > The i

Re: [PATCH v1 1/6] perf record: Record the first sample time and last sample time to perf file header

2017-09-21 Thread Jiri Olsa
On Wed, Sep 20, 2017 at 11:12:32PM +0800, Jin Yao wrote: SNIP > @@ -2686,6 +2731,11 @@ int perf_file_header__read(struct perf_file_header > *header, > } > } > > + if (ph->needs_swap && format_time) { > + header->first_sample_time = bswap_64(header->first_sam

Re: [PATCH v1 1/6] perf record: Record the first sample time and last sample time to perf file header

2017-09-21 Thread Jiri Olsa
On Wed, Sep 20, 2017 at 11:12:32PM +0800, Jin Yao wrote: SNIP > @@ -2647,11 +2681,22 @@ int perf_file_header__read(struct perf_file_header > *header, > > if (header->size != sizeof(*header)) { > /* Support the previous format */ > - if (header->size == offsetof(

Re: [PATCH 2/2] mfd: intel-lpss: switch to suspend_late()/resume_early()

2017-09-21 Thread Rafael J. Wysocki
On Thu, Sep 21, 2017 at 5:17 PM, Lee Jones wrote: > On Thu, 21 Sep 2017, Rafael J. Wysocki wrote: > >> On Thursday, September 21, 2017 10:41:33 AM CEST Lee Jones wrote: >> > On Thu, 21 Sep 2017, Rafael J. Wysocki wrote: >> > >> > > On Thu, Sep 21, 2017 at 12:31 AM, Rajat Jain wrote: >> > > > Ref:

Re: [PATCH v2 2/3] ASoC: fsl-asoc-card: Handle return value of devm_kasprintf

2017-09-21 Thread Mark Brown
On Thu, Sep 21, 2017 at 10:50:03AM +0530, Arvind Yadav wrote: > devm_kasprintf() can fail here and we must check its return value. This is patch 2/3 but you've only sent me this one patch. This is missing the point of numbering a series, the only reason for the numbering is to provide ordering wi

Re: [RFC PATCH 0/7] AXP803 AC/Battery support

2017-09-21 Thread Icenowy Zheng
于 2017年9月21日 GMT+08:00 下午10:46:21, Jonathan Cameron 写到: >On Wed, 20 Sep 2017 23:18:07 +0800 >Icenowy Zheng wrote: > >> The AXP803 PMIC, used by most Allwinner A64 boards, features 3 power >inputs: >> AC, USB and Battery. >> >> This patchset adds support for the AC and Battery supplies, which

Re: [PATCH v1 1/6] perf record: Record the first sample time and last sample time to perf file header

2017-09-21 Thread Jiri Olsa
On Thu, Sep 21, 2017 at 05:18:42PM +0200, Jiri Olsa wrote: > On Wed, Sep 20, 2017 at 11:12:32PM +0800, Jin Yao wrote: > > SNIP > > > @@ -2647,11 +2681,22 @@ int perf_file_header__read(struct perf_file_header > > *header, > > > > if (header->size != sizeof(*header)) { > > /* Sup

[PATCH v2] iommu/iova: Use raw_cpu_ptr() instead of get_cpu_ptr() for ->fq

2017-09-21 Thread Sebastian Andrzej Siewior
get_cpu_ptr() disabled preemption and returns the ->fq object of the current CPU. raw_cpu_ptr() does the same except that it not disable preemption which means the scheduler can move it to another CPU after it obtained the per-CPU object. In this case this is not bad because the data structure itse

Re: [PATCH v3 01/31] usercopy: Prepare for usercopy whitelisting

2017-09-21 Thread Christopher Lameter
On Wed, 20 Sep 2017, Kees Cook wrote: > diff --git a/include/linux/stddef.h b/include/linux/stddef.h > index 9c61c7cda936..f00355086fb2 100644 > --- a/include/linux/stddef.h > +++ b/include/linux/stddef.h > @@ -18,6 +18,8 @@ enum { > #define offsetof(TYPE, MEMBER) ((size_t)&((TYPE *)0)->MEM

Re: [PATCH v3 02/31] usercopy: Enforce slab cache usercopy region boundaries

2017-09-21 Thread Christopher Lameter
On Wed, 20 Sep 2017, Kees Cook wrote: > diff --git a/mm/slab.c b/mm/slab.c > index 87b6e5e0cdaf..df268999cf02 100644 > --- a/mm/slab.c > +++ b/mm/slab.c > @@ -4408,7 +4408,9 @@ module_init(slab_proc_init); > > #ifdef CONFIG_HARDENED_USERCOPY > /* > - * Rejects objects that are incorrectly sized.

Re: [PATCH V3] acpi: apei: clear error status before acknowledging the error

2017-09-21 Thread Tyler Baicar
On 9/13/2017 8:40 AM, Baicar, Tyler wrote: On 8/29/2017 2:16 AM, Borislav Petkov wrote: On Mon, Aug 28, 2017 at 10:53:41AM -0600, Tyler Baicar wrote: Currently we acknowledge errors before clearing the error status. This could cause a new error to be populated by firmware in-between the error a

Re: [PATCH v3 03/31] usercopy: Mark kmalloc caches as usercopy caches

2017-09-21 Thread Christopher Lameter
On Wed, 20 Sep 2017, Kees Cook wrote: > --- a/mm/slab.c > +++ b/mm/slab.c > @@ -1291,7 +1291,8 @@ void __init kmem_cache_init(void) >*/ > kmalloc_caches[INDEX_NODE] = create_kmalloc_cache( > kmalloc_info[INDEX_NODE].name, > -

Re: [RFC PATCH v2 1/2] membarrier: Provide register expedited private command

2017-09-21 Thread Paul E. McKenney
On Thu, Sep 21, 2017 at 03:11:43PM +, Mathieu Desnoyers wrote: > - On Sep 21, 2017, at 8:13 AM, Peter Zijlstra pet...@infradead.org wrote: > > > On Mon, Sep 18, 2017 at 06:36:47PM -0400, Mathieu Desnoyers wrote: > >> @@ -3373,6 +3362,7 @@ static void __sched notrace __schedule(bool preempt

Re: Memory-ordering recipes

2017-09-21 Thread Paul E. McKenney
On Thu, Sep 21, 2017 at 02:45:31PM +0200, Peter Zijlstra wrote: > On Sun, Sep 17, 2017 at 04:05:09PM -0700, Paul E. McKenney wrote: > > > So what litmus tests are needed? Here is my initial set: > > > > 1. Release-acquire chains, AKA ISA2, Z6.2, LB, and 3.LB I grouped the expansions of all of

[PATCH v5 0/4] Add STM32 DMAMUX support

2017-09-21 Thread Pierre-Yves MORDRET
This patchset adds support for the STM32 DMA multiplexer. It allows to map any peripheral DMA request to any channel of the product DMAs. This IP has been introduced with STM32H7 SoC. --- Version history: v5: * Set selected channel ID within a lock to avoid race condition. R

[PATCH v5 4/4] ARM: configs: stm32: Add DMAMUX support in STM32 defconfig

2017-09-21 Thread Pierre-Yves MORDRET
This patch adds DMAMUX support in STM32 defconfig file Signed-off-by: M'boumba Cedric Madianga Signed-off-by: Pierre-Yves MORDRET --- Version history: v5: v4: v3: v2: * None --- --- arch/arm/configs/stm32_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/arc

[PATCH v5 2/4] dmaengine: Add STM32 DMAMUX driver

2017-09-21 Thread Pierre-Yves MORDRET
This patch implements the STM32 DMAMUX driver. The DMAMUX request multiplexer allows routing a DMA request line between the peripherals and the DMA controllers of the product. The routing function is ensured by a programmable multi-channel DMA request line multiplexer. Each channel selects a uniqu

[PATCH v5 3/4] dt-bindings: stm32-dma: add a property to handle STM32 DMAMUX

2017-09-21 Thread Pierre-Yves MORDRET
STM32 DMA controller has to exposed its number of request line to be addressed via STM32 DMAMUX. Signed-off-by: M'boumba Cedric Madianga Signed-off-by: Pierre-Yves MORDRET --- Version history: v5: v4: * get rid of st,dmamux property * number of DMA requests is exposed fo

Re: Query regarding synchronize_sched_expedited and resched_cpu

2017-09-21 Thread Paul E. McKenney
On Thu, Sep 21, 2017 at 03:55:46PM +0200, Peter Zijlstra wrote: > On Mon, Sep 18, 2017 at 11:11:05AM -0400, Steven Rostedt wrote: > > On Sun, 17 Sep 2017 11:37:06 +0530 > > Neeraj Upadhyay wrote: > > > > > Hi Paul, how about replacing raw_spin_trylock_irqsave with > > > raw_spin_lock_irqsave in r

[PATCH v5 1/4] dt-bindings: Document the STM32 DMAMUX bindings

2017-09-21 Thread Pierre-Yves MORDRET
This patch adds the documentation of device tree bindings for the STM32 DMAMUX. Signed-off-by: M'boumba Cedric Madianga Signed-off-by: Pierre-Yves MORDRET --- Version history: v5: v4: * Add multi-master ability for STM32 DMAMUX * Get rid of st,dmamux properties v3:

Re: Query regarding synchronize_sched_expedited and resched_cpu

2017-09-21 Thread Paul E. McKenney
On Thu, Sep 21, 2017 at 03:57:49PM +0200, Peter Zijlstra wrote: > On Mon, Sep 18, 2017 at 09:55:27AM -0700, Paul E. McKenney wrote: > > On Mon, Sep 18, 2017 at 12:29:31PM -0400, Steven Rostedt wrote: > > > On Mon, 18 Sep 2017 09:24:12 -0700 > > > "Paul E. McKenney" wrote: > > > > > > > > > > As

Re: [PATCH 2/2] x86/asm: Fix inline asm call constraints for clang

2017-09-21 Thread Ingo Molnar
* Josh Poimboeuf wrote: > On Wed, Sep 20, 2017 at 10:32:43AM -0700, H. Peter Anvin wrote: > > On 09/19/17 11:45, Josh Poimboeuf wrote: > > > For inline asm statements which have a CALL instruction, we list the > > > stack pointer as a constraint to convince GCC to ensure the frame > > > pointer

Re: [PATCH 7/7] fs-writeback: only allow one inflight and pending full flush

2017-09-21 Thread Jens Axboe
On 09/21/2017 09:05 AM, Christoph Hellwig wrote: > On Wed, Sep 20, 2017 at 09:33:02AM -0600, Jens Axboe wrote: >> When someone calls wakeup_flusher_threads() or >> wakeup_flusher_threads_bdi(), they schedule writeback of all dirty >> pages in the system (or on that bdi). If we are tight on memory,

usb/sound/usx2y: warning in usb_stream_new/__alloc_pages_slowpath

2017-09-21 Thread Andrey Konovalov
Hi! I've got the following report while fuzzing the kernel with syzkaller. On commit ebb2c2437d8008d46796902ff390653822af6cc4 (Sep 18). [ cut here ] WARNING: CPU: 1 PID: 1846 at mm/page_alloc.c:3883 __alloc_pages_slowpath+0x1ef2/0x2d70 Modules linked in: CPU: 1 PID: 1846

usb/storage/uas: slab-out-of-bounds in uas_probe

2017-09-21 Thread Andrey Konovalov
Hi! I've got the following report while fuzzing the kernel with syzkaller. On commit ebb2c2437d8008d46796902ff390653822af6cc4 (Sep 18). The issue occurs when we iterate over interface altsettings, but I don't see the driver doing anything wrong. I might be missing something, or this might be an

usb/media/smsusb: use-after-free in worker_thread

2017-09-21 Thread Andrey Konovalov
Hi! I've got the following report while fuzzing the kernel with syzkaller. On commit ebb2c2437d8008d46796902ff390653822af6cc4 (Sep 18). smsusb:smsusb_probe: board id=1, interface number 0 smsusb:siano_media_device_register: media controller created smsusb:smsusb1_detectmode: product string not f

usb/net/hso: global-out-of-bounds in hso_probe

2017-09-21 Thread Andrey Konovalov
Hi! I've got the following report while fuzzing the kernel with syzkaller. On commit ebb2c2437d8008d46796902ff390653822af6cc4 (Sep 18). Is seems there's no check on the if_num value when it's used in ((u32 *)(id->driver_info))[if_num]. ===

usb/net/hso: warning in hso_free_net_device

2017-09-21 Thread Andrey Konovalov
Hi! I've got the following report while fuzzing the kernel with syzkaller. On commit ebb2c2437d8008d46796902ff390653822af6cc4 (Sep 18). Looks like hso_create_net_device() can do goto exit before registering network device. hso 1-1:4.0: Can't find BULK IN endpoint [ cut here ]---

usb/media/hdpvr: trying to register non-static key in hdpvr_probe

2017-09-21 Thread Andrey Konovalov
Hi! I've got the following report while fuzzing the kernel with syzkaller. On commit ebb2c2437d8008d46796902ff390653822af6cc4 (Sep 18). INFO: trying to register non-static key. the code is fine but needs lockdep annotation. turning off the locking correctness validator. CPU: 0 PID: 24 Comm: kwor

Re: [kernel-hardening] Re: [PATCH v3 03/31] usercopy: Mark kmalloc caches as usercopy caches

2017-09-21 Thread Kees Cook
On Thu, Sep 21, 2017 at 8:27 AM, Christopher Lameter wrote: > On Wed, 20 Sep 2017, Kees Cook wrote: > >> --- a/mm/slab.c >> +++ b/mm/slab.c >> @@ -1291,7 +1291,8 @@ void __init kmem_cache_init(void) >>*/ >> kmalloc_caches[INDEX_NODE] = create_kmalloc_cache( >>

Re: [PATCH] net: phy: Fix truncation of large IRQ numbers in phy_attached_print()

2017-09-21 Thread Andrew Lunn
On Thu, Sep 21, 2017 at 01:27:02PM +0200, Geert Uytterhoeven wrote: > Given NR_IRQS is 2048 on sparc64, and even 32784 on alpha, 3 digits is > not enough to represent interrupt numbers on all architectures. Hence > PHY interrupt numbers may be truncated during printing. > > Increase the buffer si

[GIT PULL] Kbuild fixes for v4.14-rc2

2017-09-21 Thread Masahiro Yamada
Hi Linus, Here are some early Kbuild fixes. The in-kernel firmware was removed during the previous merge window. Since then, some bug reports of broken rpm building are flying in ML. We need to fix it now. The following changes since commit 2bd6bf03f4c1c59381d62c61d03f6cc3fe71f66e: Linux 4.1

Re: [PATCH 2/2] uio: add default compatible string to uio_pdrv_genirq

2017-09-21 Thread Chris Packham
On 21/09/17 19:50, Andy Shevchenko wrote: > On Thu, Sep 21, 2017 at 3:53 AM, Chris Packham > wrote: > > >> -module_param_string(of_id, uio_of_genirq_match[0].compatible, 128, 0); >> +module_param_string(of_id, uio_of_genirq_match[1].compatible, 128, ); > > 0 -> looks like bogus checkpa

Re: Query regarding synchronize_sched_expedited and resched_cpu

2017-09-21 Thread Steven Rostedt
On Thu, 21 Sep 2017 15:55:46 +0200 Peter Zijlstra wrote: > On Mon, Sep 18, 2017 at 11:11:05AM -0400, Steven Rostedt wrote: > > On Sun, 17 Sep 2017 11:37:06 +0530 > > Neeraj Upadhyay wrote: > > > > > Hi Paul, how about replacing raw_spin_trylock_irqsave with > > > raw_spin_lock_irqsave in resc

Re: [RFC PATCH v2 1/2] membarrier: Provide register expedited private command

2017-09-21 Thread Mathieu Desnoyers
- On Sep 21, 2017, at 11:27 AM, Paul E. McKenney paul...@linux.vnet.ibm.com wrote: > On Thu, Sep 21, 2017 at 03:11:43PM +, Mathieu Desnoyers wrote: >> - On Sep 21, 2017, at 8:13 AM, Peter Zijlstra pet...@infradead.org wrote: >> >> > On Mon, Sep 18, 2017 at 06:36:47PM -0400, Mathieu D

[PATCH RT] locking/rtmutex: don't drop the wait_lock twice

2017-09-21 Thread Sebastian Andrzej Siewior
Since the futex rework, __rt_mutex_start_proxy_lock() does no longer acquire the wait_lock so it must not drop it. Otherwise the lock is not only unlocked twice but also the preemption counter is underflown. Cc: rt-sta...@vger.kernel.org Reported-by: Gusenleitner Klaus Signed-off-by: Sebastian An

[PATCH v2 2/2] uio: add default compatible string to uio_pdrv_genirq

2017-09-21 Thread Chris Packham
Add a default compatible string "linux,uio-pdrv-genirq" to uio_pdrv_genirq to make it usable without supplying a module parameter. The module parameter is still supported in addition to the default. Signed-off-by: Chris Packham --- drivers/uio/uio_pdrv_genirq.c | 3 ++- 1 file changed, 2 inserti

[PATCH v2 0/2] using uio_pdrv_genirq without module param

2017-09-21 Thread Chris Packham
I found myself about to add a driver that was a sub-optimal clone of uio_pdrv_genirq the only difference was that I didn't want to modify the args passed to the kernel by my bootloader. If uio_pdrv_genirq had a default of_match entry I could simply use that. This series attempts to implement this.

[PATCH v2 1/2] uio: dt-bindings: document binding for uio-pdrv-genirq

2017-09-21 Thread Chris Packham
Document the device tree bindings for the uio-prv-genirq driver. Provide some examples on how it can be used. Signed-off-by: Chris Packham --- .../bindings/uio/linux,uio-pdrv-genirq.txt | 28 ++ 1 file changed, 28 insertions(+) create mode 100644 Documentation/devic

[PATCH v5 6/6] iommu/iova: Simplify cached node logic

2017-09-21 Thread Robin Murphy
The logic of __get_cached_rbnode() is a little obtuse, but then __get_prev_node_of_cached_rbnode_or_last_node_and_update_limit_pfn() wouldn't exactly roll off the tongue... Now that we have the invariant that there is always a valid node to start searching downwards from, everything gets a bit eas

[PATCH v5 0/6] Optimise 64-bit IOVA allocations

2017-09-21 Thread Robin Murphy
v4: https://www.mail-archive.com/linux-kernel@vger.kernel.org/msg1493704.html Right, this is hopefully the last version - I've put things back in a sensible order with the new additions at the end, so if they prove contentious the first 4 previously-tested patches can still get their time in -next

[PATCH v5 1/6] iommu/iova: Optimise rbtree searching

2017-09-21 Thread Robin Murphy
From: Zhen Lei Checking the IOVA bounds separately before deciding which direction to continue the search (if necessary) results in redundantly comparing both pfns twice each. GCC can already determine that the final comparison op is redundant and optimise it down to 3 in total, but we can go one

<    1   2   3   4   5   6   7   8   9   >