Received: by 2002:ac0:a679:0:0:0:0:0 with SMTP id p54csp1241234imp; Thu, 21 Feb 2019 22:51:27 -0800 (PST) X-Google-Smtp-Source: AHgI3IaIgpy4JZb1ylmStJRBpjWHv+7rTc8jPp/+Wsh3ugnA/rK3EDftK5ceSyvfMG3c7IfDm97c X-Received: by 2002:aa7:82d7:: with SMTP id f23mr2718007pfn.114.1550818287797; Thu, 21 Feb 2019 22:51:27 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1550818287; cv=none; d=google.com; s=arc-20160816; b=omLwnUIVOGmE3z+2tdBK4F9dqdGnLbaDKXtTnOhyNf6hDKa9YX7kOOzQ4GThC9bFxa o1kX5BucE7mnc27IehiTnyJMaKp7gNIGu4OH+jTgMUyJQQy7YeVIhHRGjapZMu+cKaQi 8JglDcQPbUaW+GJT55yY9oZ5BmxxrCpl4o+Cx9G0mI/iwg1Zmm9NW5oMtX0BLvNESHJq h2Fm/KVS9QkU121iTKyMps/uRPZFIcV5MLKsQb02rYbJKS5p5pc07tks04XCeh95mkFe H/Nhm2eZlV6CP1QWAiWolrhdy1E6UmZv37nPpuyDj6zMWkg5V+wjix1LQy2CISM7bzuM HCxw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:dkim-signature; bh=mympUM6ORB7plBDV8SjWpei7YxOFpwTHdtr56pFBpwo=; b=mBPGdzW+Qg65vGDwn3kwlt7vYJrcIxrXUvuL1uKuxnmfpK06RawdImNRjtXWaWcfdt aDa1d0oLasmRhndvRS7qnN3pYVZvZK0PGxluJ0rp8SWHOtc1Vu2wkan3JDyg7gUg58wV FkTu8JGCd0pQOBZzowLvKhckQZk37pyX8PrNILpIjRB8kGcEK65qYDvdGhWSybiFoAaG rIL4a3SyEnElXWxKwQ/sX48gkZEWKiaHgc4V/CfTqkLv1T/BRxRyl4JpfySOYUaFosI+ /VHLBaBmhsluo6Jsbi39SxwJEFbl4bS6qM1srXbDemV7dmYCsLApFmcaIzcn9kvfCWEJ j8EA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b="VTTF/Iss"; 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 Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id d34si649013pld.290.2019.02.21.22.51.12; Thu, 21 Feb 2019 22:51:27 -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; dkim=pass header.i=@kernel.org header.s=default header.b="VTTF/Iss"; 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 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726183AbfBVGuL (ORCPT + 99 others); Fri, 22 Feb 2019 01:50:11 -0500 Received: from mail.kernel.org ([198.145.29.99]:44248 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725821AbfBVGuL (ORCPT ); Fri, 22 Feb 2019 01:50:11 -0500 Received: from localhost (5356596B.cm-6-7b.dynamic.ziggo.nl [83.86.89.107]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 237CD20823; Fri, 22 Feb 2019 06:50:09 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1550818210; bh=HrvpjxFAAc3p55NaHBg4/woNZPf9C0qNVi9dsGK+Ap4=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=VTTF/Iss9Ze4Fxzc4CY2tMJPo+jUHLps/RdiR6jWG/EUCSVmDp3SWMhgtW7SKSSRW aUMrUjkQwdnjLUw0HkogIjKHNUfJXK6rsr6+472Y/QxKoyOvm51BP4oebluoaePHJf rSzyl9bH3UYHjlLb1HaI9jb1zVlaECjLIDXIe6Fs= Date: Fri, 22 Feb 2019 07:50:08 +0100 From: Greg KH To: Jerry Hoemann Cc: Matt Hsiao , linux-kernel@vger.kernel.org, arnd@arndb.de, david.altobelli@hpe.com, mark.rusk@hpe.com Subject: Re: [PATCH 2/4] misc: hpilo: Exclude unsupported device via blacklist Message-ID: <20190222065008.GB15860@kroah.com> References: <1550736282-25416-1-git-send-email-matt.hsiao@hpe.com> <1550736282-25416-3-git-send-email-matt.hsiao@hpe.com> <20190221083355.GB6397@kroah.com> <20190222043506.GC31132@anatevka> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20190222043506.GC31132@anatevka> User-Agent: Mutt/1.11.3 (2019-02-01) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Feb 21, 2019 at 09:35:07PM -0700, Jerry Hoemann wrote: > On Thu, Feb 21, 2019 at 09:33:55AM +0100, Greg KH wrote: > > On Thu, Feb 21, 2019 at 04:04:40PM +0800, Matt Hsiao wrote: > > > > +static const struct pci_device_id ilo_blacklist[] = { > > > + /* auxiliary iLO */ > > > + {PCI_DEVICE_SUB(PCI_VENDOR_ID_HP, 0x3307, PCI_VENDOR_ID_HP, 0x1979)}, > > > + {} > > > +}; > > > > > ... > > > > > And why do some devices need to be blacklisted, shouldn't there only be > > a whitelist in the first place? Do you need to tighten up your original > > device ids? > > Hi Greg, > > I related the underlying reason for the black listing on another message > of this thread. I can fill you in on why we've taken this approach to > white/black listing. > > HPE hardware/firmware teams will put out minor updates to the iLO using > the same device info except for the subsystem device id. > > The approach we've taken in both the hpilo and hpwdt drivers is > to claim based upon {Vendor, PC DevID, SubVendor}. > > This allows old software to work on new hardware without patching. > > As our primary way to support our customers is via distros, this patching > when it does happen requires us to not just submit a patch upstream, but > to then to have the patches back ported to multiple releases of multiple > distros. This process takes many many months. > > So far, the approach we've taken has worked fairly well as this is only > the second time in 10+ years that we've needed to blacklist an instance. Ok, that's fine, but you should put that information in the changelog text so that we understand what is going on here. thanks, greg k-h