Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751074AbdCMGnc (ORCPT ); Mon, 13 Mar 2017 02:43:32 -0400 Received: from mail-pf0-f169.google.com ([209.85.192.169]:36219 "EHLO mail-pf0-f169.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750866AbdCMGnZ (ORCPT ); Mon, 13 Mar 2017 02:43:25 -0400 MIME-Version: 1.0 From: Sreekanth Reddy Date: Mon, 13 Mar 2017 12:13:23 +0530 Message-ID: Subject: Getting "Wrong diagnostic page; asked for 7 got 0" error message on HBA's virtual SES device To: "linux-scsi@vger.kernel.org" , "linux-kernel@vger.kernel.org" Cc: Caleb Randall , wasim.qureshi@broadcom.com, Sayantan Bhattacharya Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by mail.home.local id v2D6hehm000947 Content-Length: 2924 Lines: 68 Hi, Our LSI(Broadcom) SAS3.5 HBA device's support virtual SES device. Whenever we load the mpt3sas driver then we are observing below error message, "Wrong diagnostic page; asked for 7 got 0" Our virtual SES device doesn't support Diagnostic page 7, it supports only below diagnostic pages, • 00h List of Supported Diagnostic Pages • 01h SES Configuration Diagnostic Page • 02h SES Enclosure Control/Enclosure Status Diagnostic Page • 0Ah SES Additional Element Status Diagnostic Page Page Code 07 is Element Descriptor Diagnostic Page Per SES3 spec (see table 10) this page is optional and not supported by our internal VSES. But 'ses' kernel module is issuing a RECEIVE_DIAGNOSTIC command for diagnostic page 7 without checking whether target device supports this page or not (i.e. without looking into 00h page) as shown below, result = ses_recv_diag(sdev, 7, hdr_buf, INIT_ALLOC_SIZE); if (result) goto simple_populate; As the virtual SES devices doesn't support this page 7, so it has failed this RECEIVE_DIAGNOSTIC command with illegal request sense key as shown below, ses 11:0:0:0: tag#0 Send: scmd 0xffff883fee898000 ses 11:0:0:0: tag#0 CDB: Receive Diagnostic 1c 01 07 00 20 00 ses 11:0:0:0: tag#0 CDB: Receive Diagnostic 1c 01 07 00 20 00 mpt3sas_cm1: sas_address(0x510600b012345600), phy(16) mpt3sas_cm1: enclosure_logical_id(0x500605b012345600),slot(16) mpt3sas_cm1: enclosure level(0x0000), connector name( ) mpt3sas_cm1: handle(0x0011), ioc_status(success)(0x0000), smid(1) mpt3sas_cm1: request_len(32), underflow(0), resid(32) mpt3sas_cm1: tag(0), transfer_count(0), sc->result(0x00000002) mpt3sas_cm1: scsi_status(check condition)(0x02), scsi_state(autosense valid )(0x01) mpt3sas_cm1: [sense_key,asc,ascq]: [0x05,0x26,0x00], count(18) mpt3sas_cm1: log_info(0x31200205): originator(PL), code(0x20), sub_code(0x0205) ses 11:0:0:0: tag#0 Done: SUCCESS Result: hostbyte=DID_TARGET_FAILURE driverbyte=DRIVER_OK ses 11:0:0:0: tag#0 CDB: Receive Diagnostic 1c 01 07 00 20 00 ses 11:0:0:0: tag#0 Sense Key : Illegal Request [current] ses 11:0:0:0: tag#0 Add. Sense: Invalid field in parameter list ses 11:0:0:0: tag#0 scsi host busy 1 failed 0 ses 11:0:0:0: Notifying upper driver of completion (result 8100002) ses 11:0:0:0: tag#0 0 sectors total, 32 bytes done. ses 11:0:0:0: Wrong diagnostic page; asked for 7 got 0 As the command is failed with illegal request sense key, so the buffer used In function 'ses_recv_diag' will be not updated and so it will have all zero's. so the page_code will be zero and we observe below wrong error message even though vSES device has failed the command with illegal request error message and it has not returned any wrong diagnostic page. sdev_printk(KERN_ERR, sdev, "Wrong diagnostic page; asked for %d got %u\n", page_code, recv_page_code); Thanks, Sreekanth