From owner-svn-src-stable@FreeBSD.ORG Fri Feb 19 13:56:21 2010 Return-Path: Delivered-To: svn-src-stable@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 9DC711065670; Fri, 19 Feb 2010 13:56:21 +0000 (UTC) (envelope-from lstewart@freebsd.org) Received: from lauren.room52.net (lauren.room52.net [210.50.193.198]) by mx1.freebsd.org (Postfix) with ESMTP id 1A6398FC14; Fri, 19 Feb 2010 13:56:21 +0000 (UTC) Received: from lawrence1.loshell.room52.net (unknown [59.167.184.191]) by lauren.room52.net (Postfix) with ESMTPSA id 6AED77E87D; Sat, 20 Feb 2010 00:56:19 +1100 (EST) Message-ID: <4B7E9882.3000703@freebsd.org> Date: Sat, 20 Feb 2010 00:56:18 +1100 From: Lawrence Stewart User-Agent: Mozilla/5.0 (X11; U; FreeBSD amd64; en-AU; rv:1.9.1.5) Gecko/20100105 Thunderbird/3.0 MIME-Version: 1.0 To: Alexander Motin References: <201002141938.o1EJcRpx065470@svn.freebsd.org> <4B7D4962.8070706@freebsd.org> <4B7D626A.9040301@feral.com> <4B7E2B94.3090605@freebsd.org> <4B7E50DC.3070103@FreeBSD.org> In-Reply-To: <4B7E50DC.3070103@FreeBSD.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Cc: svn-src-stable@FreeBSD.org, svn-src-all@FreeBSD.org, src-committers@FreeBSD.org, Matthew Jacob , svn-src-stable-8@FreeBSD.org Subject: Re: svn commit: r203889 - in stable/8/sys: cam cam/ata cam/scsi dev/ahci dev/asr dev/ata dev/ciss dev/hptiop dev/hptrr dev/mly dev/mpt dev/ppbus dev/siis dev/trm dev/twa dev/usb/storage X-BeenThere: svn-src-stable@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for all the -stable branches of the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 19 Feb 2010 13:56:21 -0000 On 02/19/10 19:50, Alexander Motin wrote: > Lawrence Stewart wrote: >> On 02/19/10 02:53, Matthew Jacob wrote: >>> Just a total swag here, but reduce the openings via camcontrol to< 32, >>> or even< 16 >> >> Thanks for the suggestion. camcontrol doesn't seem able to futz with the >> mpt controller e.g.: >> >> root@server# camcontrol identify da0 -v >> (pass0:mpt0:0:2:0): ATAPI_IDENTIFY. ACB: a1 00 00 00 00 40 00 00 00 00 >> 00 00 >> (pass0:mpt0:0:2:0): CAM status: CCB request was invalid > > IDENTIFY is an ATA command. It is not working there now, as mpt on some > level translates ATA into SCSI. `camcontrol inquiry da0` and the others > probably should work. root@server# camcontrol inquiry da0 pass0: Fixed Direct Access SCSI-2 device pass0: 300.000MB/s transfers, Command Queueing Enabled > >> However, there does appear to be a relevant sysctl: >> >> root@server# sysctl dev.mpt.0 >> dev.mpt.0.%desc: LSILogic SAS/SATA Adapter >> dev.mpt.0.%driver: mpt >> dev.mpt.0.%location: slot=3 function=0 >> dev.mpt.0.%pnpinfo: vendor=0x1000 device=0x0050 subvendor=0x1000 >> subdevice=0x3060 class=0x010000 >> dev.mpt.0.%parent: pci2 >> dev.mpt.0.debug: 3 >> dev.mpt.0.role: 1 >> dev.mpt.0.vol_member_wce: NC >> dev.mpt.0.vol_queue_depth: 128 >> dev.mpt.0.vol_resync_rate: 0 >> dev.mpt.0.nonoptimal_volumes: 0 >> >> Setting dev.mpt.0.vol_queue_depth=32, I can still trigger the >> "mpt0: mpt_cam_event: 0x16" messages by doing an svn up on the src tree. >> Same if I set depth to 16, so unfortunately it doesn't seem to do help >> in this case. > > I am not sure what this sysctl does, but it is mpt driver own sysctl. It > has nothing common with number of tags used by CAM (at least directly). Ah ok, I misunderstood what openings meant in this context... I thought it was the queue depth. So if I restore dev.mpt.0.vol_queue_depth=128 and do the following: root@server# camcontrol tags da0 (pass0:mpt0:0:2:0): device openings: 255 root@server# camcontrol tags da0 -N 64 (pass0:mpt0:0:2:0): tagged openings now 64 (pass0:mpt0:0:2:0): device openings: 64 I can still trigger the "mpt0: mpt_cam_event: 0x16" messages. Same with openings set to 32. If I lower the openings down to 16, I still get fairly large stalls in disk IO, but the mpt_cam_event messages no longer appear. Hmmmm... Cheers, Lawrence