Re: question about cpusets vs sched_setaffinity()

2015-12-11 Thread Jason Baron
On 12/10/2015 04:30 PM, Chris Friesen wrote: > Hi, > > I've got a question about the interaction between cpusets and > sched_setaffinity(). > > If I put a task into a cpuset and then call sched_setaffinity() on it, > it will be affined to the intersection of the two sets of cpus. (Those > spec

Re: [Patch v5 1/2] media: v4l: ti-vpe: Add CAL v4l2 camera capture driver

2015-12-11 Thread Benoit Parrot
Mauro Carvalho Chehab wrote on Thu [2015-Dec-03 11:19:22 -0200]: > Em Wed, 18 Nov 2015 14:47:11 -0600 > Benoit Parrot escreveu: > > > The Camera Adaptation Layer (CAL) is a block which consists of a dual > > port CSI2/MIPI camera capture engine. > > Port #0 can handle CSI2 camera connected to u

RE: [PATCHV2 3/3] x86, ras: Add mcsafe_memcpy() function to recover from machine checks

2015-12-11 Thread Luck, Tony
> I'm missing something, though. The normal fixup_exception path > doesn't touch rax at all. The memory_failure path does. But couldn't > you distinguish them by just pointing the exception handlers at > different landing pads? Perhaps I'm just trying to take a short cut to avoid writing some c

Re: [PATCH] devpts: Sensible /dev/ptmx & force newinstance

2015-12-11 Thread H. Peter Anvin
On 12/11/15 13:48, Andy Lutomirski wrote: > On Fri, Dec 11, 2015 at 1:11 PM, Eric W. Biederman > wrote: >> Al Viro writes: >> >>> On Fri, Dec 11, 2015 at 01:40:40PM -0600, Eric W. Biederman wrote: >>> +inode = path.dentry->d_inode; +filp->f_path = path; +filp->f_inode =

Re: [PATCH] devpts: Sensible /dev/ptmx & force newinstance

2015-12-11 Thread H. Peter Anvin
On 12/11/15 14:12, Andy Lutomirski wrote: >> >> For the newinstance case st_dev should match between the master and the >> slave. Unfortunately this is not the case for a legacy ptmx, as a >> stat() on the master descriptor still returns the st_dev, st_rdev, and >> st_ino for the ptmx device node.

Re: [PATCHV2 3/3] x86, ras: Add mcsafe_memcpy() function to recover from machine checks

2015-12-11 Thread Dan Williams
On Fri, Dec 11, 2015 at 2:17 PM, Luck, Tony wrote: >> Also, would it be more straightforward if the mcexception landing pad >> looked up the va -> pa mapping by itself? Or is that somehow not >> reliable? > > If we did get all the above right, then we could have > target use virt_to_phys() to con

[PATCH] drivers: memory: check for missing sections in test_pages_in_a_zone

2015-12-11 Thread Andrew Banman
test_pages_in_a_zone does not account for the possibility of missing sections in the given pfn range. pfn_valid_within always returns 1 when CONFIG_HOLES_IN_ZONE is not set, allowing invalid pfns from missing sections to pass the test, leading to a kernel oops. Wrap an additional pfn loop with PAG

Re: [PATCH] iwlwifi: mvm: protect RCU dereference in iwl_mvm_get_key_sta_id

2015-12-11 Thread Laura Abbott
On 12/11/2015 12:13 AM, Johannes Berg wrote: From: Johannes Berg Properly protect the RCU dereference in iwl_mvm_get_key_sta_id() when coming from iwl_mvm_update_tkip_key() which cannot hold the mvm->mutex by moving the call into the RCU critical section. Modify the check to use rcu_dereference

Re: [RFC][PATCH] module: Limit line length of module prints

2015-12-11 Thread Laura Abbott
On 12/11/2015 01:36 AM, Geyslan G. Bem wrote: Hello, What do you think of get the strlen of name and module_flags before printing them? So the check against MAX CHARS would be more precise and uniform. Or the limit is not strictly necessary? It's not really necessary. If it goes over a littl

Re: [PATCH] proc: fix -ESRCH error when writing to /proc/$pid/coredump_filter

