From patchwork Thu Mar 15 08:19:58 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Lin Ming X-Patchwork-Id: 146849 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 7E50EB6F62 for ; Thu, 15 Mar 2012 19:20:10 +1100 (EST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758822Ab2COIUI (ORCPT ); Thu, 15 Mar 2012 04:20:08 -0400 Received: from mga01.intel.com ([192.55.52.88]:42620 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756317Ab2COIUA (ORCPT ); Thu, 15 Mar 2012 04:20:00 -0400 Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by fmsmga101.fm.intel.com with ESMTP; 15 Mar 2012 01:20:00 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.71,315,1320652800"; d="scan'208";a="139189481" Received: from minggr.sh.intel.com (HELO [10.239.36.45]) ([10.239.36.45]) by fmsmga002.fm.intel.com with ESMTP; 15 Mar 2012 01:19:59 -0700 Subject: Re: DMA doesn't work since "make ata port as parent device of scsi host" From: Lin Ming To: =?ISO-8859-1?Q?J=F6rg?= Sommer Cc: Jeff Garzik , Tejun Heo , linux-ide@vger.kernel.org In-Reply-To: <20120315074835.GA18295@alea.gnuu.de> References: <20120315014807.GA3051@alea.gnuu.de> <1331776741.3436.33.camel@minggr> <1331781968.3436.43.camel@minggr> <20120315074835.GA18295@alea.gnuu.de> Date: Thu, 15 Mar 2012 16:19:58 +0800 Message-ID: <1331799598.3436.46.camel@minggr> Mime-Version: 1.0 X-Mailer: Evolution 2.30.3 Sender: linux-ide-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-ide@vger.kernel.org On Thu, 2012-03-15 at 08:48 +0100, Jörg Sommer wrote: > Lin Ming hat am Thu 15. Mar, 11:26 (+0800) geschrieben: > > On Thu, 2012-03-15 at 09:59 +0800, Lin Ming wrote: > > > On Thu, 2012-03-15 at 02:48 +0100, Jörg Sommer wrote: > > > > Hi, > > > > > > > > I'm getting these messages in a KVM virtualized host and the access to > > > > the disks is very slow. Using libata.dma=0 suppresses the warnings, but > > > > the disks are still slow. > > > > > > Hi Jörg, > > > > > > Let me try to reproduce this issue first. > > > > I tried below commands on x86_32, but can't reproduce it. > > > > qemu-system-i386 -kernel /root/vmlinuz-3.3.0-rc7 -append "root=/dev/sda1 > > zcache" -hda /root/debian-32.img -hdb /root/data.img > > > > Maybe because i386 uses different ata controller than ppc(ata_piix vs > > pata-macio). > > > > I'll try qemu-system-ppc. > > > > Did you need to run some workload to trigger these warings? > > Or did you get these warnings right after booting the VM? > > These warnings come up before »INIT started«. I don't have to do > anything, just wait and see. > > # truncate -s100M /mnt/data/new > # mke2fs -Fq /mnt/data/new > # mount -o loop /mnt/data/new /mnt/other > # cp --parents /bin/zsh-static /mnt/other/zsh > # umount /mnt/other > # qemu-system-ppc -enable-kvm -M mac99 -cpu G4 -k de -kernel /boot/vmlinuz-3.3.0-rc5-04520-g8d233c0 -append 'root=/dev/sda ro console=ttyPZ0 init=/zsh' -hda /mnt/data/new > > I've uploaded my config and the kernel: > http://alioth.debian.org/~jo-guest/config-3.3.0-rc5-04520-g8d233c0 > http://alioth.debian.org/~jo-guest/vmlinuz-3.3.0-rc5-04520-g8d233c0 > > You have to switch to the serial console (Ctrl-Alt-3 or with -nographic > Ctrl-a c). All kernel messages arrive there. > > > Would you please try to disable ata port runtime pm? > > You can disable it by, for example, > > Is this also possible with a kernel parameter? It takes very long until I > get a shell prompt. No, but you can try below debug patch to disable port runtime pm. > > Bye, Jörg. --- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/drivers/ata/libata-transport.c b/drivers/ata/libata-transport.c index 74aaee3..8def3fc 100644 --- a/drivers/ata/libata-transport.c +++ b/drivers/ata/libata-transport.c @@ -293,7 +293,6 @@ int ata_tport_add(struct device *parent, device_enable_async_suspend(dev); pm_runtime_set_active(dev); - pm_runtime_enable(dev); transport_add_device(dev); transport_configure_device(dev);