lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Fri, 23 Nov 2018 11:37:32 +0900 From: Masahiro Yamada <yamada.masahiro@...ionext.com> To: Paul Burton <paul.burton@...s.com> Cc: Christoph Hellwig <hch@....de>, mporter@...nel.crashing.org, Alex Bounine <alex.bou9@...il.com>, Dominik Brodowski <linux@...inikbrodowski.net>, Linux Kbuild mailing list <linux-kbuild@...r.kernel.org>, linux-arm-kernel <linux-arm-kernel@...ts.infradead.org>, linux-pci@...r.kernel.org, linux-arch <linux-arch@...r.kernel.org>, Linux Kernel Mailing List <linux-kernel@...r.kernel.org>, linuxppc-dev <linuxppc-dev@...ts.ozlabs.org>, linux-alpha@...r.kernel.org, Linux-MIPS <linux-mips@...ux-mips.org> Subject: Re: [PATCH 3/9] MIPS: remove the HT_PCI config option On Tue, Nov 20, 2018 at 6:07 AM Paul Burton <paul.burton@...s.com> wrote: > > On Mon, Nov 19, 2018 at 01:01:41PM -0800, Paul Burton wrote: > > On Thu, Nov 15, 2018 at 08:05:31PM +0100, Christoph Hellwig wrote: > > > This option is always selected from LOONGSON_MACH3X. Switch to just > > > seleting PCI from that option and definining LOONGSON_PCIIO_BASE based > > > on CONFIG_LOONGSON_MACH3X. > > > > > > Signed-off-by: Christoph Hellwig <hch@....de> > > > --- > > > arch/mips/Kconfig | 11 ----------- > > > arch/mips/include/asm/mach-loongson64/loongson.h | 2 +- > > > arch/mips/loongson64/Kconfig | 2 +- > > > 3 files changed, 2 insertions(+), 13 deletions(-) > > > > > > diff --git a/arch/mips/Kconfig b/arch/mips/Kconfig > > > index 8272ea4c7264..7d28c9dd75d0 100644 > > > --- a/arch/mips/Kconfig > > > +++ b/arch/mips/Kconfig > > > @@ -3040,17 +3040,6 @@ config PCI > > > your box. Other bus systems are ISA, EISA, or VESA. If you have PCI, > > > say Y, otherwise N. > > > > > > -config HT_PCI > > > - bool "Support for HT-linked PCI" > > > - default y > > > - depends on CPU_LOONGSON3 > > > - select PCI > > > - select PCI_DOMAINS > > > - help > > > - Loongson family machines use Hyper-Transport bus for inter-core > > > - connection and device connection. The PCI bus is a subordinate > > > - linked at HT. Choose Y for Loongson-3 based machines. > > > - > > > config PCI_DOMAINS > > > bool > > > > > > diff --git a/arch/mips/loongson64/Kconfig b/arch/mips/loongson64/Kconfig > > > index c865b4b9b775..781a5156ab21 100644 > > > --- a/arch/mips/loongson64/Kconfig > > > +++ b/arch/mips/loongson64/Kconfig > > > @@ -76,7 +76,7 @@ config LOONGSON_MACH3X > > > select CPU_HAS_WB > > > select HW_HAS_PCI > > > select ISA > > > - select HT_PCI > > > + select PCI > > > select I8259 > > > select IRQ_MIPS_CPU > > > select NR_CPUS_DEFAULT_4 > > > > Should this also select PCI_DOMAINS to preserve the existing behavior? > > > > If not, could you explain why in the commit message? > > Ah, I see - PCI already selects PCI_DOMAINS. I think it would have been > worth mentioning OK, I added "PCI already selects PCI_DOMAINS" in the commit description. > but I don't mind if you don't think it a big enough > deal to respin the patch, so: > > Acked-by: Paul Burton <paul.burton@...s.com> > > Thanks, > Paul -- Best Regards Masahiro Yamada
Powered by blists - more mailing lists