[PATCH 0/5] Multiqueue virtio-scsi

2012-08-28 Thread Paolo Bonzini
Hi all, this series adds multiqueue support to the virtio-scsi driver, based on Jason Wang's work on virtio-net. It uses a simple queue steering algorithm that expects one queue per CPU. LUNs in the same target always use the same queue (so that commands are not reordered); queue switching

[PATCH 1/5] virtio-ring: move queue_index to vring_virtqueue

2012-08-28 Thread Paolo Bonzini
From: Jason Wang jasow...@redhat.com Instead of storing the queue index in transport-specific virtio structs, this patch moves them to vring_virtqueue and introduces an helper to get the value. This lets drivers simplify their management and tracing of virtqueues. Signed-off-by: Jason Wang

[PATCH 2/5] virtio: introduce an API to set affinity for a virtqueue

2012-08-28 Thread Paolo Bonzini
From: Jason Wang jasow...@redhat.com Sometimes, virtio device need to configure irq affinity hint to maximize the performance. Instead of just exposing the irq of a virtqueue, this patch introduce an API to set the affinity for a virtqueue. The api is best-effort, the affinity hint may not be

[PATCH 3/5] virtio-scsi: allocate target pointers in a separate memory block

2012-08-28 Thread Paolo Bonzini
We will place the request virtqueues in the flexible array member. Refining the virtqueue API would let us drop the sglist copy, at which point the pointer-to-array-of-pointers can become a simple pointer-to-array. It would both simplify the allocation and remove a dereference in several hot

[PATCH 4/5] virtio-scsi: pass struct virtio_scsi to virtqueue completion function

2012-08-28 Thread Paolo Bonzini
This will be needed soon in order to retrieve the per-target struct. Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- drivers/scsi/virtio_scsi.c | 17 + 1 files changed, 9 insertions(+), 8 deletions(-) diff --git a/drivers/scsi/virtio_scsi.c b/drivers/scsi/virtio_scsi.c

[PATCH 5/5] virtio-scsi: introduce multiqueue support

2012-08-28 Thread Paolo Bonzini
This patch adds queue steering to virtio-scsi. When a target is sent multiple requests, we always drive them to the same queue so that FIFO processing order is kept. However, if a target was idle, we can choose a queue arbitrarily. In this case the queue is chosen according to the current VCPU,

Re: [PATCH v3 01/17] hashtable: introduce a small and naive hashtable

2012-08-28 Thread Mathieu Desnoyers
* Sasha Levin (levinsasha...@gmail.com) wrote: On 08/28/2012 12:11 PM, Mathieu Desnoyers wrote: * Sasha Levin (levinsasha...@gmail.com) wrote: On 08/25/2012 06:24 AM, Mathieu Desnoyers wrote: * Tejun Heo (t...@kernel.org) wrote: Hello, On Sat, Aug 25, 2012 at 12:59:25AM +0200, Sasha

[PATCH for 3.6 0/3] urgent virtio-scsi fixes

2012-08-28 Thread Paolo Bonzini
James, here are three important bugfixes for virtio-scsi. All three have been on the list for a while, I'm grouping them in this post to ease your processing. Two of them are also marked for stable branches. Paolo Paolo Bonzini (1): virtio-scsi: fix LUNs greater than 255 Richard W.M. Jones

[PATCH for 3.6 2/3] virtio-scsi: initialize scatterlist structure for events

2012-08-28 Thread Paolo Bonzini
From: Richard W.M. Jones rjo...@redhat.com The sg struct for events is used without being initialized. This triggers a BUG_ON if CONFIG_DEBUG_SG is defined. This was reported at https://bugzilla.redhat.com/show_bug.cgi?id=847548. Signed-off-by: Richard W.M. Jones rjo...@redhat.com

Re: [PATCH] hwmon/lm70: adding support for NS LM74 chip

2012-08-28 Thread leroy christophe
Le 26/08/2012 19:47, Guenter Roeck a écrit : On Thu, Aug 23, 2012 at 11:13:17AM -0700, Guenter Roeck wrote: On Thu, Aug 23, 2012 at 05:32:12PM +0200, Christophe Leroy wrote: Hello, Hi Christophe, Hi again, [ ... ] Hi Guenter, - /* 3-wire link (shared SI/SO) for LM70 */ - if

[PATCH] usb: otg: Move phy interface to separate file.

2012-08-28 Thread Venu Byravarasu
As otg.h is containing lots of phy interface related stuff, moving all phy interface related stuff to new file named phy.h Signed-off-by: Venu Byravarasu vbyravar...@nvidia.com --- include/linux/usb/otg.h | 211 + include/linux/usb/phy.h | 222

RE: [PATCH] usb: tegra: moving phy driver into drivers directory

2012-08-28 Thread Venu Byravarasu
-Original Message- From: Felipe Balbi [mailto:ba...@ti.com] Sent: Tuesday, August 28, 2012 3:19 PM To: Venu Byravarasu Cc: ccr...@android.com; o...@lixom.net; swar...@wwwdotorg.org; li...@arm.linux.org.uk; st...@rowland.harvard.edu; gre...@linuxfoundation.org; ba...@ti.com;

Re: [PATCH] regulator: disable supply regulator if it is enabled for boot-on

