From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.7 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FAKE_REPLY_C,HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,MENTIONS_GIT_HOSTING,SPF_PASS,UNPARSEABLE_RELAY, USER_AGENT_MUTT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id A4516C43381 for ; Mon, 18 Feb 2019 06:24:06 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 5E13B218AD for ; Mon, 18 Feb 2019 06:24:05 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=oracle.com header.i=@oracle.com header.b="MqCqVHGL" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726500AbfBRGYE (ORCPT ); Mon, 18 Feb 2019 01:24:04 -0500 Received: from userp2120.oracle.com ([156.151.31.85]:50160 "EHLO userp2120.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725881AbfBRGYE (ORCPT ); Mon, 18 Feb 2019 01:24:04 -0500 Received: from pps.filterd (userp2120.oracle.com [127.0.0.1]) by userp2120.oracle.com (8.16.0.27/8.16.0.27) with SMTP id x1I6NoYM059361; Mon, 18 Feb 2019 06:23:50 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.com; h=date : from : to : cc : subject : message-id : mime-version : content-type : in-reply-to; s=corp-2018-07-02; bh=FonYN5XNFxq8lUxxQIm1IaxVxzXsoJ4CQGwOGzVstSs=; b=MqCqVHGLQ7rjqSZ3DIAHX5V8gdqUa9UTL3imwwbHGxIC3mgmQbZL/4V1WMqgUMSC1pb2 5c1/ejZGb/U5r8aiT9gb+ijCrt3AbVKErde/18HindUG/W6sWnF85SlxrqwRP8ub08wh W1aWuYKLViE9pkcTT6DLCgAkJMZgUIM0I3+ouvifUh+UUEEsZogUP5A3yi3azYekOJLH Epy92Hm0r1V5kaFSiURkw601pfzyfSM4rLQL0MsdBOyFoLyVh28XM8zfcA0kVpCNqVoT 73yODnx8kFEkibzMTG6X4nChY2mWvQVn65qsFldaw0ya5k0ib86J8H09uWJ4+tznbwyK tw== Received: from userv0022.oracle.com (userv0022.oracle.com [156.151.31.74]) by userp2120.oracle.com with ESMTP id 2qpb5r49mv-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Mon, 18 Feb 2019 06:23:50 +0000 Received: from userv0122.oracle.com (userv0122.oracle.com [156.151.31.75]) by userv0022.oracle.com (8.14.4/8.14.4) with ESMTP id x1I6Nn9q026951 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Mon, 18 Feb 2019 06:23:49 GMT Received: from abhmp0003.oracle.com (abhmp0003.oracle.com [141.146.116.9]) by userv0122.oracle.com (8.14.4/8.14.4) with ESMTP id x1I6NllD016276; Mon, 18 Feb 2019 06:23:48 GMT Received: from kadam (/197.157.0.22) by default (Oracle Beehive Gateway v4.0) with ESMTP ; Sun, 17 Feb 2019 22:23:46 -0800 Date: Mon, 18 Feb 2019 09:23:37 +0300 From: Dan Carpenter To: kbuild@01.org, Hannes Reinecke Cc: kbuild-all@01.org, Christoph Hellwig , "Martin K. Petersen" , James Bottomley , linux-scsi@vger.kernel.org, Ewan Milne , Hannes Reinecke , stable@vger.kernel.org, Hannes Reinecke Subject: Re: [PATCH 3/4] bfa: fix calls to dma_set_mask_and_coherent() Message-ID: <20190218062337.GK2304@kadam> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20190213114234.67275-4-hare@suse.de> User-Agent: Mutt/1.9.4 (2018-02-28) X-Proofpoint-Virus-Version: vendor=nai engine=5900 definitions=9170 signatures=668683 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 priorityscore=1501 malwarescore=0 suspectscore=0 phishscore=0 bulkscore=0 spamscore=0 clxscore=1011 lowpriorityscore=0 mlxscore=0 impostorscore=0 mlxlogscore=999 adultscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1810050000 definitions=main-1902180050 Sender: stable-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: stable@vger.kernel.org Hi Hannes, url: https://github.com/0day-ci/linux/commits/Hannes-Reinecke/scsi-fixup-dma_set_mask_and_coherent-calls/20190214-044535 base: https://git.kernel.org/pub/scm/linux/kernel/git/mkp/scsi.git for-next smatch warnings: drivers/scsi/bfa/bfad.c:805 bfad_pci_init() error: uninitialized symbol 'rc'. # https://github.com/0day-ci/linux/commit/48d00fbe6a39ffd9e97f505206b4394efb9803bc git remote add linux-review https://github.com/0day-ci/linux git remote update linux-review git checkout 48d00fbe6a39ffd9e97f505206b4394efb9803bc vim +/rc +805 drivers/scsi/bfa/bfad.c 7725ccfda Jing Huang 2009-09-23 726 7725ccfda Jing Huang 2009-09-23 727 int 7725ccfda Jing Huang 2009-09-23 728 bfad_pci_init(struct pci_dev *pdev, struct bfad_s *bfad) 7725ccfda Jing Huang 2009-09-23 729 { 48d00fbe6 Hannes Reinecke 2019-02-13 730 int rc; 7725ccfda Jing Huang 2009-09-23 731 7725ccfda Jing Huang 2009-09-23 732 if (pci_enable_device(pdev)) { a36c61f90 Krishna Gudipati 2010-09-15 733 printk(KERN_ERR "pci_enable_device fail %p\n", pdev); 7725ccfda Jing Huang 2009-09-23 734 goto out; ^^^^^^^^^ 7725ccfda Jing Huang 2009-09-23 735 } 7725ccfda Jing Huang 2009-09-23 736 7725ccfda Jing Huang 2009-09-23 737 if (pci_request_regions(pdev, BFAD_DRIVER_NAME)) 7725ccfda Jing Huang 2009-09-23 738 goto out_disable_device; ^^^^^^^^^^^^^^^^^^^^^^^^ 7725ccfda Jing Huang 2009-09-23 739 7725ccfda Jing Huang 2009-09-23 740 pci_set_master(pdev); 7725ccfda Jing Huang 2009-09-23 741 48d00fbe6 Hannes Reinecke 2019-02-13 742 rc = dma_set_mask_and_coherent(&pdev->dev, DMA_BIT_MASK(64)); 48d00fbe6 Hannes Reinecke 2019-02-13 743 if (rc) 48d00fbe6 Hannes Reinecke 2019-02-13 744 rc = dma_set_mask_and_coherent(&pdev->dev, DMA_BIT_MASK(32)); 48d00fbe6 Hannes Reinecke 2019-02-13 745 48d00fbe6 Hannes Reinecke 2019-02-13 746 if (rc) { a69b08002 Christoph Hellwig 2018-10-18 747 printk(KERN_ERR "dma_set_mask_and_coherent fail %p\n", pdev); 7725ccfda Jing Huang 2009-09-23 748 goto out_release_region; 7725ccfda Jing Huang 2009-09-23 749 } 48d00fbe6 Hannes Reinecke 2019-02-13 750 rc = -ENODEV; 7725ccfda Jing Huang 2009-09-23 751 881c1b3c4 Krishna Gudipati 2012-08-22 752 /* Enable PCIE Advanced Error Recovery (AER) if kernel supports */ 881c1b3c4 Krishna Gudipati 2012-08-22 753 pci_enable_pcie_error_reporting(pdev); 881c1b3c4 Krishna Gudipati 2012-08-22 754 b3522f08e Jing Huang 2010-03-19 755 bfad->pci_bar0_kva = pci_iomap(pdev, 0, pci_resource_len(pdev, 0)); 111892082 Krishna Gudipati 2011-06-13 756 bfad->pci_bar2_kva = pci_iomap(pdev, 2, pci_resource_len(pdev, 2)); 7725ccfda Jing Huang 2009-09-23 757 7725ccfda Jing Huang 2009-09-23 758 if (bfad->pci_bar0_kva == NULL) { a36c61f90 Krishna Gudipati 2010-09-15 759 printk(KERN_ERR "Fail to map bar0\n"); 7725ccfda Jing Huang 2009-09-23 760 goto out_release_region; 7725ccfda Jing Huang 2009-09-23 761 } 7725ccfda Jing Huang 2009-09-23 762 7725ccfda Jing Huang 2009-09-23 763 bfad->hal_pcidev.pci_slot = PCI_SLOT(pdev->devfn); 7725ccfda Jing Huang 2009-09-23 764 bfad->hal_pcidev.pci_func = PCI_FUNC(pdev->devfn); 7725ccfda Jing Huang 2009-09-23 765 bfad->hal_pcidev.pci_bar_kva = bfad->pci_bar0_kva; 7725ccfda Jing Huang 2009-09-23 766 bfad->hal_pcidev.device_id = pdev->device; 1a4d8e1bd Krishna Gudipati 2011-06-24 767 bfad->hal_pcidev.ssid = pdev->subsystem_device; 7725ccfda Jing Huang 2009-09-23 768 bfad->pci_name = pci_name(pdev); 7725ccfda Jing Huang 2009-09-23 769 7725ccfda Jing Huang 2009-09-23 770 bfad->pci_attr.vendor_id = pdev->vendor; 7725ccfda Jing Huang 2009-09-23 771 bfad->pci_attr.device_id = pdev->device; 7725ccfda Jing Huang 2009-09-23 772 bfad->pci_attr.ssid = pdev->subsystem_device; 7725ccfda Jing Huang 2009-09-23 773 bfad->pci_attr.ssvid = pdev->subsystem_vendor; 7725ccfda Jing Huang 2009-09-23 774 bfad->pci_attr.pcifn = PCI_FUNC(pdev->devfn); 7725ccfda Jing Huang 2009-09-23 775 7725ccfda Jing Huang 2009-09-23 776 bfad->pcidev = pdev; a36c61f90 Krishna Gudipati 2010-09-15 777 a36c61f90 Krishna Gudipati 2010-09-15 778 /* Adjust PCIe Maximum Read Request Size */ c0102c00d Yijing Wang 2013-09-05 779 if (pci_is_pcie(pdev) && pcie_max_read_reqsz) { c0102c00d Yijing Wang 2013-09-05 780 if (pcie_max_read_reqsz >= 128 && c0102c00d Yijing Wang 2013-09-05 781 pcie_max_read_reqsz <= 4096 && c0102c00d Yijing Wang 2013-09-05 782 is_power_of_2(pcie_max_read_reqsz)) { c0102c00d Yijing Wang 2013-09-05 783 int max_rq = pcie_get_readrq(pdev); a36c61f90 Krishna Gudipati 2010-09-15 784 printk(KERN_WARNING "BFA[%s]: " a36c61f90 Krishna Gudipati 2010-09-15 785 "pcie_max_read_request_size is %d, " c0102c00d Yijing Wang 2013-09-05 786 "reset to %d\n", bfad->pci_name, max_rq, a36c61f90 Krishna Gudipati 2010-09-15 787 pcie_max_read_reqsz); c0102c00d Yijing Wang 2013-09-05 788 pcie_set_readrq(pdev, pcie_max_read_reqsz); c0102c00d Yijing Wang 2013-09-05 789 } else { c0102c00d Yijing Wang 2013-09-05 790 printk(KERN_WARNING "BFA[%s]: invalid " c0102c00d Yijing Wang 2013-09-05 791 "pcie_max_read_request_size %d ignored\n", c0102c00d Yijing Wang 2013-09-05 792 bfad->pci_name, pcie_max_read_reqsz); a36c61f90 Krishna Gudipati 2010-09-15 793 } a36c61f90 Krishna Gudipati 2010-09-15 794 } a36c61f90 Krishna Gudipati 2010-09-15 795 881c1b3c4 Krishna Gudipati 2012-08-22 796 pci_save_state(pdev); 881c1b3c4 Krishna Gudipati 2012-08-22 797 7725ccfda Jing Huang 2009-09-23 798 return 0; 7725ccfda Jing Huang 2009-09-23 799 7725ccfda Jing Huang 2009-09-23 800 out_release_region: 7725ccfda Jing Huang 2009-09-23 801 pci_release_regions(pdev); 7725ccfda Jing Huang 2009-09-23 802 out_disable_device: 7725ccfda Jing Huang 2009-09-23 803 pci_disable_device(pdev); 7725ccfda Jing Huang 2009-09-23 804 out: 7725ccfda Jing Huang 2009-09-23 @805 return rc; 7725ccfda Jing Huang 2009-09-23 806 } 7725ccfda Jing Huang 2009-09-23 807 :::::: The code at line 805 was first introduced by commit :::::: 7725ccfda59715ecf8f99e3b520a0b84cc2ea79e [SCSI] bfa: Brocade BFA FC SCSI driver :::::: TO: Jing Huang :::::: CC: James Bottomley --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation