Received: by 10.223.185.116 with SMTP id b49csp6450008wrg; Wed, 28 Feb 2018 09:34:26 -0800 (PST) X-Google-Smtp-Source: AH8x224rg7OblV4758KQoeOd15rBJnoEFthw6zXxXRg2JyxLKiSE/2goJl5CbBW9ZObe47dyQHgA X-Received: by 2002:a17:902:5716:: with SMTP id k22-v6mr19090172pli.229.1519839265980; Wed, 28 Feb 2018 09:34:25 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1519839265; cv=none; d=google.com; s=arc-20160816; b=Ys3UO15lv/MdR1k0w49g7gGPMD4Twj0A8cqvynQ9Da7g3/dZX2hw8FZM0/ONdUW85/ KX0OFnX5aY+PL2rDlzMl8ytBQejm8pBxL+YwNpAvR9QpH5OH2e/HnZFCXEjnlHD9XJAm qiXLrvLY2Fuzwg5wURZYozRrslTtT8gDzOrGetNLOzGA3NBnXF9u4+dRkIpsiXmrf5Lv q9wz+G9WvxStYenUEh66BZw6vrCMgxdH08iWIwrRmMI/sNgtuBkbmMtml2vdbCBAqt9M /z5CxCIBgXzchPHge8dzXvF3JXxuCShWvVqaXs3S5nuGn09jXXwSrgmwwi23GRf3NelP ZKIA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:in-reply-to:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :arc-authentication-results; bh=D/6QexqiyyeO2r4PqHHUhtxTAtouTJHmL8dJmgqBk9s=; b=Q7QRq7dkWjKHG+NHZLhApkoMpGKwAzCPFjqQ/+vXVuNTViDfDJwB/USlHmyUaFMfyd 3mINIbW56ignZMt5lmT5OCAn05RC8nJvP/IIakJl5POQi5rWU3jhsVKGfEPSUemCGwJy zXf9psXCvxF3zd/fc2a0sjzTOn2WS5xzT+daC3Yab1s6uarXzyvHA/lyAaprnHvF7VJr aiUyDHf0M7sE3BxCnbUTSIDj+mry6UOrg9e9t3Ly3ZyhRbKBI1Id2K1TKQu+fN9W0dU8 CaM0JdKJin9D87d8ZZ5egaqZIkQ4QHqWUqA1cvNunZy2P8HrpetJEoxSZlX2Ak60+c7r I/xQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=redhat.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id c3-v6si1546587plo.45.2018.02.28.09.34.10; Wed, 28 Feb 2018 09:34:25 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=redhat.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933480AbeB1RdC (ORCPT + 99 others); Wed, 28 Feb 2018 12:33:02 -0500 Received: from mx3-rdu2.redhat.com ([66.187.233.73]:50160 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S932236AbeB1Rc7 (ORCPT ); Wed, 28 Feb 2018 12:32:59 -0500 Received: from smtp.corp.redhat.com (int-mx05.intmail.prod.int.rdu2.redhat.com [10.11.54.5]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 8E59A8E69F; Wed, 28 Feb 2018 17:32:58 +0000 (UTC) Received: from redhat.com (ovpn-123-220.rdu2.redhat.com [10.10.123.220]) by smtp.corp.redhat.com (Postfix) with SMTP id CEA43946AB; Wed, 28 Feb 2018 17:32:53 +0000 (UTC) Date: Wed, 28 Feb 2018 19:32:53 +0200 From: "Michael S. Tsirkin" To: =?iso-8859-1?Q?Marc-Andr=E9?= Lureau Cc: linux-kernel@vger.kernel.org, arnd@arndb.de, bhe@redhat.com, slp@redhat.com, somlo@cmu.edu, xiaolong.ye@intel.com Subject: Re: [PATCH v16 06/11] fw_cfg: handle fw_cfg_read_blob() error Message-ID: <20180228193142-mutt-send-email-mst@kernel.org> References: <20180228150615.17317-1-marcandre.lureau@redhat.com> <20180228150615.17317-7-marcandre.lureau@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20180228150615.17317-7-marcandre.lureau@redhat.com> X-Scanned-By: MIMEDefang 2.79 on 10.11.54.5 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.11.55.2]); Wed, 28 Feb 2018 17:32:58 +0000 (UTC) X-Greylist: inspected by milter-greylist-4.5.16 (mx1.redhat.com [10.11.55.2]); Wed, 28 Feb 2018 17:32:58 +0000 (UTC) for IP:'10.11.54.5' DOMAIN:'int-mx05.intmail.prod.int.rdu2.redhat.com' HELO:'smtp.corp.redhat.com' FROM:'mst@redhat.com' RCPT:'' Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org > @@ -557,7 +566,10 @@ static int fw_cfg_sysfs_probe(struct platform_device *pdev) > goto err_probe; > > /* get revision number, add matching top-level attribute */ > - fw_cfg_read_blob(FW_CFG_ID, &rev, 0, sizeof(rev)); > + err = fw_cfg_read_blob(FW_CFG_ID, &rev, 0, sizeof(rev)); > + if (err < 0) > + goto err_probe; > + > fw_cfg_rev = le32_to_cpu(rev); > err = sysfs_create_file(fw_cfg_top_ko, &fw_cfg_rev_attr.attr); > if (err) So on this one, maybe we should just drop this attribute. Does anyone use it? Removing it will speed up boot slightly. > -- > 2.16.1.73.g5832b7e9f2