Re: RFC: Allow block drivers to poll for I/O instead of sleeping

2013-06-23 Thread Ingo Molnar
* Matthew Wilcox wi...@linux.intel.com wrote: A paper at FAST2012 (http://static.usenix.org/events/fast12/tech/full_papers/Yang.pdf) pointed out the performance overhead of taking interrupts for low-latency block I/Os. The solution the author investigated was to spin waiting for each I/O

[PATCH 1/1] scsi: qla2xxx: change some variables to hexadecimal string via %*phC or %*phN format

2013-06-23 Thread Oleksandr Khoshaba
The patch modifies some variables using the format '%*ph[CN]' and prints them as a hexadecimal string with the separator ':' or ''. Signed-off-by: Oleksandr Khoshaba oleksandr.khosh...@gmail.com --- drivers/scsi/qla2xxx/qla_bsg.c |8 +--- drivers/scsi/qla2xxx/qla_gs.c | 82

[PATCH v3 0/4] introduce sg_pcopy_from_buffer() and sg_pcopy_to_buffer()

2013-06-23 Thread Akinobu Mita
This patch set introduces sg_pcopy_from_buffer() and sg_pcopy_to_buffer(), which copy data between a linear buffer and an SG list. The only difference between sg_pcopy_{from,to}_buffer() and sg_copy_{from,to}_buffer() is an additional argument that specifies the number of bytes to skip the SG

[PATCH v3 2/4] lib/scatterlist: introduce sg_pcopy_from_buffer() and sg_pcopy_to_buffer()

2013-06-23 Thread Akinobu Mita
The only difference between sg_pcopy_{from,to}_buffer() and sg_copy_{from,to}_buffer() is an additional argument that specifies the number of bytes to skip the SG list before copying. Signed-off-by: Akinobu Mita akinobu.m...@gmail.com Cc: Tejun Heo t...@kernel.org Cc: Imre Deak

[PATCH v3 1/4] lib/scatterlist: factor out sg_miter_get_next_page() from sg_miter_next()

2013-06-23 Thread Akinobu Mita
This function is used to proceed page iterator to the next page if necessary, and will be used to implement the variants of sg_copy_{from,to}_buffer() later. Signed-off-by: Akinobu Mita akinobu.m...@gmail.com Acked-by: Tejun Heo t...@kernel.org Cc: Tejun Heo t...@kernel.org Cc: Imre Deak

[PATCH 24/45] [SCSI] fcoe: Use get/put_online_cpus_atomic() to prevent CPU offline

2013-06-23 Thread Srivatsa S. Bhat
Once stop_machine() is gone from the CPU offline path, we won't be able to depend on disabling preemption to prevent CPUs from going offline from under us. Use the get/put_online_cpus_atomic() APIs to prevent CPUs from going offline, while invoking from atomic context. Cc: Robert Love

Re: JMicron JMB363 PCI SATA/IDE Card Support

2013-06-23 Thread Matthias Prager
I did some more digging and came up with a partial workaround: After adding the line: { PCI_VDEVICE(JMICRON, 0x236f), board_ahci_ign_iferr }, (at line 301 of drivers/ata/ahci.c) The the sata ports of my two cards get detected and lspci -k shows they are using the ahci driver. My guess is

Re: JMicron JMB363 PCI SATA/IDE Card Support

2013-06-23 Thread Matthias Prager
Looks like the RAID Mode is the default one and quirk_jmicron_ata() in drivers/pci/quirks.c is supposed to deal with it by changing the PCI device configuration ... this does not happen or does not have the desired result (maybe this is caused by working in a pass-trough environment?). - Matthias

Re: RFC: Allow block drivers to poll for I/O instead of sleeping

2013-06-23 Thread Linus Torvalds
On Sun, Jun 23, 2013 at 12:09 AM, Ingo Molnar mi...@kernel.org wrote: The spinning approach you add has the disadvantage of actively wasting CPU time, which could be used to run other tasks. In general it's much better to make sure the completion IRQs are rate-limited and just schedule. This

Re: [trivial] treewide: Add __GFP_NOWARN to k.alloc calls with v.alloc fallbacks

2013-06-23 Thread Theodore Ts'o
On Wed, Jun 19, 2013 at 12:15:53PM -0700, Joe Perches wrote: Don't emit OOM warnings when k.alloc calls fail when there there is a v.alloc immediately afterwards. Signed-off-by: Joe Perches j...@perches.com For fs/ext4/super.c: Acked-by: Theodore Ts'o ty...@mit.edu

Re: [PATCH v11 5/9] Avoid saving/restoring interrupt state inside scsi_remove_host()

2013-06-23 Thread Mike Christie
On 6/12/13 7:54 AM, Bart Van Assche wrote: Since it is not allowed to invoke scsi_remove_host() with interrupts disabled, avoid saving and restoring the interrupt state inside scsi_remove_host(). This patch does not change the functionality of the function scsi_remove_host(). Signed-off-by:

Re: [PATCH v11 6/9] Make scsi_remove_host() wait until error handling finished

2013-06-23 Thread Mike Christie
On 6/12/13 7:55 AM, Bart Van Assche wrote: A SCSI LLD may start cleaning up host resources as soon as scsi_remove_host() returns. These host resources may be needed by the LLD in an implementation of one of the eh_* functions. So if one of the eh_* functions is in progress when

Re: [PATCH v11 8/9] Save and restore host_scribble during error handling

2013-06-23 Thread James Bottomley
On Sun, 2013-06-23 at 20:21 -0500, Mike Christie wrote: On 6/12/13 7:57 AM, Bart Van Assche wrote: A SCSI LLD may overwrite host_scribble in its queuecommand() implementation. Several drivers need that field to process requests and aborts correctly. Hence this field must be saved by

Re: [PATCH v11 2/9] Remove get_device() / put_device() pair from scsi_request_fn()

2013-06-23 Thread James Bottomley
On Wed, 2013-06-12 at 14:51 +0200, Bart Van Assche wrote: Now that all scsi_request_fn() callers hold a reference on the SCSI device that function is invoked for What makes you think that this is a true statement? The usual caller is the block layer, which doesn't really know anything about