[PATCH 3/6] virtio-scsi: avoid cancelling uninitialized work items

2014-06-03 Thread Paolo Bonzini
Calling the workqueue interface on uninitialized work items isn't a good idea even if they're zeroed. It's not failing catastrophically only through happy accidents, and a debug kernel rightfully complains. Cc: sta...@vger.kernel.org Signed-off-by: Paolo Bonzini pbonz...@redhat.com ---

[PATCH 4/6] scsi_error: fix invalid setting of host byte

2014-06-03 Thread Paolo Bonzini
From: Ulrich Obergfell uober...@redhat.com After scsi_try_to_abort_cmd returns, the eh_abort_handler may have already found that the command has completed in the device, causing the host_byte to be nonzero (e.g. it could be DID_ABORT). When this happens, ORing DID_TIME_OUT into the host byte

[PATCH 5/6] virtio-scsi: fix various bad behavior on aborted requests

2014-06-03 Thread Paolo Bonzini
Even though the virtio-scsi spec guarantees that all requests related to the TMF will have been completed by the time the TMF itself completes, the request queue's callback might not have run yet. This causes requests to be completed more than once, and as a result triggers a variety of BUGs or

[PATCH 6/6] virtio-scsi: Implement change_queue_depth for virtscsi targets

2014-06-03 Thread Paolo Bonzini
From: Venkatesh Srinivas venkate...@google.com change_queue_depth allows changing per-target queue depth via sysfs. It also allows the SCSI midlayer to ramp down the number of concurrent inflight requests in response to a SCSI BUSY status response and allows the midlayer to ramp the count back

[PATCH resend 0/6] virtio-scsi patches for 3.16 + a midlayer one-liner

2014-06-03 Thread Paolo Bonzini
Sorry for the delay, mostly caused by the discussion and testing of patch 5. That's the most important of the bunch, and debugging the problem also led to the discovery of a midlayer bug fixed by patch 4. I hope these can make their way into 3.16. Thanks! resend: forgot linux-scsi. :( Paolo

Fwd: [PATCH] fix coding style issues

2014-06-03 Thread Nicolas Koch
From: Nicolas Koch nioko1...@gmail.com Date: Tue, 3 Jun 2014 21:21:18 +0200 Subject: [PATCH] fix coding style issues To: de...@driverdev.osuosl.org Cc: Greg Kroah-Hartman gre...@linuxfoundation.org, Aaro Koskinen aaro.koski...@iki.fi, Nicolas Koch nioko1...@googlemail.com Done as task 10 of the

[PATCH 2/6] virtio-scsi: replace target spinlock with seqcount

2014-06-03 Thread Paolo Bonzini
From: Ming Lei ming@canonical.com The spinlock of tgt_lock is only for serializing read and write req_vq, one lockless seqcount is enough for the purpose. On one 16core VM with vhost-scsi backend, the patch can improve IOPS with 3% on random read test. Signed-off-by: Ming Lei

[PATCH 3/6] virtio-scsi: avoid cancelling uninitialized work items

2014-06-03 Thread Paolo Bonzini
Calling the workqueue interface on uninitialized work items isn't a good idea even if they're zeroed. It's not failing catastrophically only through happy accidents, and a debug kernel rightfully complains. Cc: sta...@vger.kernel.org Signed-off-by: Paolo Bonzini pbonz...@redhat.com ---

Re: [PATCH 1/2] block,scsi: verify return pointer from blk_get_request

2014-06-03 Thread Jeff Moyer
Joe Lawrence joe.lawre...@stratus.com writes: The blk-core dead queue checks introduce an error scenario to blk_get_request that returns NULL if the request queue has been shutdown. This affects the behavior for __GFP_WAIT callers, who should verify the return value before dereferencing.

[PATCH 0/6] virtio-scsi patches for 3.16 + a midlayer one-liner

2014-06-03 Thread Paolo Bonzini
Sorry for the delay, mostly caused by the discussion and testing of patch 5. That's the most important of the bunch, and debugging the problem also led to the discovery of a midlayer bug fixed by patch 4. I hope these can make their way into 3.16. Thanks! Paolo Ming Lei (2): virtio_scsi:

[PATCH 1/6] virtio_scsi: remove ACCESS_ONCE() and smp_read_barrier_depends()

2014-06-03 Thread Paolo Bonzini
From: Ming Lei tom.leim...@gmail.com Access to tgt-req_vq is strictly serialized by spin_lock of tgt-tgt_lock, so the ACCESS_ONCE() isn't necessary. smp_read_barrier_depends() in virtscsi_req_done was introduced to order reading req_vq and decreasing tgt-reqs, but it isn't needed now because

[PATCH 2/6] virtio-scsi: replace target spinlock with seqcount

2014-06-03 Thread Paolo Bonzini
From: Ming Lei ming@canonical.com The spinlock of tgt_lock is only for serializing read and write req_vq, one lockless seqcount is enough for the purpose. On one 16core VM with vhost-scsi backend, the patch can improve IOPS with 3% on random read test. Signed-off-by: Ming Lei

[PATCH] PCI: quirk dma_alias_devfn for HighPoint RocketRaid 642L (Marvell 9235)

2014-06-03 Thread Jérôme Carretero
This vendor has similar boards based on the same Marvell 88SE9235 chipset, but this patch was only tested with the 642L. Tested on ASUS Sabertooth 990FX (AMD). If you listen to the vendor support stating: Dear Customer, Sorry about that. Currently all our products' software package don't