2012-08-28 Thread Laxman Dewangan
On Saturday 25 August 2012 05:10 AM, Rabin Vincent wrote: On Fri, Aug 24, 2012 at 11:22:05PM +0530, Laxman Dewangan wrote: I tried to reproduce the issue but could not able to do this. Can you please send me your board/dt files where you are porviding platform data for regulator? This will help

[PATCH v2 1/2] virtio-ring: Use threshold for switching to indirect descriptors

2012-08-28 Thread Sasha Levin
Currently if VIRTIO_RING_F_INDIRECT_DESC is enabled we will use indirect descriptors even if we have plenty of space in the ring. This means that we take a performance hit at all times due to the overhead of creating indirect descriptors. Instead, use it only after we're below a configurable

[PATCH v2 2/2] virtio-ring: Allocate indirect buffers from cache when possible

2012-08-28 Thread Sasha Levin
Currently if VIRTIO_RING_F_INDIRECT_DESC is enabled we will use indirect descriptors and allocate them using a simple kmalloc(). This patch adds a cache which will allow indirect buffers under a configurable size to be allocated from that cache instead. Signed-off-by: Sasha Levin

Re: [PATCH] fuse: mark variables uninitialized

2012-08-28 Thread Miklos Szeredi
On Tue, 2012-08-28 at 10:38 +0200, Daniel Mack wrote: gcc 4.6.3 complains about uninitialized variables in fs/fuse/control.c: CC fs/fuse/control.o fs/fuse/control.c: In function ‘fuse_conn_congestion_threshold_write’: fs/fuse/control.c:165:29: warning: ‘val’ may be used uninitialized

[PATCH] staging: rtl8192e: remove pointless conditional before kfree_skb()

2012-08-28 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove pointless conditional before kfree_skb(). Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging/rtl8192e/rtl8192e/rtl_core.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

[PATCH] staging: ozwpan: remove pointless conditional before kfree_skb()

2012-08-28 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove pointless conditional before kfree_skb(). Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging/ozwpan/ozpd.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/ozwpan/ozpd.c

[PATCH] audit: remove pointless conditional before kfree_skb()

2012-08-28 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove pointless conditional before kfree_skb(). Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- kernel/audit.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/kernel/audit.c b/kernel/audit.c index

Re: [PATCH v2 1/2] virtio-ring: Use threshold for switching to indirect descriptors

2012-08-28 Thread Michael S. Tsirkin
On Tue, Aug 28, 2012 at 03:04:02PM +0200, Sasha Levin wrote: Currently if VIRTIO_RING_F_INDIRECT_DESC is enabled we will use indirect descriptors even if we have plenty of space in the ring. This means that we take a performance hit at all times due to the overhead of creating indirect

Re: [PATCH v2 2/2] virtio-ring: Allocate indirect buffers from cache when possible

2012-08-28 Thread Michael S. Tsirkin
On Tue, Aug 28, 2012 at 03:04:03PM +0200, Sasha Levin wrote: Currently if VIRTIO_RING_F_INDIRECT_DESC is enabled we will use indirect descriptors and allocate them using a simple kmalloc(). This patch adds a cache which will allow indirect buffers under a configurable size to be allocated

[PATCH 00/14] aoe driver v49 performance and usability improvements

2012-08-28 Thread Ed Cashin
In this resubmission of the patchset, several suggestions from Andrew Morton have been incorporated to clean up the changes. I will not be able to be very responsive during the first part of September, so the sooner you can provide feedback, the better. These patches go a long way to updating

[PATCH 01/14] aoe: for performance support larger packet payloads

2012-08-28 Thread Ed Cashin
This patch adds the ability to work with large packets composed of a number of segments, using the scatter gather feature of the block layer (biovecs) and the network layer (skb frag array). The motivation is the performance gained by using a packet data payload greater than a page size and by

[PATCH 02/14] aoe: kernel thread handles I/O completions for simple locking

2012-08-28 Thread Ed Cashin
This patch makes the frames the aoe driver uses to track the relationship between bios and packets more flexible and detached, so that they can be passed to an aoe_ktio thread for completion of I/O. The frames are handled much like skbs, with a capped amount of preallocation so that real-world

[PATCH 03/14] aoe: become I/O request queue handler for increased user control

2012-08-28 Thread Ed Cashin
To allow users to choose an elevator algorithm for their particular workloads, change from a make_request-style driver to an I/O-request-queue-handler-style driver. We have to do a couple of things that might be surprising. We manipulate the page _count directly on the assumption that we still

[PATCH 04/14] aoe: use a kernel thread for transmissions

2012-08-28 Thread Ed Cashin
The dev_queue_xmit function needs to have interrupts enabled, so the most simple way to get the locking right but still fulfill that requirement is to use a process that can call dev_queue_xmit serially over queued transmissions. Signed-off-by: Ed Cashin ecas...@coraid.com ---

[PATCH 05/14] aoe: use packets that work with the smallest-MTU local interface

2012-08-28 Thread Ed Cashin
Users with several network interfaces dedicated to AoE generally do not configure them to support different-sized AoE data payloads on purpose. For a given AoE target, there will be a set of local network interfaces that can reach it. Using only the payload that will fit in the smallest-sized

[PATCH 06/14] aoe: failover remote interface based on aoe_deadsecs parameter

2012-08-28 Thread Ed Cashin
The aoe_deadsecs module parameter allows the user to specify a hard limit on the number of seconds an AoE command can be retransmitted before the AoE block device is considered to have failed. Using aoe_deadsecs to determine the time we try using a different remote interface helps to ensure that

