From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934720AbYETW7R (ORCPT ); Tue, 20 May 2008 18:59:17 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1762022AbYETW66 (ORCPT ); Tue, 20 May 2008 18:58:58 -0400 Received: from pentafluge.infradead.org ([213.146.154.40]:48740 "EHLO pentafluge.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758427AbYETW65 (ORCPT ); Tue, 20 May 2008 18:58:57 -0400 Date: Tue, 20 May 2008 15:58:47 -0700 From: Arjan van de Ven To: , linux-kernel@vger.kernel.org Subject: [PATCH] ata: fix sleep-while-holding-spinlock in sata_nv Message-ID: <20080520155847.12a22dfb@infradead.org> Organization: Intel X-Mailer: Claws Mail 3.3.1 (GTK+ 2.12.9; i386-redhat-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-SRS-Rewrite: SMTP reverse-path rewritten from by pentafluge.infradead.org See http://www.infradead.org/rpr.html Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Arjan van de Ven Subject: [PATCH] ata: fix sleep-while-holding-spinlock in sata_nv blk_queue_bounce_limit() is a sleeping function, so reorganize the code a little to not call it while holding a spinlock. Signed-off-by: Arjan van de Ven --- drivers/ata/sata_nv.c | 8 +++++--- 1 files changed, 5 insertions(+), 3 deletions(-) diff --git a/drivers/ata/sata_nv.c b/drivers/ata/sata_nv.c index 858f706..a7bc56d 100644 --- a/drivers/ata/sata_nv.c +++ b/drivers/ata/sata_nv.c @@ -671,6 +671,11 @@ static int nv_adma_slave_config(struct scsi_device *sdev) port1 = ap->host->ports[1]->private_data; sdev0 = ap->host->ports[0]->link.device[0].sdev; sdev1 = ap->host->ports[1]->link.device[0].sdev; + + blk_queue_segment_boundary(sdev->request_queue, segment_boundary); + blk_queue_max_hw_segments(sdev->request_queue, sg_tablesize); + spin_unlock_irqrestore(ap->lock, flags); + if ((port0->flags & NV_ADMA_ATAPI_SETUP_COMPLETE) || (port1->flags & NV_ADMA_ATAPI_SETUP_COMPLETE)) { /** We have to set the DMA mask to 32-bit if either port is in @@ -701,14 +706,11 @@ static int nv_adma_slave_config(struct scsi_device *sdev) pp->adma_dma_mask); } - blk_queue_segment_boundary(sdev->request_queue, segment_boundary); - blk_queue_max_hw_segments(sdev->request_queue, sg_tablesize); ata_port_printk(ap, KERN_INFO, "DMA mask 0x%llX, segment boundary 0x%lX, hw segs %hu\n", (unsigned long long)*ap->host->dev->dma_mask, segment_boundary, sg_tablesize); - spin_unlock_irqrestore(ap->lock, flags); return rc; } -- 1.5.4.5