Re: [PATCH 2/2] block,scsi: convert and handle ERR_PTR from blk_get_request

2014-06-03 Thread Jeff Moyer
Joe Lawrence joe.lawre...@stratus.com writes: The blk_get_request function may fail in low-memory conditions or during device removal (even if __GFP_WAIT is set). To distinguish between these errors, modify the blk_get_request call stack to return the appropriate ERR_PTR. Verify that all

Re: [PATCH] platform/chrome: Add support for Acer C720

2014-06-03 Thread Benson Leung
Hi Mika! Thanks for putting this patch together. It looks like this does the job of distinguishing the two busses apart. Thanks! On Tue, Jun 3, 2014 at 6:02 AM, Mika Westerberg mika.westerb...@linux.intel.com wrote: Acer C720 has touchpad and light sensor connected to a separate I2C buses.

Re: [PATCH] firmware: Added fw for qat_dh895xxcc hw crypto accelerator

2014-06-03 Thread Ben Hutchings
On Tue, 2014-06-03 at 11:24 -0700, Tadeusz Struk wrote: Hi, This adds firmware for Intel QAT dh895xcc HW accelerator. Signed-off-by: Tadeusz Struk tadeusz.st...@intel.com --- LICENCE.qat_dh895xcc_firmware | 39 +++ WHENCE| 6

Re: [PATCH] perf, tools: Support spark lines in perf stat v5

2014-06-03 Thread Jiri Olsa
On Fri, May 30, 2014 at 02:42:16PM -0700, Andi Kleen wrote: From: Andi Kleen a...@linux.intel.com perf stat -rX prints the stddev for multiple measurements. Just looking at the stddev for judging the quality of the data is a bit dangerous The simplest sanity check is to just look at a

Re: [PATCH v5 6/6] seccomp: add SECCOMP_EXT_ACT_TSYNC and SECCOMP_FILTER_TSYNC

2014-06-03 Thread Kees Cook
On Mon, Jun 2, 2014 at 6:15 PM, Andy Lutomirski l...@amacapital.net wrote: On Mon, Jun 2, 2014 at 6:09 PM, Kees Cook keesc...@chromium.org wrote: On Mon, Jun 2, 2014 at 5:29 PM, Andy Lutomirski l...@amacapital.net wrote: On Mon, Jun 2, 2014 at 5:14 PM, Kees Cook keesc...@chromium.org wrote: On

[PATCH V2 2/2] drivers/s390/cio/qdio_main.c: replace shift loop by ilog2

2014-06-03 Thread Fabian Frederick
account_sbals is called by get_inbound_buffer_frontier and get_outbound_buffer_frontier with 'count' value 0 so we can safely convert shift loop to ilog2. Cc: Sebastian Ott seb...@linux.vnet.ibm.com Cc: Joe Perches j...@perches.com Cc: Andrew Morton a...@linux-foundation.org Signed-off-by:

[PATCH V2 1/2] drivers/s390/cio/qdio_main.c: use unsigned int for count everywhere

2014-06-03 Thread Fabian Frederick
count was used unsigned int and int. Cc: Sebastian Ott seb...@linux.vnet.ibm.com Cc: Joe Perches j...@perches.com Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- This is untested. V2: -count int - unsigned int (suggested by Joe Perches)

[patch] Staging: octeon: minor style cleanups

2014-06-03 Thread Nicolas Koch
Done as task 10 of the eudyptula challenge. Signed-off-by: Nicolas Koch nioko1...@googlemail.com --- drivers/staging/octeon/ethernet-mem.c | 11 +++ 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/drivers/staging/octeon/ethernet-mem.c

[PATCH 1/1] fs/file.c: use SIZE_MAX instead of ~(size_t)0

2014-06-03 Thread Fabian Frederick
Use kernel.h definition. Cc: Alexander Viro v...@zeniv.linux.org.uk Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- fs/file.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/file.c b/fs/file.c index 8f294cf..d63f731 100644 ---

Re: [RFC 1/4] cfg80211: Add channel flags limiting availability to OCB mode only

2014-06-03 Thread Johannes Berg
On Fri, 2014-05-30 at 18:56 +0200, Rostislav Lisovy wrote: IEEE 802.11p operates in its own 5.9GHz band. When there will be a record for the 5.9GHz band in the regulatory daemon, it must be limited to the OCB mode only -- using the newly added flags. Is this really a *regulatory* limitation?

Re: [BUG] signal: sighand unprotected when accessed by /proc

2014-06-03 Thread Oleg Nesterov
On 06/03, Linus Torvalds wrote: On Tue, Jun 3, 2014 at 10:26 AM, Oleg Nesterov o...@redhat.com wrote: looks like, SLAB_DESTROY_BY_RCU logic is broken? I haven't looked at the code yet, but SLAB_DESTROY_BY_RCU can be subtle and very dangerous. The danger is that the *slab* itself is

Re: [PATCH 3/5] irq_work: Implement remote queueing