Re: [PATCH v2 2/2] virtio-ring: Allocate indirect buffers from cache when possible

2012-08-28 Thread Sasha Levin
On 08/28/2012 03:20 PM, Michael S. Tsirkin wrote: On Tue, Aug 28, 2012 at 03:04:03PM +0200, Sasha Levin wrote: Currently if VIRTIO_RING_F_INDIRECT_DESC is enabled we will use indirect descriptors and allocate them using a simple kmalloc(). This patch adds a cache which will allow indirect

[PATCH 07/14] aoe: do revalidation steps in order

2012-08-28 Thread Ed Cashin
The discovery process begins with an optional AoE config query command and an AoE config query response. Normally when an aoe device is already open, the config query response does not trigger an ATA identify device command to be sent out, since the response contains storage capacity information

[PATCH 08/14] aoe: disallow unsupported AoE minor addresses

2012-08-28 Thread Ed Cashin
A guard is inserted to prevent AoE minor addresses (slot addresses) higher than 15 to be used, as they are not yet supported by the driver. There is a change coming that will allow the aoe driver to overcome this limit by using system device minor numbers dynamically, but until then, this guard

[PATCH 09/14] aoe: associate frames with the AoE storage target

2012-08-28 Thread Ed Cashin
In the driver code, target and aoetgt refer to a particular remote interface on the AoE storage target. The latter is identified by its AoE major and minor addresses. Commands that are being sent to an AoE storage target {major, minor} can be sent or retransmitted to any of the remote MAC

[PATCH 10/14] aoe: increase net_device reference count while using it

2012-08-28 Thread Ed Cashin
This change eliminates the danger that the user could rmmod the driver for a network interface that is being used for AoE by the aoe driver. Signed-off-by: Ed Cashin ecas...@coraid.com --- drivers/block/aoe/aoecmd.c |4 drivers/block/aoe/aoedev.c |7 +++ 2 files changed, 11

[PATCH 11/14] aoe: remove unused code and add cosmetic improvements

2012-08-28 Thread Ed Cashin
This change removes some unused code and attempts to increase code consistency. Signed-off-by: Ed Cashin ecas...@coraid.com --- drivers/block/aoe/aoe.h| 10 +++--- drivers/block/aoe/aoechr.c |1 + drivers/block/aoe/aoecmd.c | 13 - drivers/block/aoe/aoenet.c |3

[PATCH 12/14] aoe: update internal version number to 49

2012-08-28 Thread Ed Cashin
The internal version number of the aoe driver appears in a console message when the driver loads and is usually obtained by the user with the userland aoe-version tool, part of the aoetools.[1] Although this patchset includes bugfixes backported from higher-numbered versions published on the

[PATCH 13/14] aoe: update copyright year in touched files

2012-08-28 Thread Ed Cashin
Signed-off-by: Ed Cashin ecas...@coraid.com --- drivers/block/aoe/aoe.h |2 +- drivers/block/aoe/aoeblk.c |2 +- drivers/block/aoe/aoechr.c |2 +- drivers/block/aoe/aoecmd.c |2 +- drivers/block/aoe/aoedev.c |2 +- drivers/block/aoe/aoemain.c |2 +-

[PATCH 14/14] aoe: update documentation with new URL and VM settings reference

2012-08-28 Thread Ed Cashin
The old area has a new URL. Also, now that the driver can perform better, it is worth mentioning the VM settings that help aoe to sink dirty pages out early, avoiding unecessary memory pressure when much I/O is going on. Signed-off-by: Ed Cashin ecas...@coraid.com --- Documentation/aoe/aoe.txt

Re: Add rq-nr_uninterruptible count to dest cpu's rq while CPU goes down.

2012-08-28 Thread Paul E. McKenney
On Tue, Aug 28, 2012 at 12:57:09PM +0600, Rakib Mullick wrote: Hello Paul, On 8/28/12, Paul E. McKenney paul...@linux.vnet.ibm.com wrote: On Mon, Aug 20, 2012 at 09:26:57AM -0700, Paul E. McKenney wrote: On Mon, Aug 20, 2012 at 11:26:57AM +0200, Peter Zijlstra wrote: How about the

Re: [PATCH] Remove CONFIG_EXPERIMENTAL

2012-08-28 Thread Rusty Russell
Kees Cook keesc...@chromium.org writes: This config item has not carried much meaning for a while now and is almost always enabled by default. Remove it and adjust various config logic and documentation. Acked-by: Rusty Russell ru...@rustcorp.com.au (lguest part) Cheers, Rusty. -- To

Re: [PATCH] usb: otg: Move phy interface to separate file.

2012-08-28 Thread Felipe Balbi
Hi, On Tue, Aug 28, 2012 at 05:50:11PM +0530, Venu Byravarasu wrote: As otg.h is containing lots of phy interface related stuff, moving all phy interface related stuff to new file named phy.h Signed-off-by: Venu Byravarasu vbyravar...@nvidia.com this should be ok.. I'll apply it once

[PATCH] Chinese translation of Documentation/gpio.txt

2012-08-28 Thread Ninja Tekkaman
This is a Chinese translated version of Documentation/gpio.txt Signed-off-by: Fu Wei tekkamanni...@gmail.com --- Documentation/zh_CN/gpio.txt | 662 ++ 1 file changed, 662 insertions(+) create mode 100644 Documentation/zh_CN/gpio.txt diff --git