2015-12-11 Thread Kees Cook
On Thu, Dec 10, 2015 at 5:04 PM, Colin King wrote: > From: Colin Ian King > > echo 0 > /proc/self/coredump_filter > bash: echo: write error: No such process > > Writing to /proc/$pid/coredump_filter always returns -ESRCH because > commit 774636e19ed51 ("proc: convert to kstrto*()/kstrto*_from_use

Re: [PATCH linux-next (v3) 1/3] MIPS: bcm963xx: Add Broadcom BCM963xx board nvram data structure

2015-12-11 Thread Simon Arlott
On 11/12/15 22:02, Jonas Gorski wrote: > Hi, > > On Fri, Dec 11, 2015 at 10:54 PM, Simon Arlott wrote: >> Broadcom BCM963xx boards have multiple nvram variants across different >> SoCs with additional checksum fields added whenever the size of the >> nvram was extended. >> >> Add this structure a

Re: [PATCH] devpts: Sensible /dev/ptmx & force newinstance

2015-12-11 Thread Andy Lutomirski
On Fri, Dec 11, 2015 at 2:18 PM, H. Peter Anvin wrote: > On 12/11/15 14:12, Andy Lutomirski wrote: >>> >>> For the newinstance case st_dev should match between the master and the >>> slave. Unfortunately this is not the case for a legacy ptmx, as a >>> stat() on the master descriptor still return

Re: [PATCH] mmc: sdhci-acpi: set non-removable in ACPI table

2015-12-11 Thread Rafael J. Wysocki
On Friday, December 11, 2015 10:17:18 AM Adrian Hunter wrote: > On 10/12/15 22:57, Philip Elcan wrote: > > > > On 12/07/2015 03:30 AM, Adrian Hunter wrote: > >> On 04/12/15 17:40, Philip Elcan wrote: > >>> On 12/03/2015 09:14 AM, Adrian Hunter wrote: > On 03/12/15 15:48, Philip Elcan wrote: >

Re: [RFC][PATCH] module: Limit line length of module prints

2015-12-11 Thread Laura Abbott
On 12/11/2015 01:39 AM, Rusty Russell wrote: Laura Abbott writes: print_modules currently uses pr_cont to print all module information. This has the side effect of printing lots of modules on one very long line. This makes copy/pasting oopses more effort if manual wrapping is required. Place a

Re: [PATCH perf/core 00/22] perf refcnt debugger API and fixes

2015-12-11 Thread Arnaldo Carvalho de Melo
Em Wed, Dec 09, 2015 at 10:41:38AM -0300, Arnaldo Carvalho de Melo escreveu: > Em Wed, Dec 09, 2015 at 11:10:48AM +0900, Masami Hiramatsu escreveu: > > General refcnt miscodings > > = > > > > BTW, while applying this change, I've found that there are refcnt > > coding m

Re: [PATCHV2 3/3] x86, ras: Add mcsafe_memcpy() function to recover from machine checks

2015-12-11 Thread Andy Lutomirski
On Fri, Dec 11, 2015 at 2:20 PM, Dan Williams wrote: > On Fri, Dec 11, 2015 at 2:17 PM, Luck, Tony wrote: >>> Also, would it be more straightforward if the mcexception landing pad >>> looked up the va -> pa mapping by itself? Or is that somehow not >>> reliable? >> >> If we did get all the above

Re: new warning on sysrq kernel crash trigger

2015-12-11 Thread Paul E. McKenney
On Fri, Dec 11, 2015 at 05:10:43PM -0500, Rik van Riel wrote: > On 12/11/2015 03:44 PM, Ani Sinha wrote: > > > > > > On Thu, 10 Dec 2015, Paul E. McKenney wrote: > > > >> On Thu, Dec 10, 2015 at 03:57:09PM -0800, Ani Sinha wrote: > >>> Hi guys > >>> > >>> I am noticing a new warning in linux 3.1

Re: [PATCH 1/6] mm: Add a vm_special_mapping .fault method

2015-12-11 Thread Andrew Morton
On Thu, 10 Dec 2015 19:21:42 -0800 Andy Lutomirski wrote: > From: Andy Lutomirski > > Requiring special mappings to give a list of struct pages is > inflexible: it prevents sane use of IO memory in a special mapping, > it's inefficient (it requires arch code to initialize a list of > struct pag

Re: [PATCH] devpts: Sensible /dev/ptmx & force newinstance

2015-12-11 Thread H. Peter Anvin
On 12/11/15 14:24, Andy Lutomirski wrote: > > To do the whole shebang at once: > > ioctl(ptmx_fd, TIOCWHATEVER, fd_to_devpts_mount); > > returns the slave number if fd_to_devpts_mount points to the right > place or an error if not. > > ptsname(fd) logically does: > > fd_to_devpts_mount = open(

Re: [PATCH 2/6] mm: Add vm_insert_pfn_prot

2015-12-11 Thread Andrew Morton
On Thu, 10 Dec 2015 19:21:43 -0800 Andy Lutomirski wrote: > The x86 vvar mapping contains pages with differing cacheability > flags. This is currently only supported using (io_)remap_pfn_range, > but those functions can't be used inside page faults. Foggy. What does "support" mean here? > Add

Re: FW: Commit 81a43adae3b9 (locking/mutex: Use acquire/release semantics) causing failures on arm64 (ThunderX)

2015-12-11 Thread Paul E. McKenney
On Fri, Dec 11, 2015 at 02:48:03PM +0100, Peter Zijlstra wrote: > On Fri, Dec 11, 2015 at 01:33:14PM +, Will Deacon wrote: > > On Fri, Dec 11, 2015 at 01:26:47PM +0100, Peter Zijlstra wrote: > > > > While we're there, the acquire in osq_wait_next() seems somewhat ill > > > documented too. > >

RE: [PATCHV2 3/3] x86, ras: Add mcsafe_memcpy() function to recover from machine checks

2015-12-11 Thread Luck, Tony
> Also, are there really PCOMMIT-capable CPUs that still forcibly > broadcast MCE? If, so, that's unfortunate. PCOMMIT and LMCE arrive together ... though BIOS is in the decision path to enable LMCE, so it is possible that some systems could still broadcast if the BIOS writer decides to not allow

Re: [PATCHV2 3/3] x86, ras: Add mcsafe_memcpy() function to recover from machine checks

2015-12-11 Thread Andy Lutomirski
On Fri, Dec 11, 2015 at 2:35 PM, Luck, Tony wrote: >> Also, are there really PCOMMIT-capable CPUs that still forcibly >> broadcast MCE? If, so, that's unfortunate. > > PCOMMIT and LMCE arrive together ... though BIOS is in the decision > path to enable LMCE, so it is possible that some systems co

Re: [PATCH v0 3/5] perf: Introduce instruction trace filtering

2015-12-11 Thread Peter Zijlstra
On Fri, Dec 11, 2015 at 07:13:10PM +0200, Alexander Shishkin wrote: > Peter Zijlstra writes: > > On second thought; we should not inherit the filters at all. > > > > We should always use event->parent (if exists) for filters. Otherwise > > inherited events will get different filters if you change

[PATCH 2/2] powercap/rapl: reduce ipi calls

2015-12-11 Thread Jacob Pan
Reduce remote CPU calls for MSR access by combining read modify write into one function. Suggested-by: Peter Zijlstra Signed-off-by: Jacob Pan --- drivers/powercap/intel_rapl.c | 71 ++- 1 file changed, 37 insertions(+), 34 deletions(-) diff --git a/driv

Re: [PATCH v0 3/5] perf: Introduce instruction trace filtering

2015-12-11 Thread Peter Zijlstra
On Fri, Dec 11, 2015 at 11:11:48AM -0700, Mathieu Poirier wrote: > On 11 December 2015 at 06:36, Alexander Shishkin > > /** > > + * Instruction trace (ITRACE) filter > > + */ > > +struct perf_itrace_filter { > > + struct list_headentry; > > + struct rcu_head rcu_head;

[PATCH 1/2] x86/msr: add on cpu read/modify/write function

2015-12-11 Thread Jacob Pan
Remote CPU read/modify/write is often needed but currently without a lib call. This patch adds an API to perform on CPU safe read/modify/write so that callers don't have to invent such function. Based on initial code from: Peter Zijlstra Suggested-by: Srinivas Pandruvada Signed-off-by: Jacob Pa

[PATCH 0/2] combine remote cpu msr access

2015-12-11 Thread Jacob Pan
IPI calls to access MSRs on remote CPUs are expensive. So add a lib call for read/modify/write access and use it in Intel RAPL driver. Jacob Pan (2): x86/msr: add on cpu read/modify/write function powercap/rapl: reduce ipi calls arch/x86/include/asm/msr.h| 24 +++ arch/x86/li

Re: [PATCH] devpts: Sensible /dev/ptmx & force newinstance

2015-12-11 Thread Eric W. Biederman
Andy Lutomirski writes: > On Fri, Dec 11, 2015 at 2:07 PM, H. Peter Anvin wrote: >> On 12/11/15 13:48, Andy Lutomirski wrote: >>> On Fri, Dec 11, 2015 at 1:11 PM, Eric W. Biederman >>> wrote: Al Viro writes: > On Fri, Dec 11, 2015 at 01:40:40PM -0600, Eric W. Biederman wrote: >>>

Re: [PATCH 2/6] mm: Add vm_insert_pfn_prot

2015-12-11 Thread Andy Lutomirski
On Fri, Dec 11, 2015 at 2:33 PM, Andrew Morton wrote: > On Thu, 10 Dec 2015 19:21:43 -0800 Andy Lutomirski wrote: > >> The x86 vvar mapping contains pages with differing cacheability >> flags. This is currently only supported using (io_)remap_pfn_range, >> but those functions can't be used insid

RE: [PATCHV2 3/3] x86, ras: Add mcsafe_memcpy() function to recover from machine checks

2015-12-11 Thread Luck, Tony
>> But a machine check safe copy_from_user() would be useful >> current generation cpus that broadcast all the time. > > Fair enough. Thanks for spending the time to look at this. Coaxing me to re-write the tail of do_machine_check() has made that code much better. Too many years of one patch on

[PATCH v2] spi: dts: sun4i: Add support for inter-word wait cycles using the SPI Wait Clock Register

2015-12-11 Thread Marcus Weseloh
Adds support and binding documentation for a new slave device property "sun4i,spi-word-wait-ns" that allows to set a hardware based delay between the transmission of words using the SPI Wait Clock Register. The SPI hardware needs 3 clock cycles to set up the delay, which makes the minimum non-zero

Re: [PATCH] devpts: Sensible /dev/ptmx & force newinstance

2015-12-11 Thread Andy Lutomirski
On Fri, Dec 11, 2015 at 2:35 PM, Eric W. Biederman wrote: > Andy Lutomirski writes: > >> On Fri, Dec 11, 2015 at 2:07 PM, H. Peter Anvin wrote: >>> On 12/11/15 13:48, Andy Lutomirski wrote: On Fri, Dec 11, 2015 at 1:11 PM, Eric W. Biederman wrote: > Al Viro writes: > >> O

Re: [PATCH] drivers: memory: check for missing sections in test_pages_in_a_zone

2015-12-11 Thread andrew banman
On Fri, Dec 11, 2015 at 04:21:03PM -0600, Andrew Banman wrote: > test_pages_in_a_zone does not account for the possibility of missing > sections in the given pfn range. pfn_valid_within always returns 1 when > CONFIG_HOLES_IN_ZONE is not set, allowing invalid pfns from missing > sections to pass th

Re: [PATCHV2 3/3] x86, ras: Add mcsafe_memcpy() function to recover from machine checks

2015-12-11 Thread Andy Lutomirski
On Fri, Dec 11, 2015 at 2:45 PM, Luck, Tony wrote: >>> But a machine check safe copy_from_user() would be useful >>> current generation cpus that broadcast all the time. >> >> Fair enough. > > Thanks for spending the time to look at this. Coaxing me to re-write the > tail of do_machine_check() ha

[PATCH v5 0/3] [media] Fix race between graph enumeration and entities registration

2015-12-11 Thread Javier Martinez Canillas
Hello, This series fixes the issue of media device nodes being registered before all the media entities and pads links are created so if user-space tries to enumerate the graph too early, it may get a partial graph enumeration since everything may not been registered yet. The solution (suggested

[PATCH v5 3/3] [media] media-device: set topology version 0 at media registration

2015-12-11 Thread Javier Martinez Canillas
The G_TOPOLOGY ioctl is used to get a graph topology and since in the future a graph can be dynamically updated, there is a way to know the topology version so userspace can be aware that the graph has changed. The version 0 is reserved to indicate that the graph is static (i.e no graphs updates s

Re: [PATCH] devpts: Sensible /dev/ptmx & force newinstance

2015-12-11 Thread Jann Horn
On Fri, Dec 11, 2015 at 02:52:01PM -0800, Andy Lutomirski wrote: > On Fri, Dec 11, 2015 at 2:35 PM, Eric W. Biederman > wrote: > > Andy Lutomirski writes: > > > >> On Fri, Dec 11, 2015 at 2:07 PM, H. Peter Anvin wrote: > >>> On 12/11/15 13:48, Andy Lutomirski wrote: > On Fri, Dec 11, 2015 a

[PATCH v5 2/3] [media] media-device: split media initialization and registration

2015-12-11 Thread Javier Martinez Canillas
The media device node is registered and so made visible to user-space before entities are registered and links created which means that the media graph obtained by user-space could be only partially enumerated if that happens too early before all the graph has been created. To avoid this race cond

Re: [PATCH v3 4/4] printk/nmi: Increase the size of NMI buffer and make it configurable

2015-12-11 Thread Andrew Morton
On Fri, 11 Dec 2015 13:41:59 +0100 Petr Mladek wrote: > On Fri 2015-12-11 12:10:02, Geert Uytterhoeven wrote: > > On Wed, Dec 9, 2015 at 2:21 PM, Petr Mladek wrote: > > > --- a/init/Kconfig > > > +++ b/init/Kconfig > > > @@ -866,6 +866,28 @@ config LOG_CPU_MAX_BUF_SHIFT > > >

[PATCH v5 1/3] [media] media-device: check before unregister if mdev was registered

2015-12-11 Thread Javier Martinez Canillas
Most media functions that unregister, check if the corresponding register function succeed before. So these functions can safely be called even if a registration was never made or the component as already been unregistered. Add the same check to media_device_unregister() function for consistency.

Re: [PATCH] devpts: Sensible /dev/ptmx & force newinstance

2015-12-11 Thread H. Peter Anvin
On December 11, 2015 2:35:16 PM PST, ebied...@xmission.com wrote: >Andy Lutomirski writes: > >> On Fri, Dec 11, 2015 at 2:07 PM, H. Peter Anvin >wrote: >>> On 12/11/15 13:48, Andy Lutomirski wrote: On Fri, Dec 11, 2015 at 1:11 PM, Eric W. Biederman wrote: > Al Viro writes: > >

Re: [PATCH] devpts: Sensible /dev/ptmx & force newinstance

2015-12-11 Thread Andy Lutomirski
On Fri, Dec 11, 2015 at 2:58 PM, Jann Horn wrote: > On Fri, Dec 11, 2015 at 02:52:01PM -0800, Andy Lutomirski wrote: >> On Fri, Dec 11, 2015 at 2:35 PM, Eric W. Biederman >> wrote: >> > Andy Lutomirski writes: >> > >> >> On Fri, Dec 11, 2015 at 2:07 PM, H. Peter Anvin wrote: >> >>> On 12/11/15

[PATCH] perf/x86/amd: Adding support for new IOMMU performance event

2015-12-11 Thread Suravee Suthikulpanit
From: Suravee Suthikulpanit This patch adds new IOMMU performance event based on the information in table 74 of the AMD I/O Virtualization Technology (IOMMU) Specification (Document Id: 4882, Rev 2.62, Feb 2015) Link: http://support.amd.com/TechDocs/48882_IOMMU.pdf Signed-off-by: Suravee Suthiku

[PATCH v2 6/6] Nuke another unsigned >= 0 assert

2015-12-11 Thread Valdis Kletnieks
Clean up another case of the compiler remininding the programmer they are an idiot: drivers/staging/lustre/lustre/ptlrpc/sec_bulk.c:308:34: warning: comparison of unsigned expression >= 0 is always true [-Wtype-limits] LASSERT(page_pools.epp_waitqlen >= 0); Just lose the assert, and save a pag

[PATCH v2 5/6] Nuke an unsigned >= 0 assert

2015-12-11 Thread Valdis Kletnieks
Writing asserts for almost-never-can-happen things can be valuable. Writing an assert that tests that an "unsigned int" hasn't gone negative isn't. And it generates an *ugly* message: drivers/staging/lustre/lustre/llite/rw.c:763:20: warning: comparison of unsigned expression >= 0 is always true

[PATCH v2 1/6] staging/lustre: Silence warning about 'inline'

2015-12-11 Thread Valdis Kletnieks
Low-hanging fruit first: CC [M] drivers/staging/lustre/lustre/fid/fid_request.o In file included from drivers/staging/lustre/lustre/fid/../include/lustre_net.h:66:0, from drivers/staging/lustre/lustre/fid/../include/lustre_lib.h:64, from drivers/staging/lustr

[PATCH] iommu/amd: Assign default IOMMU when there is only one IOMMU

2015-12-11 Thread Suravee Suthikulpanit
Current driver makes assumption that device with devid zero is always included in the range of devices to be managed by IOMMU. However, certain FW does not include devid zero in IVRS table. This has caused IOMMU perf driver to fail to initialize. This patch implements a workaround for this case by

[PATCH v2 3/6] Clean up another C warnining: set but not used

2015-12-11 Thread Valdis Kletnieks
drivers/staging/lustre/lustre/fid/../include/lustre_cfg.h: In function 'lustre_cfg_free': drivers/staging/lustre/lustre/fid/../include/lustre_cfg.h:253:6: warning: variable 'len' set but not used [-Wunused-but-set-variable] int len; Yep, we're just gonna call kfree, no need to calculate len. B

[PATCH v2 4/6] Fix another C compiler whine: set but not used

2015-12-11 Thread Valdis Kletnieks
CC [M] drivers/staging/lustre/lustre/libcfs/module.o drivers/staging/lustre/lustre/libcfs/module.c: In function 'lustre_insert_debugfs': drivers/staging/lustre/lustre/libcfs/module.c:670:17: warning: variable 'entry' set but not used [-Wunused-but-set-variable] struct dentry *entry;

[PATCH v2 0/6] Patch series to make lustre safe(r) for W=1 compiles

2015-12-11 Thread Valdis Kletnieks
Start of a batch series to clean up the Lustre tree. Other people have done some sparse and checkpatch cleanups, but I found a bunch of stuff building with W=1. There's probably more, but this was the really low-hanging obvious fruit. Valdis Kletnieks (6): staging/lustre: Silence warning about '

[PATCH v2 2/6] Fix set-but-unused whinge.

2015-12-11 Thread Valdis Kletnieks
drivers/staging/lustre/lustre/fid/lproc_fid.c: In function 'ldebugfs_fid_write_common': drivers/staging/lustre/lustre/fid/lproc_fid.c:67:6: warning: variable 'rc' set but not used [-Wunused-but-set-variable] int rc; We fix it by *using* the return code to help bulletproof it. It says it's tes

Re: [PATCH] devpts: Sensible /dev/ptmx & force newinstance

2015-12-11 Thread H. Peter Anvin
On December 11, 2015 3:00:49 PM PST, Andy Lutomirski wrote: >On Fri, Dec 11, 2015 at 2:58 PM, Jann Horn wrote: >> On Fri, Dec 11, 2015 at 02:52:01PM -0800, Andy Lutomirski wrote: >>> On Fri, Dec 11, 2015 at 2:35 PM, Eric W. Biederman >>> wrote: >>> > Andy Lutomirski writes: >>> > >>> >> On Fri

Re: [PATCH] arm: dts: rockchip: Fix typo in rk32288 sdmmc card detect pin name

2015-12-11 Thread Heiko Stübner
Hi Matthias, Am Freitag, 11. Dezember 2015, 15:45:58 schrieb Matthias Brugger: > The card detect pin is currently called sdmcc-cd. > This patch fixes the typo and renames the pin to sdmmc-cd. > > Signed-off-by: Matthias Brugger applied to my dts32 branch for 4.5, after fixing the rk32288 in the

Re: [PATCH linux-next (v3) 3/3] mtd: part: Add BCM962368 CFE partitioning support

2015-12-11 Thread Jonas Gorski
Hi, On Fri, Dec 11, 2015 at 11:02 PM, Simon Arlott wrote: > Add partitioning support for BCM963268 boards with CFE bootloaders. > The following partitions are defined: > "boot": CFE and nvram data > "rootfs": Currently selected rootfs > "data": Configuration data

[RFC] selftests: report proper exit statuses

2015-12-11 Thread Brian Norris
There are several places where we don't report proper exit statuses, and this can have consequences -- for instance, the gen_kselftest_tar.sh script might try to produce a tarball for you, even if the 'make' or 'make install' steps didn't complete properly. This is only an RFC (and really, it's mo

Re: [PATCHSET v2] ->follow_link() without dropping from RCU mode

2015-12-11 Thread Al Viro
On Fri, Dec 11, 2015 at 08:49:24AM +0100, Rasmus Villemoes wrote: > union delayed_call_fn { > void (*fn)(void *); > void (*kfree_like)(const void *); > } __attribute__((__transparent_union__)); > > void > set_delayed_call(struct delayed_call *call, union delayed_call_fn u, void > *arg

Re: [PATCH] devpts: Sensible /dev/ptmx & force newinstance

2015-12-11 Thread Andy Lutomirski
On Fri, Dec 11, 2015 at 3:07 PM, H. Peter Anvin wrote: > On December 11, 2015 3:00:49 PM PST, Andy Lutomirski > wrote: >>On Fri, Dec 11, 2015 at 2:58 PM, Jann Horn wrote: >>> On Fri, Dec 11, 2015 at 02:52:01PM -0800, Andy Lutomirski wrote: On Fri, Dec 11, 2015 at 2:35 PM, Eric W. Biederman

[PATCH v2] extcon: add Maxim MAX3355 driver

2015-12-11 Thread Sergei Shtylyov
Maxim Integrated MAX3355E chip integrates a charge pump and comparators to enable a system with an integrated USB OTG dual-role transceiver to function as an USB OTG dual-role device. In addition to sensing/controlling Vbus, the chip also passes thru the ID signal from the USB OTG connector

Re: [PATCH v3 4/4] printk/nmi: Increase the size of NMI buffer and make it configurable

2015-12-11 Thread Russell King - ARM Linux
On Fri, Dec 11, 2015 at 02:57:25PM -0800, Andrew Morton wrote: > This is a bit messy. NEED_PRINTK_NMI is an added-on hack for one > particular arm variant. From the changelog: > >"One exception is arm where the deferred printing is used for > printing backtraces even without NMI. For th

Re: mtd, nand, omap2: parse cmdline partition fail

2015-12-11 Thread Brian Norris
On Thu, Dec 10, 2015 at 08:13:15AM +0100, Frans Klaver wrote: > On Thu, Dec 10, 2015 at 12:19 AM, Brian Norris > wrote: > > On Fri, Dec 04, 2015 at 09:42:06AM +0100, Heiko Schocher wrote: > >> >>But wondering, if there are two or more identical nand chips in the > >> >>system, they will have the s

Re: [PATCH v3 4/4] printk/nmi: Increase the size of NMI buffer and make it configurable

2015-12-11 Thread Jiri Kosina
On Fri, 11 Dec 2015, Russell King - ARM Linux wrote: > I'm personally happy with the existing code, and I've been wondering why > there's this effort to apply further cleanups - to me, the changelogs > don't seem to make that much sense, unless we want to start using > printk() extensively in NMI

Re: [PATCH 0/2] ARM: dts: uniphier: clean up DTSIs by factoring the common parts out

2015-12-11 Thread Arnd Bergmann
On Thursday 03 December 2015 15:33:55 Masahiro Yamada wrote: > Masahiro Yamada (2): > ARM: dts: uniphier: change IRQ number of UART3 of PH1-Pro4 SoC > ARM: dts: uniphier: factor out common nodes to uniphier-common32.dtsi > Applied both to next/dt, thanks! Arnd -- To unsubscribe from

Re: [PATCH v2] PCI/AER: enable SERR# forwarding for bridges and switches

2015-12-11 Thread Sinan Kaya
On 12/10/2015 5:37 PM, Bjorn Helgaas wrote: > On Thu, Dec 10, 2015 at 03:28:35PM -0500, Sinan Kaya wrote: >> Hi Bjorn, >> >> On 12/4/2015 4:06 PM, Bjorn Helgaas wrote: >>> I'm sitting on this for the moment because if you have _HPP, it seems >>> like that should be enough to get SERR# forwarding tu

Re: [PATCH v3 4/4] printk/nmi: Increase the size of NMI buffer and make it configurable

2015-12-11 Thread Andrew Morton
On Fri, 11 Dec 2015 23:21:13 + Russell King - ARM Linux wrote: > On Fri, Dec 11, 2015 at 02:57:25PM -0800, Andrew Morton wrote: > > This is a bit messy. NEED_PRINTK_NMI is an added-on hack for one > > particular arm variant. From the changelog: > > > >"One exception is arm where the d

Re: [PATCH linux-next (v3) 1/3] MIPS: bcm963xx: Add Broadcom BCM963xx board nvram data structure

2015-12-11 Thread Jonas Gorski
On Fri, Dec 11, 2015 at 11:24 PM, Simon Arlott wrote: > On 11/12/15 22:02, Jonas Gorski wrote: >> Hi, >> >> On Fri, Dec 11, 2015 at 10:54 PM, Simon Arlott wrote: >>> Broadcom BCM963xx boards have multiple nvram variants across different >>> SoCs with additional checksum fields added whenever the

Re: [PATCH] devpts: Sensible /dev/ptmx & force newinstance

2015-12-11 Thread H. Peter Anvin
On December 11, 2015 3:16:48 PM PST, Andy Lutomirski wrote: >On Fri, Dec 11, 2015 at 3:07 PM, H. Peter Anvin wrote: >> On December 11, 2015 3:00:49 PM PST, Andy Lutomirski > wrote: >>>On Fri, Dec 11, 2015 at 2:58 PM, Jann Horn wrote: On Fri, Dec 11, 2015 at 02:52:01PM -0800, Andy Lutomirsk

[PATCH] mm/zswap: change incorrect strncmp use to strcmp

2015-12-11 Thread Dan Streetman
Change the use of strncmp in zswap_pool_find_get() to strcmp. The use of strncmp is no longer correct, now that zswap_zpool_type is not an array; sizeof() will return the size of a pointer, which isn't the right length to compare. We don't need to use strncmp anyway, because the existing params a

Re: question about cpusets vs sched_setaffinity()

2015-12-11 Thread Chris Friesen
On 12/11/2015 04:15 PM, Jason Baron wrote: On 12/10/2015 04:30 PM, Chris Friesen wrote: If I put a task into a cpuset and then call sched_setaffinity() on it, it will be affined to the intersection of the two sets of cpus. (Those specified on the set, and those specified in the syscall.) How

Re: [PATCH v5 1/3] [media] media-device: check before unregister if mdev was registered

2015-12-11 Thread Shuah Khan
On 12/11/2015 03:57 PM, Javier Martinez Canillas wrote: > Most media functions that unregister, check if the corresponding register > function succeed before. So these functions can safely be called even if a > registration was never made or the component as already been unregistered. > > Add the

Re: new warning on sysrq kernel crash trigger

2015-12-11 Thread Ani Sinha
On Fri, Dec 11, 2015 at 2:27 PM, Paul E. McKenney wrote: > On Fri, Dec 11, 2015 at 05:10:43PM -0500, Rik van Riel wrote: >> On 12/11/2015 03:44 PM, Ani Sinha wrote: >> > >> > >> > On Thu, 10 Dec 2015, Paul E. McKenney wrote: >> > >> >> On Thu, Dec 10, 2015 at 03:57:09PM -0800, Ani Sinha wrote: >>

Re: [PATCH] mm/zswap: change incorrect strncmp use to strcmp

2015-12-11 Thread Andrew Morton
On Fri, 11 Dec 2015 18:33:11 -0500 Dan Streetman wrote: > Change the use of strncmp in zswap_pool_find_get() to strcmp. > > The use of strncmp is no longer correct, now that zswap_zpool_type is > not an array; sizeof() will return the size of a pointer, which isn't > the right length to compare.

Re: [PATCH v2 3/4] ARM: dts: rockchip: add core rk3228 dtsi

2015-12-11 Thread Heiko Stübner
Am Freitag, 11. Dezember 2015, 12:25:32 schrieb Heiko Stübner: > Hi Jeffy, > > Am Freitag, 11. Dezember 2015, 09:30:51 schrieb Jeffy Chen: > > Initial release for rk3228 shared dtsi. > > > > Signed-off-by: Jeffy Chen > > Booth dts look good now, just need to wait on Linus Walleij picking up the

[PATCH for-4.4] mtd: fix cmdlinepart parser, early naming for auto-filled MTD

2015-12-11 Thread Brian Norris
Commit 807f16d4db95 ("mtd: core: set some defaults when dev.parent is set") attempted to provide some default settings for MTDs that (a) assign the parent device and (b) don't provide their own name or owner However, this isn't a perfect drop-in replacement for the boilerplate found in some driv

Re: new warning on sysrq kernel crash trigger

2015-12-11 Thread Paul E. McKenney
On Fri, Dec 11, 2015 at 03:41:04PM -0800, Ani Sinha wrote: > On Fri, Dec 11, 2015 at 2:27 PM, Paul E. McKenney > wrote: > > On Fri, Dec 11, 2015 at 05:10:43PM -0500, Rik van Riel wrote: > >> On 12/11/2015 03:44 PM, Ani Sinha wrote: > >> > > >> > > >> > On Thu, 10 Dec 2015, Paul E. McKenney wrote:

Re: [linux-next] iwl_mvm_get_key_sta_id() suspicious RCU usage

2015-12-11 Thread Sergey Senozhatsky
On (12/11/15 14:49), Johannes Berg wrote: > On Fri, 2015-12-11 at 22:44 +0900, Sergey Senozhatsky wrote: > > > [ 6385.246300] drivers/net/wireless/intel/iwlwifi/mvm/sta.c:1226 > > suspicious rcu_dereference_protected() usage! > > > > Funny, Laura Abbott also reported this bug earlier today :) >

Re: new warning on sysrq kernel crash trigger

2015-12-11 Thread Ani Sinha
I backported your ee376dbdf277 ("rcu: Consolidate rcu_synchronize and wakeme_after_rcu()" & ec90a194ae2cb8b8e("rcu: Create a synchronize_rcu_mult()") and tested this on our 3.18 kernel running on our board. The sysrq kernel crash seems to have been fixed (behavior as per our old 3.4 kernel). I wi

[PATCH] perf, tools: Disable branch flags/cycles for --callgraph lbr

2015-12-11 Thread Andi Kleen
From: Andi Kleen [The kernel patch needed for this is in tip now (b16a5b52eb9 perf/x86: Add option to disable ...) So this user tools patch to make use of it should be merged now] Automatically disable collecting branch flags and cycles with --call-graph lbr. This allows avoiding a bunch of extr

Re: [linux-next] iwl_mvm_get_key_sta_id() suspicious RCU usage

2015-12-11 Thread Sergey Senozhatsky
On (12/12/15 09:05), Sergey Senozhatsky wrote: > On (12/11/15 14:49), Johannes Berg wrote: > > On Fri, 2015-12-11 at 22:44 +0900, Sergey Senozhatsky wrote: > > > > > [ 6385.246300] drivers/net/wireless/intel/iwlwifi/mvm/sta.c:1226 > > > suspicious rcu_dereference_protected() usage! > > > > > > >

Re: [PATCH 0/4] arm,arm64: uniphier: add a new driver, device tree updates

2015-12-11 Thread Arnd Bergmann
On Tuesday 24 November 2015 18:39:18 Masahiro Yamada wrote: > > Here is another series for UniPhier SoC family: > > - 1/4: add a new driver. The UniPhier System Bus is an external bus > where on-board devices are connected to the SoC. > (please check if the binding specification

Re: new warning on sysrq kernel crash trigger

2015-12-11 Thread Ani Sinha
On Fri, 11 Dec 2015, Paul E. McKenney wrote: > On Fri, Dec 11, 2015 at 05:10:43PM -0500, Rik van Riel wrote: > > On 12/11/2015 03:44 PM, Ani Sinha wrote: > > > > > > > > > On Thu, 10 Dec 2015, Paul E. McKenney wrote: > > > > > >> On Thu, Dec 10, 2015 at 03:57:09PM -0800, Ani Sinha wrote: > >

Re: [PATCH] locking/osq: fix ordering of node initialisation in osq_lock

2015-12-11 Thread David Daney
On 12/11/2015 09:46 AM, Will Deacon wrote: The Cavium guys reported a soft lockup on their arm64 machine, caused by c55a6ffa6285 ("locking/osq: Relax atomic semantics"): [ 68.909948] [] mutex_optimistic_spin+0x9c/0x1d0 [ 68.909951] [] __mutex_lock_slowpath+0x44/0x158 [ 68.909953] [] mutex_

Re: [PATCH 0/2] arm64: initial series for UniPhier ARMv8 SoCs support

2015-12-11 Thread Arnd Bergmann
On Tuesday 24 November 2015 18:08:27 Masahiro Yamada wrote: > Hi Arnd, Olof, > > This series adds ARM64 Linux support for Socionext's UniPhier SoCs. > > The device trees will be sent in another series. Applied both to next/config64, thanks! Arnd -- To unsubscribe from this list: send th

Re: [PATCH 0/4] arm: dts: qcom-apq8064: add smem and hwspinlock support

2015-12-11 Thread Bjorn Andersson
On Fri 11 Dec 10:26 PST 2015, Srinivas Kandagatla wrote: > Hi Andy, > > Here are 3 patches for smem/hwspinlock which I have tested with QDSP on > IFC6410. > Also a fix from Ivan which I think can be taken aswell. > As far as I can tell my patch for adding smem and hwmutex are already in linux

Re: [PATCH 2/2] netcp: add more __le32 annotations

2015-12-11 Thread David Miller
From: Arnd Bergmann Date: Tue, 08 Dec 2015 16:32:59 +0100 > The handling of epib and psdata remains a bit unclear in the driver, > as we access the same fields both as CPU-endian and through DMA > from the device. > > Sparse warns about this: > ti/netcp_core.c:1147:21: warning: incorrect type in

Re: [PATCH 1/2] netcp: try to reduce type confusion in descriptors

2015-12-11 Thread David Miller
From: Arnd Bergmann Date: Tue, 08 Dec 2015 16:32:27 +0100 > The netcp driver produces tons of warnings when CONFIG_LPAE is enabled > on ARM: > > drivers/net/ethernet/ti/netcp_core.c: In function 'netcp_tx_map_skb': > drivers/net/ethernet/ti/netcp_core.c:1084:13: warning: passing argument 1 of >

[GIT PULL] VC4 3D support

2015-12-11 Thread Eric Anholt
Hi Dave, Since you asked for early pull requests, here's what I've got queued up. I may want to add some more after this if I can, but this will let us ship 3D support on 4.5 kernels. It's also what I'm pushing to the Raspberry Pi Foundation kernel for their update to 4.2. The following changes

RE: [PATCH V2 02/10] Drivers: hv: utils: run polling callback always in interrupt context

2015-12-11 Thread KY Srinivasan
> -Original Message- > From: Dexuan Cui > Sent: Thursday, December 10, 2015 11:33 PM > To: KY Srinivasan ; gre...@linuxfoundation.org; linux- > ker...@vger.kernel.org; de...@linuxdriverproject.org; o...@aepfle.de; > a...@canonical.com; vkuzn...@redhat.com; jasow...@redhat.com > Subject: R

Re: new warning on sysrq kernel crash trigger

2015-12-11 Thread Paul E. McKenney
On Fri, Dec 11, 2015 at 04:16:37PM -0800, Ani Sinha wrote: > On Fri, 11 Dec 2015, Paul E. McKenney wrote: > > On Fri, Dec 11, 2015 at 05:10:43PM -0500, Rik van Riel wrote: > > > On 12/11/2015 03:44 PM, Ani Sinha wrote: > > > > > > > > > > > > On Thu, 10 Dec 2015, Paul E. McKenney wrote: > > > >

Re: [PATCH] mtd: omap_elm: print interrupt resource using %pr

2015-12-11 Thread Brian Norris
Hi Arnd, On Tue, Dec 08, 2015 at 04:39:45PM +0100, Arnd Bergmann wrote: > When CONFIG_LPAE is set on ARM, resource_size_t is 64-bit wide > and we get a warning about an incorrect format string for printing > the interrupt number in elm_probe: > > drivers/mtd/nand/omap_elm.c: In function 'elm_prob

[PATCH] drivers: memory: check for missing sections in test_pages_in_a_zone

2015-12-11 Thread Andrew Banman
test_pages_in_a_zone does not account for the possibility of missing sections in the given pfn range. pfn_valid_within always returns 1 when CONFIG_HOLES_IN_ZONE is not set, allowing invalid pfns from missing sections to pass the test, leading to a kernel oops. Wrap an additional pfn loop with PAG

Re: [PATCH net] ipv6: sctp: clone options to avoid use after free

2015-12-11 Thread David Miller
From: Eric Dumazet Date: Wed, 09 Dec 2015 07:25:06 -0800 > From: Eric Dumazet > > SCTP is lacking proper np->opt cloning at accept() time. > > TCP and DCCP use ipv6_dup_options() helper, do the same > in SCTP. > > We might later factorize this code in a common helper to avoid > future mistake

[PATCH 03/10] usb: host: ehci-sched: remove useless assignments

2015-12-11 Thread Geyslan G. Bem
This patch removes useless assignments. Tested by compilation only. Caught by cppcheck. Signed-off-by: Geyslan G. Bem --- drivers/usb/host/ehci-sched.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/usb/host/ehci-sched.c b/drivers/usb/host/ehci-sched.c index 89b

[PATCH 04/10] usb: host: ehci-sched: add spaces around operators

2015-12-11 Thread Geyslan G. Bem
This patch adds spaces around operators. Tested by compilation only. Caught by checkpatch. Signed-off-by: Geyslan G. Bem --- drivers/usb/host/ehci-sched.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/host/ehci-sched.c b/drivers/usb/host/ehci-sched.c index 4bb6

[PATCH 00/10] usb: host: ehci-sched: cleanup

2015-12-11 Thread Geyslan G. Bem
Cleanup done with the help of coccinelle, checkpatch and cppcheck tools. Geyslan G. Bem (10): usb: host: ehci-sched: refactor scan_isoc function usb: host: ehci-sched: move constants to right usb: host: ehci-sched: remove useless assignments usb: host: ehci-sched: add spaces around operato

[PATCH 01/10] usb: host: ehci-sched: refactor scan_isoc function

2015-12-11 Thread Geyslan G. Bem
This patch removes an infinite 'for' loop and makes use of the already existing 'restart' tag instead, reducing one leading tab. The comments and code were corrected conforming coding style. Tested by compilation only. Caught by checkpatch: WARNING: Too many leading tabs - consider code refactor

[PATCH 05/10] usb: host: ehci-sched: remove prohibited spaces

2015-12-11 Thread Geyslan G. Bem
This patch removes prohibited spaces before open parenthesis and open brackets. It also removes an assignment inside condition and unnecessary braces in single statement block. Tested by compilation only. Caught by checkpatch. Signed-off-by: Geyslan G. Bem --- drivers/usb/host/ehci-sched.c | 2

[PATCH 07/10] usb: host: ehci-sched: use C89-style comments

2015-12-11 Thread Geyslan G. Bem
This patch changes comments conforming coding style. Caught by checkpatch. Signed-off-by: Geyslan G. Bem --- drivers/usb/host/ehci-sched.c | 26 ++ 1 file changed, 14 insertions(+), 12 deletions(-) diff --git a/drivers/usb/host/ehci-sched.c b/drivers/usb/host/ehci-sched

[PATCH 10/10] usb: host: ehci-sched: remove unnecessary braces

2015-12-11 Thread Geyslan G. Bem
This patch removes unnecessary braces in single statement blocks at the same time as replaces the if statement with a ternary conditional. Tested by compilation only. Caught by checkpatch. Signed-off-by: Geyslan G. Bem --- drivers/usb/host/ehci-sched.c | 6 +- 1 file changed, 1 insertion(+)

[PATCH 08/10] usb: host: ehci-sched: add line after declarations

2015-12-11 Thread Geyslan G. Bem
This patch adds a blank line after declarations. Caught by checkpatch. Signed-off-by: Geyslan G. Bem --- drivers/usb/host/ehci-sched.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/usb/host/ehci-sched.c b/drivers/usb/host/ehci-sched.c index cf203d6..2e7d20f 100644 --- a/drivers/

<    3   4   5   6   7   8   9   >