2014-06-03 Thread Eric Dumazet
On Tue, 2014-06-03 at 18:37 +0200, Peter Zijlstra wrote: On Tue, Jun 03, 2014 at 09:29:07AM -0700, Eric Dumazet wrote: +bool irq_work_queue_on(struct irq_work *work, int cpu) +{ + /* All work should have been flushed before going offline */ + WARN_ON_ONCE(cpu_is_offline(cpu)); +

[PATCH linux-firmware] README: Update the submission address to linux-firmw...@kernel.org

2014-06-03 Thread Ben Hutchings
This new alias will reach all maintainers. Signed-off-by: Ben Hutchings b...@decadent.org.uk --- I've just cleared out my patch queue, applied this change and pushed. If I get any later submissions with only individual addresses, I'll request re-submission to linux-firmw...@kernel.org Ben.

[PATCH v5 00/15] Thunderbolt driver for Apple MacBooks

2014-06-03 Thread Andreas Noever
Hi, Updated version of the driver. Includes changes by Matthew to make it work on Falcon Ridge hardware (present in the newest Mac(Book)Pro). Changes since v4 (suggested by Matthew): - Changed the PCI quirks to check just for Apple hardware instead of checking against a list of machines.

Re: [BUG] signal: sighand unprotected when accessed by /proc

2014-06-03 Thread Oleg Nesterov
Damn, sorry for noise... On 06/03, Oleg Nesterov wrote: I'll try to recheck rt_mutex_unlock() tomorrow. _Perhaps_ rcu_read_unlock() should be shifted from lock_task_sighand() to unlock_task_sighand() to ensure that rt_mutex_unlock() does nothihg with this memory after it makes another

[PATCH v5 06/15] thunderbolt: Add thunderbolt capability handling

2014-06-03 Thread Andreas Noever
Thunderbolt config areas contain capability lists similar to those found on pci devices. This patch introduces a tb_find_cap utility method to search for capabilities. Signed-off-by: Andreas Noever andreas.noe...@gmail.com --- drivers/thunderbolt/Makefile | 2 +- drivers/thunderbolt/cap.c|

[PATCH v5 14/15] thunderbolt: Read switch uid from EEPROM

2014-06-03 Thread Andreas Noever
Add eeprom access code and read the uid during switch initialization. The UID will be used to check device identity after suspend. Signed-off-by: Andreas Noever andreas.noe...@gmail.com --- drivers/thunderbolt/Makefile | 2 +- drivers/thunderbolt/eeprom.c | 189

[PATCH v5 12/15] PCI: Add pci_fixup_suspend_late quirk pass

2014-06-03 Thread Andreas Noever
Add pci_fixup_suspend_late as a new pci_fixup_pass. The pass is called from suspend_noirq and poweroff_noirq. Using the same pass for suspend and hibernate is consistent with resume_early which is called by resume_noirq and restore_noirq. The new quirk pass is required for Thunderbolt support on

[PATCH 1/1] kernel/audit.c: use ARRAY_SIZE instead of sizeof/sizeof[0]

2014-06-03 Thread Fabian Frederick
Use kernel.h definition. Cc: Eric Paris epa...@redhat.com Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- kernel/audit.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/audit.c b/kernel/audit.c index 47845c5..48e6e5d

[PATCH v5 13/15] PCI: Suspend/resume quirks for Apple thunderbolt

2014-06-03 Thread Andreas Noever
Add two quirks to support thunderbolt suspend/resume on Apple systems. We need to perform two different actions during suspend and resume: The whole controller has to be powered down before suspend. If this is not done then the native host interface device will be gone after resume if a

[PATCH v5 10/15] thunderbolt: Add path setup code.

2014-06-03 Thread Andreas Noever
A thunderbolt path is a unidirectional channel between two thunderbolt ports. Two such paths are needed to establish a pci tunnel. This patch introduces struct tb_path as well as a set of tb_path_* methods which are used to activate deactivate paths. Signed-off-by: Andreas Noever

[PATCH v5 15/15] thunderbolt: Add suspend/hibernate support

2014-06-03 Thread Andreas Noever
We use _noirq since we have to restore the pci tunnels before the pci core wakes the tunneled devices. Signed-off-by: Andreas Noever andreas.noe...@gmail.com --- drivers/thunderbolt/nhi.c| 33 + drivers/thunderbolt/switch.c | 84

Re: [RFC][PATCH 0/8] sched,idle: need resched polling rework

2014-06-03 Thread Andy Lutomirski
On Tue, Jun 3, 2014 at 11:44 AM, Andy Lutomirski l...@amacapital.net wrote: On Tue, Jun 3, 2014 at 11:28 AM, Peter Zijlstra pet...@infradead.org wrote: On Tue, Jun 03, 2014 at 07:00:18PM +0200, Peter Zijlstra wrote: On Tue, Jun 03, 2014 at 09:52:22AM -0700, Andy Lutomirski wrote: So you

[PATCH v5 07/15] thunderbolt: Enable plug events

2014-06-03 Thread Andreas Noever
Thunderbolt switches have a plug events capability. This patch adds the tb_plug_events_active method and uses it to activate plug events during switch allocation. Signed-off-by: Andreas Noever andreas.noe...@gmail.com --- drivers/thunderbolt/switch.c | 52

[PATCH v5 09/15] thunderbolt: Handle hotplug events

2014-06-03 Thread Andreas Noever
We receive a plug event callback whenever a thunderbolt device is added or removed. This patch fills in the tb_handle_hotplug method and starts reacting to these events by adding/removing switches from the hierarchy. Signed-off-by: Andreas Noever andreas.noe...@gmail.com ---

[PATCH v5 05/15] thunderbolt: Initialize root switch and ports

2014-06-03 Thread Andreas Noever
This patch adds the structures tb_switch and tb_port as well as code to initialize the root switch. Signed-off-by: Andreas Noever andreas.noe...@gmail.com --- drivers/thunderbolt/Makefile | 2 +- drivers/thunderbolt/switch.c | 186 +++

[PATCH v5 04/15] thunderbolt: Add tb_regs.h

2014-06-03 Thread Andreas Noever
Every thunderbolt device consists (logically) of a switch with multiple ports. Every port contains up to four config regions (HOPS, PORT, SWITCH, COUNTERS) which are used to configure the device. The tb_regs.h file contains all known registers and capabilities from these config regions.

[PATCH v5 08/15] thunderbolt: Scan for downstream switches

2014-06-03 Thread Andreas Noever
Add utility methods tb_port_state and tb_wait_for_port. Add tb_scan_switch which recursively checks for downstream switches. Signed-off-by: Andreas Noever andreas.noe...@gmail.com --- drivers/thunderbolt/switch.c | 97 drivers/thunderbolt/tb.c |

[PATCH] IB/mlx4: fix GFP type in mlx4_table_get() and mlx4_qp_alloc_icm()

2014-06-03 Thread Jiri Kosina
Commit 40f2287bd (IB/mlx4: Implement IB_QP_CREATE_USE_GFP_NOIO) introduced passing of GFP flags when allocating memory resources for QPs. GFP flags passed to mlx4_table_get() and mlx4_qp_alloc_icm() are always of gfp_t type, so let's not pass them as int. This fixes various sparse warnings.

[PATCH 1/1] lib/test-kstrtox.c: use ARRAY_SIZE instead of sizeof/sizeof[0]

2014-06-03 Thread Fabian Frederick
Use kernel.h definition. Cc: Alexey Dobriyan adobri...@gmail.com Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- lib/test-kstrtox.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/test-kstrtox.c b/lib/test-kstrtox.c index

[PATCH v5 02/15] thunderbolt: Add control channel interface

2014-06-03 Thread Andreas Noever
Thunderbolt devices are configured by reading/writing into their configuration space (similar to pci). This is done by sending packets through the NHI (native host interface) onto the control channel. This patch handles the low level packet based protocol and exposes higher level operations like

[PATCH v5 03/15] thunderbolt: Setup control channel

2014-06-03 Thread Andreas Noever
Add struct tb which will contain our view of the thunderbolt bus. For now it just contains a pointer to the control channel and a workqueue for hotplug events. Add thunderbolt_alloc_and_start() and thunderbolt_shutdown_and_free() which are responsible for setup and teardown of struct tb.

[PATCH v5 01/15] thunderbolt: Add initial cactus ridge NHI support

2014-06-03 Thread Andreas Noever
Thunderbolt hotplug is supposed to be handled by the firmware. But Apple decided to implement thunderbolt at the operating system level. The firmare only initializes thunderbolt devices that are present at boot time. This driver enables hotplug of thunderbolt of non-chained thunderbolt devices on

Re: [BUG] signal: sighand unprotected when accessed by /proc

2014-06-03 Thread Oleg Nesterov
Steven, I am already sleeping, probably I just need to re-read your email tomorrow, but... On 06/03, Steven Rostedt wrote: When we convert the spin_lock into a rtmutex, when we hit the lock and it was contended (the task was in the process of exiting and it takes the lock to set tsk-sighand

Re: [RFC 2/4] cfg80211: Add new interface type for OCB mode

2014-06-03 Thread Johannes Berg
On Fri, 2014-05-30 at 18:56 +0200, Rostislav Lisovy wrote: Add new OCB mode (outside the context of the BSS) interface type as well as functions necessary to configure the interface when 'joining' such network. I think you also want some API to leave (stop operating in) the network again, and

Re: [PATCH v2 0/4] Emulate VMXON region correctly

2014-06-03 Thread Bandan Das
Paolo Bonzini pbonz...@redhat.com writes: Il 06/05/2014 08:19, Bandan Das ha scritto: Reference: https://bugzilla.kernel.org/show_bug.cgi?id=54521 The vmxon region is unused by nvmx, but adding these checks are probably harmless and may detect buggy L1 hypervisors in the future! Applied

Re: [PATCH v2 RFC 1/3] MAINTAINERS: Add R: designated-reviewers tag

2014-06-03 Thread Paul E. McKenney
On Tue, Jun 03, 2014 at 12:03:07PM -0700, Joe Perches wrote: On Tue, 2014-06-03 at 08:35 -0700, Paul E. McKenney wrote: Paul Walmsley also noted the need for patch submitters to know who the key reviewers are and suggested adding an R: tag to the MAINTAINERS file to record this information

[PATCH] tty: serial: msm: Fix section mismatch warning

2014-06-03 Thread Kumar Gala
WARNING: drivers/tty/built-in.o(.data+0x3544): Section mismatch in reference from the variable msm_platform_driver to the function .init.text:msm_serial_probe() The variable msm_platform_driver references the function __init msm_serial_probe() If the reference is valid then annotate the variable

[PATCH 1/1] fs/adfs/dir_fplus.c: use ARRAY_SIZE instead of sizeof/sizeof[0]

2014-06-03 Thread Fabian Frederick
Use kernel.h definition. Cc: Andrew Morton a...@linux-foundation.org Signed-off-by: Fabian Frederick f...@skynet.be --- fs/adfs/dir_fplus.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/adfs/dir_fplus.c b/fs/adfs/dir_fplus.c index d9e3bee..7673c1f 100644 ---

Re: [PATCH v5] conversion to blk-mq

2014-06-03 Thread Matias Bjorling
On 06/03/2014 01:06 AM, Keith Busch wrote: Depending on the timing, it might fail in alloc instead of free: Jun 2 16:45:40 kbgrz1 kernel: [ 265.421243] NULL pointer dereference at (null) Jun 2 16:45:40 kbgrz1 kernel: [ 265.434284] PGD 429acf067 PUD 42ce28067 PMD 0 Jun 2

Re: [RFC 4/4] mac80211: OCB mode interface configuration

2014-06-03 Thread Johannes Berg
On Fri, 2014-05-30 at 18:56 +0200, Rostislav Lisovy wrote: + static u8 bssid_wildcard[ETH_ALEN] = { 0xff, 0xff, 0xff, +0xff, 0xff, 0xff }; const that doesn't actually exist anywhere already? Maybe not I guess. +int ieee80211_start_ocb(struct

[PATCH v5 11/15] thunderbolt: Add support for simple pci tunnels

2014-06-03 Thread Andreas Noever
A pci downstream and pci upstream port can be connected through a tunnel. To establish the tunnel we have to setup two unidirectional paths between the two ports. Right now we only support paths with two hops (i.e. no chaining) and at most one pci device per thunderbolt device. Signed-off-by:

Re: [BUG] signal: sighand unprotected when accessed by /proc

2014-06-03 Thread Steven Rostedt
On Tue, 3 Jun 2014 11:03:49 -0700 Linus Torvalds torva...@linux-foundation.org wrote: An example of (b) is having a lock that is initialized by the constructor, and that everybody else uses properly: you may have delayed people locking/unlocking it, but as long as they don't screw anything

Re: [BUG] signal: sighand unprotected when accessed by /proc

2014-06-03 Thread Steven Rostedt
On Tue, 3 Jun 2014 22:09:38 +0200 Oleg Nesterov o...@redhat.com wrote: Now, if that lock is released and reused (I didn't trace other tasks allocating these locks), it reinitializes the lock-wait_list. How? From where? This should be done by sighand_ctor() only? heh, the more I look at

Re: [PATCH v6 net-next 1/4] net: flow_dissector: avoid multiple calls in eBPF

2014-06-03 Thread Alexei Starovoitov
On Tue, Jun 3, 2014 at 1:33 AM, Daniel Borkmann dbork...@redhat.com wrote: On 06/02/2014 06:48 PM, Alexei Starovoitov wrote: imo there are pros and cons in Daniel's and Chema's proposals for classic BPF extensions. I like Chema's a bit more, since his proposal doesn't require to change

Re: [PATCH v2 RFC 1/3] MAINTAINERS: Add R: designated-reviewers tag

2014-06-03 Thread Joe Perches
On Tue, 2014-06-03 at 13:12 -0700, Paul E. McKenney wrote: On Tue, Jun 03, 2014 at 12:03:07PM -0700, Joe Perches wrote: On Tue, 2014-06-03 at 08:35 -0700, Paul E. McKenney wrote: Paul Walmsley also noted the need for patch submitters to know who the key reviewers are and suggested adding

Re: [RFC 0/4 v2] cfg80211/mac80211: Add new interface type for OCB (Outside the Context of a BSS) mode

2014-06-03 Thread Johannes Berg
On Fri, 2014-05-30 at 18:56 +0200, Rostislav Lisovy wrote: This patchset presents work in progress. Patches adding *_leave_ocb() functions and modifying RX/TX path in mac80211 to work with OCB mode will come in near future. Oh, sorry - I commented on that but failed to read the cover letter

Re: [PATCH -mm 8/8] slab: reap dead memcg caches aggressively

2014-06-03 Thread Vladimir Davydov
On Mon, Jun 02, 2014 at 10:24:09AM -0500, Christoph Lameter wrote: On Sat, 31 May 2014, Vladimir Davydov wrote: You can use a similar approach than in SLUB. Reduce the size of the per cpu array objects to zero. Then SLAB will always fall back to its slow path in cache_flusharray()

Re: [BUG] signal: sighand unprotected when accessed by /proc

2014-06-03 Thread Steven Rostedt
On Tue, 3 Jun 2014 22:09:38 +0200 Oleg Nesterov o...@redhat.com wrote: Now, if that lock is released and reused (I didn't trace other tasks allocating these locks), it reinitializes the lock-wait_list. How? From where? This should be done by sighand_ctor() only? This looks definitely

Re: [PATCH] tty: serial: msm: Fix section mismatch warning

2014-06-03 Thread Greg KH
On Tue, Jun 03, 2014 at 03:13:22PM -0500, Kumar Gala wrote: WARNING: drivers/tty/built-in.o(.data+0x3544): Section mismatch in reference from the variable msm_platform_driver to the function .init.text:msm_serial_probe() The variable msm_platform_driver references the function __init

Re: [PATCH 3/5] irq_work: Implement remote queueing

2014-06-03 Thread Frederic Weisbecker
On Tue, Jun 03, 2014 at 01:02:39PM -0700, Eric Dumazet wrote: On Tue, 2014-06-03 at 18:37 +0200, Peter Zijlstra wrote: On Tue, Jun 03, 2014 at 09:29:07AM -0700, Eric Dumazet wrote: +bool irq_work_queue_on(struct irq_work *work, int cpu) +{ + /* All work should have been

Re: [PATCH] powerpc, kexec: Fix Processor X is stuck issue during kexec from ST mode

2014-06-03 Thread Srivatsa S. Bhat
On 05/28/2014 07:01 PM, Vivek Goyal wrote: On Tue, May 27, 2014 at 04:25:34PM +0530, Srivatsa S. Bhat wrote: If we try to perform a kexec when the machine is in ST (Single-Threaded) mode (ppc64_cpu --smt=off), the kexec operation doesn't succeed properly, and we get the following messages

Re: [PATCH v5 00/15] Thunderbolt driver for Apple MacBooks

2014-06-03 Thread Greg KH
On Tue, Jun 03, 2014 at 10:03:57PM +0200, Andreas Noever wrote: Hi, Updated version of the driver. Includes changes by Matthew to make it work on Falcon Ridge hardware (present in the newest Mac(Book)Pro). Changes since v4 (suggested by Matthew): - Changed the PCI quirks to check just

Re: [PATCH -mm] mincore: apply page table walker on do_mincore() (Re: [PATCH 00/10] mm: pagewalk: huge page cleanups and VMA passing)

2014-06-03 Thread Dave Hansen
On 06/03/2014 01:01 PM, Naoya Horiguchi wrote: On Tue, Jun 03, 2014 at 08:55:04AM -0700, Dave Hansen wrote: On 06/02/2014 11:18 PM, Naoya Horiguchi wrote: And for patch 8, 9, and 10, I don't think it's good idea to add a new callback which can handle both pmd and pte (because they are

[PATCH 5/9] perf tools: Fix == into = in ui_browser__warning assignment

2014-06-03 Thread Jiri Olsa
From: zhangdianfang zhangdianf...@huawei.com Convert == into = in ui_browser__warning assignment. Bug description: https://bugzilla.kernel.org/show_bug.cgi?id=76751 Reported-by: David Binderman dcb...@hotmail.com Signed-off-by: Dianfang Zhang zhangdianf...@huawei.com Acked-by: Arnaldo Carvalho

Re: [PATCH v2 net-next 0/2] split BPF out of core networking

2014-06-03 Thread Daniel Borkmann
On 06/03/2014 05:44 PM, Alexei Starovoitov wrote: ... All of your points are valid. They are right questions to ask. I just don't see why you're still arguing about first step of filter.c split, whereas your concerns are about steps 2, 3, 4. Fair enough, lets keep them in mind though for

[PATCH 7/9] perf tools: Move elide bool into perf_hpp_fmt struct

2014-06-03 Thread Jiri Olsa
After output/sort fields refactoring, it's expensive to check the elide bool in its current location inside the 'struct sort_entry'. The perf_hpp__should_skip function gets highly noticable in workloads with high number of output/sort fields, like for: $ perf report -i perf-test.data -F

[PATCH 2/9] perf tools: Add warning when disabling perl scripting support due to missing devel files

2014-06-03 Thread Jiri Olsa
From: Arnaldo Carvalho de Melo a...@redhat.com We were just showing libperl: OFF, unlike other features where we present the user with a message helping have a feature built in. Fix it by adding the following message: config/Makefile:450: Missing perl devel files. Disabling perl scripting

[PATCH 1/9] perf trace: Warn the user when not available

2014-06-03 Thread Jiri Olsa
From: Arnaldo Carvalho de Melo a...@redhat.com When the audit-libs devel package is not found at build time we disable the 'trace' command, as we are not able to map syscall numbers to strings, but then the message the user is presented is cryptic: [root@zoo linux]# trace ls perf: 'ls' is

[GIT PULL 0/9] perf/core improvements and fixes

2014-06-03 Thread Jiri Olsa
hi Ingo, please consider pulling thanks, jirka The following changes since commit 9b261365dd73a5014b49033327ad881708e81f33: Merge tag 'perf-core-for-mingo' of git://git.kernel.org/pub/scm/linux/kernel/git/jolsa/perf into perf/core (2014-06-03 20:22:40 +0200) are available in the git

[PATCH 8/9] perf record: Fix poll return value propagation

2014-06-03 Thread Jiri Olsa
If the perf record command is interrupted in record__mmap_read_all function, the 'done' is set and err has the latest poll return value, which is most likely positive number (= number of pollfds ready to read). This 'positive err' is then propagated to the exit code, resulting in not finishing

[PATCH 9/9] perf tools: Fix 'make help' message error

2014-06-03 Thread Jiri Olsa
From: Jianyu Zhan nasa4...@gmail.com Currently 'make help' message has such hint: use make prefix=path install target to install to a particular path like make prefix=/usr/local install install-doc But this is misleading, when I specify prefix=/usr/local, it has got no respect at all.

Re: [PATCH -tip ] [BUGFIX] perf/probe: Fix perf probe to find correct variable DIE

2014-06-03 Thread Arnaldo Carvalho de Melo
Em Thu, May 29, 2014 at 09:39:14PM +0900, Masami Hiramatsu escreveu: Hi Arnaldo, Here is the patch which fixes perf probe to find variable location correctly, on the recent dwarf format. This is not related to the SEGV issue which I fixed in previous mail. Hey, if I apply just this one I

[PATCH 4/9] perf tools: Allow overriding sysfs and proc finding with env var

2014-06-03 Thread Jiri Olsa
From: Cody P Schafer d...@codyps.com SYSFS_PATH and PROC_PATH environment variables now let the user override the detection of sysfs and proc locations for testing purposes. Signed-off-by: Cody P Schafer d...@codyps.com Cc: Sukadev Bhattiprolu suka...@linux.vnet.ibm.com Link:

[PATCH 6/9] perf tools: Remove elide setup for SORT_MODE__MEMORY mode

2014-06-03 Thread Jiri Olsa
There's no need to setup elide of sort_dso sort entry again with symbol_conf.dso_list list. The only difference were list names of memory mode data, which does not make much sense to me. Acked-by: Namhyung Kim namhy...@kernel.org Cc: Arnaldo Carvalho de Melo a...@kernel.org Cc: Corey Ashford

[PATCH 3/9] perf tools: Consider header files outside perf directory in tags target

2014-06-03 Thread Jiri Olsa
From: Sebastian Andrzej Siewior bige...@linutronix.de This fixes lookups like vi -t event_format Signed-off-by: Sebastian Andrzej Siewior bige...@linutronix.de Link: http://lkml.kernel.org/r/20140528081918.ga28...@linutronix.de Signed-off-by: Jiri Olsa jo...@kernel.org ---

Re: [PATCH RFC - TAKE TWO - 00/12] New version of the BFQ I/O Scheduler

2014-06-03 Thread Pavel Machek
Hi! Well, it's all about how to actually route the changes and in general whenever avoidable we try to avoid whole-sale code replacement especially when most of the structural code is similar like in this case. Gradually evolving cfq to bfq is likely to take more work but I'm very

Re: [PATCH 2/2] block,scsi: convert and handle ERR_PTR from blk_get_request

2014-06-03 Thread Jeff Moyer
Joe Lawrence joe.lawre...@stratus.com writes: diff --git a/block/bsg.c b/block/bsg.c index 420a5a9..2b1c322 100644 --- a/block/bsg.c +++ b/block/bsg.c @@ -271,8 +271,8 @@ bsg_map_hdr(struct bsg_device *bd, struct sg_io_v4 *hdr, fmode_t has_write_perm, * map scatter-gather elements

Re: [RFC][PATCH 4/4] crypto: Updated makefiles to add QAT

2014-06-03 Thread Tadeusz Struk
On 06/03/2014 12:16 PM, Randy Dunlap wrote: +source drivers/crypto/qat/Kconfig Missing that file ^^^ Hi, Something went wrong. I need to resend v2. Thanks Randy -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to

Re: [PATCH 2/2] block,scsi: convert and handle ERR_PTR from blk_get_request

2014-06-03 Thread Joe Lawrence
On Tue, 3 Jun 2014 15:45:36 -0400 Jeff Moyer jmo...@redhat.com wrote: Joe Lawrence joe.lawre...@stratus.com writes: The blk_get_request function may fail in low-memory conditions or during device removal (even if __GFP_WAIT is set). To distinguish between these errors, modify the

Re: [PATCH RFC 1/2] MAINTAINERS: Add R: designated-reviewers tag

2014-06-03 Thread Theodore Ts'o
On Tue, Jun 03, 2014 at 01:43:47PM -0400, Steven Rostedt wrote: I know from experience that a quick 15 minute run on xfstests on a ramdisk will catch 99% of typical problems a filesystem patch might introduce. Code coverage reporting (done recently by RH QE engineers) tells me that this

Re: Pointers on debugging my machine not booting

2014-06-03 Thread Bruno Prémont
Hey Gideon, On Thu, 29 May 2014 Gideon D'souza gidisr...@gmail.com wrote: Hey guys, So I usually have the latest mainline kernel (on my fedora box) I've recently sent it tiny refactoring patches. What kind of box is it? Today, I start my machine and it just doesn't start. I had 3.11,

Re: [PATCH v2 net-next 0/2] split BPF out of core networking

2014-06-03 Thread Alexei Starovoitov
On Tue, Jun 3, 2014 at 1:35 PM, Daniel Borkmann dbork...@redhat.com wrote: On 06/03/2014 05:44 PM, Alexei Starovoitov wrote: ... All of your points are valid. They are right questions to ask. I just don't see why you're still arguing about first step of filter.c split, whereas your concerns

Re: [PATCH] tty: serial: msm: Fix section mismatch warning

2014-06-03 Thread Olof Johansson
On Tue, Jun 3, 2014 at 1:33 PM, Greg KH gre...@linuxfoundation.org wrote: On Tue, Jun 03, 2014 at 03:13:22PM -0500, Kumar Gala wrote: WARNING: drivers/tty/built-in.o(.data+0x3544): Section mismatch in reference from the variable msm_platform_driver to the function .init.text:msm_serial_probe()

[git pull] FireWire updates post v3.15

2014-06-03 Thread Stefan Richter
Linus, please pull from the tag firewire-updates at git://git.kernel.org/pub/scm/linux/kernel/git/ieee1394/linux1394.git firewire-updates to receive the following IEEE 1394 (FireWire) subsystem changes: One optimization for some VIA controllers, one fix, one kconfig brushup. Daeseok Youn

Re: [PATCH] tty: serial: msm: Fix section mismatch warning

2014-06-03 Thread Greg KH
On Tue, Jun 03, 2014 at 02:04:43PM -0700, Olof Johansson wrote: On Tue, Jun 3, 2014 at 1:33 PM, Greg KH gre...@linuxfoundation.org wrote: On Tue, Jun 03, 2014 at 03:13:22PM -0500, Kumar Gala wrote: WARNING: drivers/tty/built-in.o(.data+0x3544): Section mismatch in reference from the

Re: [PATCH v6 net-next 1/4] net: flow_dissector: avoid multiple calls in eBPF

2014-06-03 Thread Chema Gonzalez
On Tue, Jun 3, 2014 at 1:15 PM, Alexei Starovoitov a...@plumgrid.com wrote: On Tue, Jun 3, 2014 at 1:33 AM, Daniel Borkmann dbork...@redhat.com wrote: On 06/02/2014 06:48 PM, Alexei Starovoitov wrote: imo there are pros and cons in Daniel's and Chema's proposals for classic BPF extensions. I

Re: [BUG] signal: sighand unprotected when accessed by /proc

2014-06-03 Thread Thomas Gleixner
On Tue, 3 Jun 2014, Steven Rostedt wrote: On Tue, 3 Jun 2014 22:09:38 +0200 Oleg Nesterov o...@redhat.com wrote: Now, if that lock is released and reused (I didn't trace other tasks allocating these locks), it reinitializes the lock-wait_list. How? From where? This should be

Re: [PATCH] tty: serial: msm: Fix section mismatch warning

2014-06-03 Thread Olof Johansson
On Tue, Jun 3, 2014 at 2:14 PM, Greg KH gre...@linuxfoundation.org wrote: On Tue, Jun 03, 2014 at 02:04:43PM -0700, Olof Johansson wrote: On Tue, Jun 3, 2014 at 1:33 PM, Greg KH gre...@linuxfoundation.org wrote: On Tue, Jun 03, 2014 at 03:13:22PM -0500, Kumar Gala wrote: WARNING:

[PATCH] usb: phy: phy-gpio-vbus-usb: use devm_ functions

2014-06-03 Thread Himangi Saraogi
The various devm_ functions allocate memory that is released when a driver detaches. This patch uses devm_kzalloc, devm_request_irq, devm_gpio_request, devm_regulator_get etc. for data that is allocated in the probe function of a platform device and is only freed in the remove function. The

[PATCH 0/2] Fix double clk_diable and use _devm functions

2014-06-03 Thread Himangi Saraogi
The first patch fixed the bug of calling clk_disable when the clk is not enabled and the second patch uses devm function and removes the corresponding free and error handling code and does away with all labels in the probe function. Himangi Saraogi (2): usb: gadget: mv_u3d_core: Correct

[PATCH 1/2] usb: gadget: mv_u3d_core: Correct clk_disable twice

2014-06-03 Thread Himangi Saraogi
There is a call to clk_disable a few lines before the goto err_alloc_ep_context. The clk goes not get enabled anywhere after that so this is not required. This patch fixes the bug by removing the deinit and clk_disable after the goto statement. Signed-off-by: Himangi Saraogi himangi...@gmail.com

Re: [PATCH v2 1/4] dma-mapping: Add devm_ interface for dma_map_single()

2014-06-03 Thread Shuah Khan
On 06/01/2014 01:01 AM, Eli Billauer wrote: dmam_map_single() and dmam_unmap_single() are the managed counterparts for the respective dma_* functions. Note that dmam_map_single() returns a status value rather than the DMA handle. The DMA handle is passed to the caller through a pointer in the

[PATCH 2/2] usb: gadget: mv_u3d_core: use devm_ functions

2014-06-03 Thread Himangi Saraogi
The various devm_ functions allocate memory that is released when a driver detaches. This patch uses devm_kzalloc, devm_request_irq, devm_ioremap, dmam_pool_destroy, dmam_alloc_coherent, dmam_pool_create etc. for data that is allocated in the probe function of a platform device and is only freed

[PATCH v3] devicetree: bindings: separate CPU enable method descriptions

2014-06-03 Thread Alex Elder
The bindings for CPU enable methods are defined in .../arm/cpus.txt. As additional 32-bit ARM CPUS are converted to use the enable-method CPU property to imply a particular set of SMP operations to use, the list of these methods is likely to become unwieldy. The current documentation already

Re: [PATCH 2/2] block,scsi: convert and handle ERR_PTR from blk_get_request

2014-06-03 Thread Joe Lawrence
On Tue, 3 Jun 2014 16:07:01 -0400 Jeff Moyer jmo...@redhat.com wrote: Joe Lawrence joe.lawre...@stratus.com writes: diff --git a/block/bsg.c b/block/bsg.c index 420a5a9..2b1c322 100644 --- a/block/bsg.c +++ b/block/bsg.c @@ -271,8 +271,8 @@ bsg_map_hdr(struct bsg_device *bd, struct

[PATCH 0/2] ARM: sa1100: collie mtd fixes

2014-06-03 Thread Andrea Adami
After having extensivly tested three different units (2 Zaurus SL-5500, 1 SL-5500G) I finally send this small patches fixing collie's mtd. The background is the improved support for the specific chips, see pending: mtd: cfi_cmdset_0001.c: add support for Sharp LH28F640BF NOR for linux-mtd. The

<    3   4   5   6   7   8   9   10   11   12   >