Re: Possible mptsas regression post 3.5.0

2012-08-28 Thread John Drescher
I wonder if we are preventing scsi_device_dev_release_usercontext() from making forward progress? ...the attached patch should confirm this or give more info otherwise. [ 148.960318] console [netcon0] enabled [ 148.960363] netconsole: network logging started [ 170.415487] mptbase: ioc0:

Lines for locations sizing information (from Kernel Summit 2012 discussion)

2012-08-28 Thread Jason Wessel
Because I had not done any size profiling since the original RFC posting, I took 20 min and ported the patch set up to the 3.6-rc3 and booted the kernel in each configuration. There are still probably a few use cases where this patch set might be useful but the general sentiment seemed that this

Re: [PATCH 3/5] virtio-scsi: allocate target pointers in a separate memory block

2012-08-28 Thread Sasha Levin
On 08/28/2012 01:54 PM, Paolo Bonzini wrote: We will place the request virtqueues in the flexible array member. Refining the virtqueue API would let us drop the sglist copy, at which point the pointer-to-array-of-pointers can become a simple pointer-to-array. It would both simplify the

Re: [PATCH] usb: tegra: moving phy driver into drivers directory

2012-08-28 Thread Stephen Warren
On 08/28/2012 02:32 AM, Venu Byravarasu wrote: In order to keep up with the USB driver files organization, moving USB phy driver from mach-tegra to drivers/USB directory. Signed-off-by: Venu Byravarasu vbyravar...@nvidia.com diff --git a/arch/arm/mach-tegra/devices.c

Re: [PATCH] Remove CONFIG_EXPERIMENTAL

2012-08-28 Thread Jeff Garzik
On Mon, Aug 27, 2012 at 5:53 PM, Kees Cook keesc...@chromium.org wrote: This config item has not carried much meaning for a while now and is almost always enabled by default. Remove it and adjust various config logic and documentation. It does have meaning... !CONFIG_EXPERIMENTAL means more

Re: [PATCH] regulator: tps6586x: register regulator even if no init data

2012-08-28 Thread Stephen Warren
On 08/28/2012 03:29 AM, Laxman Dewangan wrote: Register all TPS6586x regulators even if there is no regulator init data for platform i.e. without any user-supplied constraints. diff --git a/drivers/mfd/tps6586x.c b/drivers/mfd/tps6586x.c @@ -396,19 +396,19 @@ static struct

Re: [PATCH 3/5] virtio-scsi: allocate target pointers in a separate memory block

