From patchwork Sun Sep 9 07:38:27 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Hugh Dickins X-Patchwork-Id: 182612 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 AA5BD2C0093 for ; Sun, 9 Sep 2012 17:40:32 +1000 (EST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752002Ab2IIHjH (ORCPT ); Sun, 9 Sep 2012 03:39:07 -0400 Received: from mail-iy0-f174.google.com ([209.85.210.174]:43803 "EHLO mail-iy0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751957Ab2IIHjG (ORCPT ); Sun, 9 Sep 2012 03:39:06 -0400 Received: by iahk25 with SMTP id k25so862561iah.19 for ; Sun, 09 Sep 2012 00:39:04 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=date:from:x-x-sender:to:cc:subject:in-reply-to:message-id :references:user-agent:mime-version:content-type; bh=rGPim4uCGSejigaGsewPXzKK4sIrlWWexZFAvZMqkA8=; b=CbLTZUzJ0RWbaLr6Fv/ZMw4ly6nhI7tJ0LhIoiQ+zsBYbmZ5iqBNnm1gk04s7JAOFW fE4OhXuxV2xbviJImCBEKYfeGcRTlMGGIJPnVjNFOS/jHki6yQeFaA9EZ7beh3zhes1v 0/+h60zhVbn0VYnmFkHPAX10jKnULRCmb5xYC1mZEkYYUVLlw42PIBl13TZJXJVTqD1w lZjvNo2l+7mkT+Vnkk8Ku7MocIpB9HNuWIkMf33GPdV2gq+vvpEneyqDNXHLdShGW1KC 24tu53Jvv6fb3bKv+YAsJvagDdIk18R6t+XdsywbASTi98HyD/5i2TwtOuvofIfxaCn6 Z3zw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=date:from:x-x-sender:to:cc:subject:in-reply-to:message-id :references:user-agent:mime-version:content-type:x-gm-message-state; bh=rGPim4uCGSejigaGsewPXzKK4sIrlWWexZFAvZMqkA8=; b=i8bewsnaVRFh1u+IMAzRhEqufUjbywvbPwi4vO5jder+PwcmZBtre2j4uYLMyEwZ+v zJwqSYOyjhe9PsW0ZJUSPy0fADh/ybIKaUdMFa0/VgejldNU8YYnekWdx2Zuq6fhCEYq xSJRDWavMcVtbFeaJXt2Ku5cUdCdryXJ8X/9Lz0TxXqxlE2yzVtNcWm/JHWV5YzBlkHl cEV4Tt4rlkWcXA8JF1ukXwy5pc1FppHLRLfqqY13oZzr4TBuyA4XFP+yyDdMxtsxG1NE IUZ5BOGfxtf+oiCDMFX2Z0EdwyCIC3N12tz0syisLKIlbRMMD+gwkD/scBhSpnLOWQXh SiAA== Received: by 10.50.7.177 with SMTP id k17mr5927610iga.27.1347176344489; Sun, 09 Sep 2012 00:39:04 -0700 (PDT) Received: by 10.50.7.177 with SMTP id k17mr5927586iga.27.1347176344196; Sun, 09 Sep 2012 00:39:04 -0700 (PDT) Received: from [192.168.1.8] (c-67-188-178-35.hsd1.ca.comcast.net. [67.188.178.35]) by mx.google.com with ESMTPS id fu4sm6211281igc.4.2012.09.09.00.39.02 (version=TLSv1/SSLv3 cipher=OTHER); Sun, 09 Sep 2012 00:39:03 -0700 (PDT) Date: Sun, 9 Sep 2012 00:38:27 -0700 (PDT) From: Hugh Dickins X-X-Sender: hugh@eggly.anvils To: Arvydas Sidorenko cc: Jeff Garzik , Zheng Liu , linux-kernel@vger.kernel.org, IDE/ATA development list Subject: Re: Storage related regression in linux-next 20120824 In-Reply-To: <503BB4BB.4080307@gmail.com> Message-ID: References: <503BA2DB.9080704@gmail.com> <503BB4BB.4080307@gmail.com> User-Agent: Alpine 2.00 (LSU 1167 2008-08-23) MIME-Version: 1.0 X-Gm-Message-State: ALoCoQlY5DfQSxdQZGejrS+faV418FNHCCOoFiVr49bOXu0J4QnEtCQdAzxSMB8ZJgRWZtAaCG1LLvhkOPrIvaZR6ZHXb36fnkgUzxHbh784mAnhrdJ8H3gxTS892Vpe6L98KPa5CUEHPPwQH5NxGPQXDxUC8mJmYfefuOcepK+zVZcmLf77PWdjBCaHhxHS7531ymfxeixB/8pwWAMwgi59DNj0cJ++Cw== Sender: linux-ide-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-ide@vger.kernel.org On Mon, 27 Aug 2012, Arvydas Sidorenko wrote: > On 08/27/2012 06:39 PM, Arvydas Sidorenko wrote: > > > Can you pastebin 'dmesg' and 'lspci'? Did this occur only once, or is > > > it reproducible? > > > > > > Jeff > > It does happen every time when booting into -next 20120824. I don't know what went into next-20120824 versus next-20120813 (which you reported to be good), but I'm seeing similar behaviour on PowerMac G5 on Thursday's mmotm based on next-20120907 - critical target error, root remounted read-only, the reboot with good kernel then has to fsck (although fsck doesn't find anything interesting to fix in my case). > > [ 11.035530] sd 0:0:0:0: [sda] > > [ 11.035533] Result: hostbyte=DID_OK driverbyte=DRIVER_SENSE > > [ 11.035535] sd 0:0:0:0: [sda] > > [ 11.035536] Sense Key : Illegal Request [current] > > [ 11.035539] sd 0:0:0:0: [sda] > > [ 11.035541] Add. Sense: Invalid field in cdb > > [ 11.035543] sd 0:0:0:0: [sda] CDB: > > [ 11.035544] Write(10): 2a 08 0b ad d1 78 00 00 08 00 > > [ 11.035550] end_request: critical target error, dev sda, sector > > 195940728 > > [ 11.035552] end_request: critical target error, dev sda, sector > > 195940728 > > [ 11.035557] Aborting journal on device sda4-8. > > [ 11.046413] EXT4-fs error (device sda4): ext4_journal_start_sb:348: > > Detected aborted journal > > [ 11.046418] EXT4-fs (sda4): Remounting filesystem read-only > > > I believe the problem is in SCSI. Mode sense command catches my attention: > [ 3.955397] sd 0:0:0:0: [sda] Mode Sense: 00 3a 00 10 > All logs from older kernels has different mode page code: 00 3a 00 00 > > sg_modes from broken kernel shows DPOFUA set, which is 0x10th bit. > Anyone knowing SCSI better could tell if that might cause the problems? I think you know your way around SCSI/libata much better than I do. I just bisected linux-next, and it comes down to the commit below, which introduces the regression for me, and I'm guessing for you also. Maybe it can be fixed up to satisfy us, but otherwise will have to be reverted: we don't invert a default if it's going to break older working systems. A good workaround for me meanwhile is to add boot option "libata.fua=0": please try that (or reverting the commit) and let us know the result. Thanks, Hugh commit 91895b786e631ab47b618c901231f22b5a44115b Author: Zheng Liu Date: Tue May 8 11:24:03 2012 +0800 libata: enable SATA disk fua detection on default Currently, SATA disk fua detection is disabled on default because most of devices don't support this feature at that time. With the development of technology, more and more SATA disks support this feature. So now we can enable this detection on default. Although fua detection is defined as a kernel module parameter, it is too hard to set its value because it must be loaded and set before system starts up. That needs to modify initrd file. So it is inconvenient for administrator who needs to manage a huge number of servers. Signed-off-by: Zheng Liu Signed-off-by: Jeff Garzik --- 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-core.c b/drivers/ata/libata-core.c index 5eee1c1..c3fbdca 100644 --- a/drivers/ata/libata-core.c +++ b/drivers/ata/libata-core.c @@ -135,9 +135,9 @@ int atapi_passthru16 = 1; module_param(atapi_passthru16, int, 0444); MODULE_PARM_DESC(atapi_passthru16, "Enable ATA_16 passthru for ATAPI devices (0=off, 1=on [default])"); -int libata_fua = 0; +int libata_fua = 1; module_param_named(fua, libata_fua, int, 0444); -MODULE_PARM_DESC(fua, "FUA support (0=off [default], 1=on)"); +MODULE_PARM_DESC(fua, "FUA support (0=off, 1=on [default])"); static int ata_ignore_hpa; module_param_named(ignore_hpa, ata_ignore_hpa, int, 0644);