[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2] pci: fix handling of PCI bridges with subordinate bus number 0xff
- To: Igor Druzhinin <igor.druzhinin@xxxxxxxxxx>
- From: Jan Beulich <jbeulich@xxxxxxxx>
- Date: Fri, 24 Sep 2021 11:59:07 +0200
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=suse.com; dmarc=pass action=none header.from=suse.com; dkim=pass header.d=suse.com; arc=none
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=5WR5TsPKpIUkmQ4IQU9GK9qAtnS1zO3umVFMyOynuwM=; b=Kzm/Lh0NzSHUIzpnSTZeaUBJNo6Xe/+zhjwxNm5/e1o7AAESgRC7VYnq0M0TZojSPLsnSBBPLVRtT4BdX+7+zDlPvuS4c9than9buxCXvHWXVwyDnkZD21NtLJatDHauVlxDdjQ5v9MZz3qckfUPaFkQ+Ez1c+n6cF2AvXpj/KPV9Ty8wR/8dZWB8Z8TbWK7bbHehJTCSk8x/9TRijmsuWF5kXZrZj51YSb2DNogqfPDOKHdDMhC2+YdVD0pwSszSFqXzokE3MJAoQeKfB51AYcqTe4EgYHnrBFSM/lRtudU3VuGtvt18gvziHkM37HX4C2igEMosui1b2goB0wHug==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ZYhDv6x1ajlroHEjGPfUkuhpCMT1sUMurmdSEy32PwQ8Evs3/vfUwIovPhJm1/wZKrlQCfuSpA72PMRhTbm6jek0FZIQb8sJTnSRZvvfQvq9lHQY+LDXYedUyoTEY2qonOdP8EJWpDHiQexvnFiXq00sHEx8Qep5VLzaeS7C4I/6iJCy6Yl3ViziFhd+YnUbPPWV6AwX9k0RN/CObmH5lveZTfed1ZKffEkauoaEFd09wK9xm3zC8iJSpf2eQxf5hw2Tv4MnAQzpkXjZLBaZXxst3EQ91UU+4dnTL6dPBWvbR5XB6MxCPkgqMvPenaKgVcAfAOwnzXX8RQ1TTVBYwQ==
- Authentication-results: lists.xenproject.org; dkim=none (message not signed) header.d=none;lists.xenproject.org; dmarc=none action=none header.from=suse.com;
- Cc: paul@xxxxxxx, xen-devel@xxxxxxxxxxxxxxxxxxxx
- Delivery-date: Fri, 24 Sep 2021 09:59:21 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On 24.09.2021 11:10, Igor Druzhinin wrote:
> Bus number 0xff is valid according to the PCI spec. Using u8 typed sub_bus
> and assigning 0xff to it will result in the following loop getting stuck.
>
> for ( ; sec_bus <= sub_bus; sec_bus++ ) {...}
>
> Just change its type to unsigned int similarly to what is already done in
> dmar_scope_add_buses().
>
> Signed-off-by: Igor Druzhinin <igor.druzhinin@xxxxxxxxxx>
Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx>
|