2012-08-28 Thread Paolo Bonzini
Il 28/08/2012 16:07, Sasha Levin ha scritto: - num_targets = sh-max_id; - for (i = 0; i num_targets; i++) { - kfree(vscsi-tgt[i]); - vscsi-tgt[i] = NULL; + if (vscsi-tgt) { + num_targets = sh-max_id; + for (i = 0; i num_targets; i++) { +

[PATCH] Chinese translation of Documentation/arm/Booting

2012-08-28 Thread Ninja Tekkaman
This is a Chinese translated version of Documentation/arm/Booting Signed-off-by: Fu Wei tekkamanni...@gmail.com --- diff --git a/Documentation/zh_CN/arm/Booting b/Documentation/zh_CN/arm/Booting new file mode 100644 index 000..e0b1673 --- /dev/null +++ b/Documentation/zh_CN/arm/Booting @@

Re: [PATCH] block: replace __getblk_slow misfix by grow_dev_page fix

2012-08-28 Thread Hugh Dickins
On Tue, 28 Aug 2012, Richard W.M. Jones wrote: On Wed, Aug 22, 2012 at 09:56:12PM -0700, Hugh Dickins wrote: [PATCH] block: replace __getblk_slow misfix by grow_dev_page fix I noticed this (second version) went upstream already. Nevertheless I tested it today and it doesn't cause a

Re: [PATCH] Chinese translation of Documentation/arm/Booting

2012-08-28 Thread PT
看了几封中文文档翻译的 patch,觉得很不错! 如果中文和英文、数字之间能有个半角空格就更好了,无论是美观度、可读性还是超链都不会有问题。 发自我的 iPod 在 2012-8-28,7:29,Ninja Tekkaman tekkamanni...@gmail.com 写道: This is a Chinese translated version of Documentation/arm/Booting Signed-off-by: Fu Wei tekkamanni...@gmail.com --- diff --git

[PATCH v2] [SCSI] mpt2sas: fix double mutex lock in NON_BLOCKING state

2012-08-28 Thread Alexey Khoroshilov
If state is NON_BLOCKING and mutex_trylock is succeed, the control flow goes to mutex_lock_interruptible() that is a deadlock. The previous version of the patch becomes obsolete after code movement in commit 913809f6. Found by Linux Driver Verification project (linuxtesting.org).

Hardening debugfs (Was Re: [PATCH] debugfs: more tightly restrict default mount mode)

2012-08-28 Thread Theodore Ts'o
On Mon, Aug 27, 2012 at 01:32:15PM -0700, Kees Cook wrote: Since the debugfs is mostly only used by root, make the default mount mode 0700. Most system owners do not need a more permissive value, but they can choose to weaken the restrictions via their fstab. Signed-off-by: Kees Cook

Re: [PATCH] hwmon/lm70: adding support for NS LM74 chip

2012-08-28 Thread Guenter Roeck
On Tue, Aug 28, 2012 at 02:09:00PM +0200, leroy christophe wrote: Le 26/08/2012 19:47, Guenter Roeck a écrit : On Thu, Aug 23, 2012 at 11:13:17AM -0700, Guenter Roeck wrote: On Thu, Aug 23, 2012 at 05:32:12PM +0200, Christophe Leroy wrote: Hello, Hi Christophe, Hi again, [ ... ] Hi

Re: Hardening debugfs (Was Re: [PATCH] debugfs: more tightly restrict default mount mode)

2012-08-28 Thread Ben Hutchings
On Tue, 2012-08-28 at 10:41 -0400, Theodore Ts'o wrote: On Mon, Aug 27, 2012 at 01:32:15PM -0700, Kees Cook wrote: Since the debugfs is mostly only used by root, make the default mount mode 0700. Most system owners do not need a more permissive value, but they can choose to weaken the

Re: Hardening debugfs (Was Re: [PATCH] debugfs: more tightly restrict default mount mode)

2012-08-28 Thread Theodore Ts'o
On Tue, Aug 28, 2012 at 07:55:58AM -0700, Ben Hutchings wrote: The problems are apparently larger than specific modules: http://lists.linux-foundation.org/pipermail/ksummit-2012-discuss/2012-July/000894.html Sure, but most of those problems require root access, or physical access to the

Re: [PATCH] regulator: tps6586x: register regulator even if no init data

2012-08-28 Thread Laxman Dewangan
On Tuesday 28 August 2012 07:48 PM, Stephen Warren wrote: - if ((devs[j].id == TPS6586X_ID_LDO_5) || - (devs[j].id == TPS6586X_ID_LDO_RTC)) - reg_idata-supply_regulator = sys_rail_name; + if

Re: [alsa-devel] [PATCH] ASoC: ams-delta: fix card initalization failure

2012-08-28 Thread Janusz Krzysztofik
On Mon, 27 Aug 2012 14:38:35 Mark Brown wrote: On Mon, Aug 27, 2012 at 11:28:30PM +0200, Janusz Krzysztofik wrote: - platform_set_drvdata(ams_delta_audio_platform_device, - ams_delta_audio_card); - - ret =

Re: 3.5-rc7: nouveau doesn't X on NVC0

2012-08-28 Thread Alexey Dobriyan
Ping! No X for me with 3.6-rc2. On Wed, Jul 18, 2012 at 12:14 AM, Alexey Dobriyan adobri...@gmail.com wrote: FB console is OK, when switching to VT7, there is no KDM prompt. Instead looping messages appear: Failed to idle channel 1 0x2634 != chid: 0x0011 PFIFO:

Re: [PATCH] Chinese translation of Documentation/gpio.txt

2012-08-28 Thread harryxiyou
2012/8/28 Ninja Tekkaman tekkamanni...@gmail.com: This is a Chinese translated version of Documentation/gpio.txt Signed-off-by: Fu Wei tekkamanni...@gmail.com Acked-by: Harry Wei harryxi...@gmail.com -- Thanks Harry Wei -- To unsubscribe from this list: send the line unsubscribe

[PATCH 0/5] Simplify code by using hotplug safe pci_get_domain_bus_and_slot()

2012-08-28 Thread Jiang Liu
Following code has a race window between pci_find_bus() and pci_get_slot() if PCI hotplug operation happens between them which removes the pci_bus. So use PCI hotplug safe interface pci_get_domain_bus_and_slot() instead, which also reduces code complexity. struct pci_bus *pci_bus =

[PATCH 1/5] PCI/IA64: simplify code by hotplug safe pci_get_domain_bus_and_slot()

2012-08-28 Thread Jiang Liu
Following code has a race window between pci_find_bus() and pci_get_slot() if PCI hotplug operation happens between them which removes the pci_bus. So use PCI hotplug safe interface pci_get_domain_bus_and_slot() instead, which also reduces code complexity. struct pci_bus *pci_bus =

[PATCH 2/5] PCI/vga: simplify code by hotplug safe pci_get_domain_bus_and_slot()

2012-08-28 Thread Jiang Liu
Following code has a race window between pci_find_bus() and pci_get_slot() if PCI hotplug operation happens between them which removes the pci_bus. So use PCI hotplug safe interface pci_get_domain_bus_and_slot() instead, which also reduces code complexity. struct pci_bus *pci_bus =

[PATCH 3/5] PCI/cpcihp: simplify code by hotplug safe pci_get_domain_bus_and_slot()

2012-08-28 Thread Jiang Liu
Following code has a race window between pci_find_bus() and pci_get_slot() if PCI hotplug operation happens between them which removes the pci_bus. So use PCI hotplug safe interface pci_get_domain_bus_and_slot() instead, which also reduces code complexity. struct pci_bus *pci_bus =

Re: [PATCH] Chinese translation of Documentation/arm/Booting

2012-08-28 Thread harryxiyou
2012/8/28 PT plati...@163.com: 看了几封中文文档翻译的 patch,觉得很不错! 如果中文和英文、数字之间能有个半角空格就更好了, 无论是美观度、可读性还是超链都不会有问题。 Hi PT, We will take actions as you said later, thanks for your sincerely suggestions. -- Thanks Harry Wei

[PATCH 4/5] PCI/IOV: simplify code by hotplug safe pci_get_domain_bus_and_slot()

2012-08-28 Thread Jiang Liu
Following code has a race window between pci_find_bus() and pci_get_slot() if PCI hotplug operation happens between them which removes the pci_bus. So use PCI hotplug safe interface pci_get_domain_bus_and_slot() instead, which also reduces code complexity. struct pci_bus *pci_bus =

[PATCH 5/5] PCI/xen-pcifront: simplify code by hotplug safe pci_get_domain_bus_and_slot()

2012-08-28 Thread Jiang Liu
Following code has a race window between pci_find_bus() and pci_get_slot() if PCI hotplug operation happens between them which removes the pci_bus. So use PCI hotplug safe interface pci_get_domain_bus_and_slot() instead, which also reduces code complexity. struct pci_bus *pci_bus =

BUG jbd2/transaction.c:332

2012-08-28 Thread genanr
I am often hitting this BUG on an OCFS2 volume shared among 6 machines. Not only does this bug cause the kernel panic on the machine it occurs on, but when it happens it block access to that volume from the other machines connected to it until the problem machine is rebooted. The machine it

Re: [PATCH] regulator: disable supply regulator if it is enabled for boot-on

2012-08-28 Thread Rabin Vincent
2012/8/28 Laxman Dewangan ldewan...@nvidia.com: I tried to reproduce the lockup issue with the following change but not seeing any lockup issue. Did you enable CONFIG_PROVE_LOCKING? Also reviewing the change, I am not seeing any call trace where the recursive locking happening. There's

Re: [PATCH v9 3/5] virtio_balloon: introduce migration primitives to balloon pages

2012-08-28 Thread Michael S. Tsirkin
On Mon, Aug 27, 2012 at 04:47:13PM -0300, Rafael Aquini wrote: On Sun, Aug 26, 2012 at 10:42:44AM +0300, Michael S. Tsirkin wrote: Reading two atomics and doing math? Result can even be negative. I did not look at use closely but it looks suspicious. Doc on atomic_read says: The read

Re: [PATCH RFC 0/3] Add guest cpu_entitlement reporting

2012-08-28 Thread Anthony Liguori
Avi Kivity a...@redhat.com writes: On 08/27/2012 02:27 PM, Michael Wolf wrote: On Mon, 2012-08-27 at 13:31 -0700, Avi Kivity wrote: On 08/27/2012 01:23 PM, Michael Wolf wrote: How would a guest know what its entitlement is? Currently the Admin/management tool setting

Re: [PATCH 3/5] x86: Only direct map addresses that are marked as E820_RAM

2012-08-28 Thread Jacob Shin
On Fri, Aug 24, 2012 at 09:54:04PM -0700, Yinghai Lu wrote: On Fri, Aug 24, 2012 at 9:24 PM, Jacob Shin jacob.s...@amd.com wrote: On Fri, Aug 24, 2012 at 06:07:01PM -0700, Yinghai Lu wrote: On Fri, Aug 24, 2012 at 4:55 PM, Jacob Shin jacob.s...@amd.com wrote: looks like you could avoid

Re: [PATCH] kernel.h: Introduce IDIV_ROUND_CLOSEST

2012-08-28 Thread Jean Delvare
Hi Guenter, On Fri, 24 Aug 2012 17:53:17 -0700, Guenter Roeck wrote: DIV_ROUND_CLOSEST returns a bad result for negative dividends: DIV_ROUND_CLOSEST(-2, 2) = 0 Most of the time this does not matter. However, in the hardware monitoring subsystem, it is often used on integers which can

Re: [PATCH -v12 02/15] resources: Add probe_resource()

2012-08-28 Thread Yinghai Lu
On Tue, Jun 26, 2012 at 3:07 PM, Yinghai Lu ying...@kernel.org wrote: On Tue, Jun 26, 2012 at 11:53 AM, Yinghai Lu ying...@kernel.org wrote: It is changed from busn_res only version, because Bjorn found that version was not holding resource_lock. Even it may be ok for busn_res not holding

Re: [PATCH 3/5] x86: Only direct map addresses that are marked as E820_RAM

2012-08-28 Thread H. Peter Anvin
The EFI runtime code we need to map, but we shouldn't use as RAM. I suspect this is also true for the ACPI areas. Jacob Shin jacob.s...@amd.com wrote: On Fri, Aug 24, 2012 at 09:54:04PM -0700, Yinghai Lu wrote: On Fri, Aug 24, 2012 at 9:24 PM, Jacob Shin jacob.s...@amd.com wrote: On Fri,

Re: Possible mptsas regression post 3.5.0

2012-08-28 Thread Dan Williams
On Tue, 2012-08-28 at 10:03 -0400, John Drescher wrote: I wonder if we are preventing scsi_device_dev_release_usercontext() from making forward progress? ...the attached patch should confirm this or give more info otherwise. [ 174.758218] scsi_remove_target[0]: reap 0:0 state: 2

Re: [PATCH v7 0/4] cgroup: add xattr support

2012-08-28 Thread Tejun Heo
Hello, On Mon, Aug 27, 2012 at 09:05:25AM -0400, Aristeu Rozanski wrote: Hi Tejun, On Fri, Aug 24, 2012 at 03:58:39PM -0700, Tejun Heo wrote: On Thu, Aug 23, 2012 at 04:53:27PM -0400, a...@redhat.com wrote: This series are a refreshed version of a patchset submitted by Li Zefan back

Re: [PATCH] kernel.h: Introduce IDIV_ROUND_CLOSEST

2012-08-28 Thread Guenter Roeck
On Tue, Aug 28, 2012 at 06:09:26PM +0200, Jean Delvare wrote: Hi Guenter, On Fri, 24 Aug 2012 17:53:17 -0700, Guenter Roeck wrote: DIV_ROUND_CLOSEST returns a bad result for negative dividends: DIV_ROUND_CLOSEST(-2, 2) = 0 Most of the time this does not matter. However, in the

Re: [PATCH] crypto: twofish - add x86_64/avx assembler implementation

2012-08-28 Thread Borislav Petkov
On Tue, Aug 28, 2012 at 12:17:43PM +0300, Jussi Kivilinna wrote: With this patch twofish-avx is faster than twofish-3way for 256, 1k and 8k tests. sizeold-vs-new new-vs-3way old-vs-3way ecb-enc ecb-dec ecb-enc ecb-dec ecb-enc ecb-dec 256 1.10x 1.11x 1.01x

Re: [PATCH v9 1/5] mm: introduce a common interface for balloon pages mobility

2012-08-28 Thread Michael S. Tsirkin
On Mon, Aug 27, 2012 at 05:28:35PM -0300, Rafael Aquini wrote: On Sun, Aug 26, 2012 at 10:55:58AM +0300, Michael S. Tsirkin wrote: On Sat, Aug 25, 2012 at 02:24:56AM -0300, Rafael Aquini wrote: Memory fragmentation introduced by ballooning might reduce significantly the number of 2MB

Re: lockdep trace from posix timers

2012-08-28 Thread Peter Zijlstra
On Fri, 2012-08-24 at 20:56 +0200, Oleg Nesterov wrote: Peter, if you think it can work for you and if you agree with the implementation I will be happy to send the patch. Yeah I think it would work, but I'm not sure why you're introducing the cmp_xchg helper just for this.. Anyway, how

[PATCH v2 1/2] linux/kernel.h: Introduce IDIV_ROUND_CLOSEST

2012-08-28 Thread Guenter Roeck
DIV_ROUND_CLOSEST returns a bad result for negative dividends: DIV_ROUND_CLOSEST(-2, 2) = 0 Most of the time this does not matter. However, in the hardware monitoring subsystem, it is sometimes used on integers which can be negative (such as temperatures). Introduce new macro

[PATCH 2/2] hwmon: Use IDIV_ROUND_CLOSEST if the dividend can be negative

2012-08-28 Thread Guenter Roeck
DIV_ROUND_CLOSEST returns bad values for negative dividends. Replace with IDIV_ROUND_CLOSEST where this can happen. Cc: Jean Delvare kh...@linux-fr.org Cc: Dirk Eibach eib...@gdsys.de Cc: Steve Glendinning steve.glendinn...@shawell.net Signed-off-by: Guenter Roeck li...@roeck-us.net ---

Re: perf backtraces off-by-1

2012-08-28 Thread Peter Zijlstra
On Sun, 2012-08-26 at 10:52 -0700, Arun Sharma wrote: On 8/26/12 9:10 AM, Peter Zijlstra wrote: On Fri, 2012-08-24 at 15:13 -0700, Arun Sharma wrote: One option is to support this for user mode only, with code to detect signal frames. Any other ideas? I guess we'd need to see what

Re: Possible mptsas regression post 3.5.0

2012-08-28 Thread John Drescher
[ 174.758218] scsi_remove_target[0]: reap 0:0 state: 2 reap: 1 dev_del: 1 Thanks! Does the attached patch fix the issue for you? That worked! [ 169.402990] netconsole: network logging started [ 200.903700] mptbase: ioc0: LogInfo(0x31110d00): Originator={PL}, Code={Reset}, SubCode(0x0d00)

Re: [PATCH 2/6] brcmfmac: Handling the interrupt in ISR directly for non-OOB

2012-08-28 Thread Franky Lin
On 08/28/2012 04:13 AM, Wei Ni wrote: On Tue, 2012-08-28 at 04:06 +0800, Stephen Warren wrote: On 08/27/2012 09:24 AM, Arend van Spriel wrote: On 08/27/2012 12:25 PM, Wei Ni wrote: In case of inband interrupts, if we handle the interrupt in dpc thread, two level of thread switching takes

Re: [PATCH v2 02/14] MFD: twl4030-audio: Convert to use devm_kzalloc

2012-08-28 Thread Mark Brown
On Tue, Aug 28, 2012 at 12:50:10PM +0300, Tero Kristo wrote: On Tue, 2012-08-14 at 17:22 +0300, Peter Ujfalusi wrote: Signed-off-by: Peter Ujfalusi peter.ujfal...@ti.com I think this one could use a short commit message, also about why kfree():s are dropped (handled internally by devm_*

Re: [PATCH 4/6] mfd: palmas add device tree handling to mfd

2012-08-28 Thread Mark Brown
On Tue, Aug 28, 2012 at 01:47:38PM +0200, Graeme Gregory wrote: Add device tree handling to the palmas MFD. This takes the values that can be set from platform data from the device tree nodes instead. Signed-off-by: Graeme Gregory g...@slimlogic.co.uk --- drivers/mfd/palmas.c | 52

Re: [v4 0/4] ARM: dma-mapping: IOMMU atomic allocation

2012-08-28 Thread Marek Szyprowski
Hello, On 8/28/2012 7:13 AM, Hiroshi Doyu wrote: The commit e9da6e9 ARM: dma-mapping: remove custom consistent dma region breaks the compatibility with existing drivers. This causes the following kernel oops(*1). That driver has called dma_pool_alloc() to allocate memory from the interrupt

Re: Add rq-nr_uninterruptible count to dest cpu's rq while CPU goes down.

2012-08-28 Thread Rakib Mullick
On 8/28/12, Paul E. McKenney paul...@linux.vnet.ibm.com wrote: On Tue, Aug 28, 2012 at 12:57:09PM +0600, Rakib Mullick wrote: Hello Paul, On 8/28/12, Paul E. McKenney paul...@linux.vnet.ibm.com wrote: On Mon, Aug 20, 2012 at 09:26:57AM -0700, Paul E. McKenney wrote: On Mon, Aug 20, 2012 at

Re: [PATCH] [RFC] cross-arch: don't corrupt personality flags upon exec()

2012-08-28 Thread Chris Metcalf
On 8/22/2012 3:27 PM, Arnd Bergmann wrote: On tile, the PER_LINUX_32BIT is always set for 32 bit compat tasks in a 64 bit kernel, which is harmless but wrong anyway. Also this, tile never calls set_personality, which is also harmless because it just means that exec_domain switching on tile is

[GIT PULL] msm fixes for v3.6

2012-08-28 Thread David Brown
The following changes since commit 28a33cbc24e4256c143dce96c7d93bf423229f92: Linux 3.5 (2012-07-21 13:58:29 -0700) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/davidb/linux-msm.git tags/msm-fix-for3.6 for you to fetch changes up to

[PATCH v1, RFC] sheep: writeback cache semantics in backend store

2012-08-28 Thread Hitoshi Mitake
v1: differences from v0 are, * check syncfs() in configure script * send SD_OP_FLUSH_PEER to all sheeps This patch implements writeback cache semantics in backend store of sheep. Current backend store farm and plain calls open() with O_DSYNC, so every object write causes slow disk access. This

Re: [PATCH v1, RFC] sheep: writeback cache semantics in backend store

2012-08-28 Thread Hitoshi Mitake
Really sorry, I mistakenly sent below patch to LKML. Please discard this... On Wed, Aug 29, 2012 at 1:56 AM, Hitoshi Mitake h.mit...@gmail.com wrote: v1: differences from v0 are, * check syncfs() in configure script * send SD_OP_FLUSH_PEER to all sheeps This patch implements writeback cache

Re: lockdep trace from posix timers

2012-08-28 Thread Oleg Nesterov
On 08/28, Peter Zijlstra wrote: On Fri, 2012-08-24 at 20:56 +0200, Oleg Nesterov wrote: Peter, if you think it can work for you and if you agree with the implementation I will be happy to send the patch. Yeah I think it would work, but I'm not sure why you're introducing the cmp_xchg

Re: [PATCH 5/5] PCI/xen-pcifront: simplify code by hotplug safe pci_get_domain_bus_and_slot()

2012-08-28 Thread Konrad Rzeszutek Wilk
On Tue, Aug 28, 2012 at 11:43:58PM +0800, Jiang Liu wrote: Following code has a race window between pci_find_bus() and pci_get_slot() if PCI hotplug operation happens between them which removes the pci_bus. So use PCI hotplug safe interface pci_get_domain_bus_and_slot() instead, which also

Re: [PATCH -v12 02/15] resources: Add probe_resource()

2012-08-28 Thread Linus Torvalds
On Tue, Aug 28, 2012 at 9:09 AM, Yinghai Lu ying...@kernel.org wrote: please check update one. -v7 Ugh. Ok, looking closer at this, I think I agree with Bjorn. This is too ugly. The whole reduce size/needed_size by 1 double loop is O(n**2). And it does ugly insert fake resource to check, and

Re: [PATCH] MFD : MC13783 : add regulators sw1x and sw2x

2012-08-28 Thread Mark Brown
On Tue, Aug 28, 2012 at 01:09:10PM +0200, Gaëtan Carlier wrote: Signed-off-by: Gaëtan Carlier gcem...@gmail.com Applied, thanks. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at

Re: Hardening debugfs (Was Re: [PATCH] debugfs: more tightly restrict default mount mode)

2012-08-28 Thread Greg Kroah-Hartman
On Tue, Aug 28, 2012 at 10:41:10AM -0400, Theodore Ts'o wrote: On Mon, Aug 27, 2012 at 01:32:15PM -0700, Kees Cook wrote: Since the debugfs is mostly only used by root, make the default mount mode 0700. Most system owners do not need a more permissive value, but they can choose to weaken

Re: lockdep trace from posix timers

2012-08-28 Thread Oleg Nesterov
On 08/28, Oleg Nesterov wrote: On 08/28, Peter Zijlstra wrote: +again: + workp = task-task_works; + work = *workp; + while (work) { + if (work-func == func) { But you can't dereference this pointer. Without some locking this can race with another task_work_cancel()

  1   2   3   4   5   6   7   8   >