All of lore.kernel.org
 help / color / mirror / Atom feed
From: Christoph Hellwig <hch@lst.de>
To: Mike Rapoport <rppt@linux.ibm.com>
Cc: Christoph Hellwig <hch@lst.de>,
	Christian Zigotzky <chzigotzky@xenosoft.de>,
	Robin Murphy <robin.murphy@arm.com>,
	linux-arch@vger.kernel.org, darren@stevens-zone.net,
	mad skateman <madskateman@gmail.com>,
	Benjamin Herrenschmidt <benh@kernel.crashing.org>,
	linux-kernel@vger.kernel.org, linux-mm@kvack.org,
	iommu@lists.linux-foundation.org,
	Rob Herring <robh+dt@kernel.org>,
	paulus@samba.org, rtd2@xtra.co.nz,
	"contact@a-eon.com" <contact@a-eon.com>,
	linuxppc-dev <linuxppc-dev@lists.ozlabs.org>,
	nsaenzjulienne@suse.de
Subject: Re: Bug 205201 - Booting halts if Dawicontrol DC-2976 UW SCSI board installed, unless RAM size limited to 3500M
Date: Wed, 4 Dec 2019 09:56:34 +0100	[thread overview]
Message-ID: <20191204085634.GA25929@lst.de> (raw)
In-Reply-To: <20191127065624.GB16913@linux.ibm.com>

On Wed, Nov 27, 2019 at 08:56:25AM +0200, Mike Rapoport wrote:
> On Tue, Nov 26, 2019 at 05:40:26PM +0100, Christoph Hellwig wrote:
> > On Tue, Nov 26, 2019 at 12:26:38PM +0100, Christian Zigotzky wrote:
> > > Hello Christoph,
> > >
> > > The PCI TV card works with your patch! I was able to patch your Git kernel 
> > > with the patch above.
> > >
> > > I haven't found any error messages in the dmesg yet.
> > 
> > Thanks.  Unfortunately this is a bit of a hack as we need to set
> > the mask based on runtime information like the magic FSL PCIe window.
> > Let me try to draft something better up, and thanks already for testing
> > this one!
> 
> Maybe we'll simply force bottom up allocation before calling
> swiotlb_init()? Anyway, it's the last memblock allocation.

So I think we should go with this fix (plus a source code comment) for
now.  Revamping the whole memory initialization is going to take a
while, and this fix also is easily backportable.

WARNING: multiple messages have this Message-ID (diff)
From: Christoph Hellwig <hch@lst.de>
To: Mike Rapoport <rppt@linux.ibm.com>
Cc: linux-arch@vger.kernel.org, darren@stevens-zone.net,
	rtd2@xtra.co.nz, linuxppc-dev <linuxppc-dev@lists.ozlabs.org>,
	linux-kernel@vger.kernel.org, linux-mm@kvack.org,
	iommu@lists.linux-foundation.org,
	Rob Herring <robh+dt@kernel.org>,
	paulus@samba.org, mad skateman <madskateman@gmail.com>,
	Christian Zigotzky <chzigotzky@xenosoft.de>,
	"contact@a-eon.com" <contact@a-eon.com>,
	Robin Murphy <robin.murphy@arm.com>,
	Christoph Hellwig <hch@lst.de>,
	nsaenzjulienne@suse.de
Subject: Re: Bug 205201 - Booting halts if Dawicontrol DC-2976 UW SCSI board installed, unless RAM size limited to 3500M
Date: Wed, 4 Dec 2019 09:56:34 +0100	[thread overview]
Message-ID: <20191204085634.GA25929@lst.de> (raw)
In-Reply-To: <20191127065624.GB16913@linux.ibm.com>

On Wed, Nov 27, 2019 at 08:56:25AM +0200, Mike Rapoport wrote:
> On Tue, Nov 26, 2019 at 05:40:26PM +0100, Christoph Hellwig wrote:
> > On Tue, Nov 26, 2019 at 12:26:38PM +0100, Christian Zigotzky wrote:
> > > Hello Christoph,
> > >
> > > The PCI TV card works with your patch! I was able to patch your Git kernel 
> > > with the patch above.
> > >
> > > I haven't found any error messages in the dmesg yet.
> > 
> > Thanks.  Unfortunately this is a bit of a hack as we need to set
> > the mask based on runtime information like the magic FSL PCIe window.
> > Let me try to draft something better up, and thanks already for testing
> > this one!
> 
> Maybe we'll simply force bottom up allocation before calling
> swiotlb_init()? Anyway, it's the last memblock allocation.

So I think we should go with this fix (plus a source code comment) for
now.  Revamping the whole memory initialization is going to take a
while, and this fix also is easily backportable.

WARNING: multiple messages have this Message-ID (diff)
From: Christoph Hellwig <hch@lst.de>
To: Mike Rapoport <rppt@linux.ibm.com>
Cc: linux-arch@vger.kernel.org, darren@stevens-zone.net,
	rtd2@xtra.co.nz,
	Benjamin Herrenschmidt <benh@kernel.crashing.org>,
	linuxppc-dev <linuxppc-dev@lists.ozlabs.org>,
	linux-kernel@vger.kernel.org, linux-mm@kvack.org,
	iommu@lists.linux-foundation.org,
	Rob Herring <robh+dt@kernel.org>,
	paulus@samba.org, mad skateman <madskateman@gmail.com>,
	Christian Zigotzky <chzigotzky@xenosoft.de>,
	"contact@a-eon.com" <contact@a-eon.com>,
	Robin Murphy <robin.murphy@arm.com>,
	Christoph Hellwig <hch@lst.de>,
	nsaenzjulienne@suse.de
Subject: Re: Bug 205201 - Booting halts if Dawicontrol DC-2976 UW SCSI board installed, unless RAM size limited to 3500M
Date: Wed, 4 Dec 2019 09:56:34 +0100	[thread overview]
Message-ID: <20191204085634.GA25929@lst.de> (raw)
In-Reply-To: <20191127065624.GB16913@linux.ibm.com>

On Wed, Nov 27, 2019 at 08:56:25AM +0200, Mike Rapoport wrote:
> On Tue, Nov 26, 2019 at 05:40:26PM +0100, Christoph Hellwig wrote:
> > On Tue, Nov 26, 2019 at 12:26:38PM +0100, Christian Zigotzky wrote:
> > > Hello Christoph,
> > >
> > > The PCI TV card works with your patch! I was able to patch your Git kernel 
> > > with the patch above.
> > >
> > > I haven't found any error messages in the dmesg yet.
> > 
> > Thanks.  Unfortunately this is a bit of a hack as we need to set
> > the mask based on runtime information like the magic FSL PCIe window.
> > Let me try to draft something better up, and thanks already for testing
> > this one!
> 
> Maybe we'll simply force bottom up allocation before calling
> swiotlb_init()? Anyway, it's the last memblock allocation.

So I think we should go with this fix (plus a source code comment) for
now.  Revamping the whole memory initialization is going to take a
while, and this fix also is easily backportable.
_______________________________________________
iommu mailing list
iommu@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/iommu

  parent reply	other threads:[~2019-12-04  8:56 UTC|newest]

Thread overview: 74+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-11-16  7:06 Bug 205201 - Booting halts if Dawicontrol DC-2976 UW SCSI board installed, unless RAM size limited to 3500M Christian Zigotzky
2019-11-16  7:06 ` Christian Zigotzky
2019-11-21  7:29 ` Christoph Hellwig
2019-11-21  7:29   ` Christoph Hellwig
2019-11-21  7:29   ` Christoph Hellwig
2019-11-21 12:16   ` Christian Zigotzky
2019-11-21 12:16     ` Christian Zigotzky
2019-11-21 12:16     ` Christian Zigotzky
2019-11-21 12:21     ` Christian Zigotzky
2019-11-21 12:21       ` Christian Zigotzky
2019-11-21 12:21       ` Christian Zigotzky
2019-11-21 13:33       ` Robin Murphy
2019-11-21 13:33         ` Robin Murphy
2019-11-21 13:33         ` Robin Murphy
2019-11-21 16:34         ` Christian Zigotzky
2019-11-21 16:34           ` Christian Zigotzky
2019-11-21 16:34           ` Christian Zigotzky
2019-11-21 18:02           ` Christoph Hellwig
2019-11-21 18:02             ` Christoph Hellwig
2019-11-21 18:02             ` Christoph Hellwig
2019-11-21 18:21             ` Christian Zigotzky
2019-11-21 18:21               ` Christian Zigotzky
2019-11-21 18:21               ` Christian Zigotzky
2019-11-23 11:42             ` Christian Zigotzky
2019-11-23 11:42               ` Christian Zigotzky
2019-11-23 11:42               ` Christian Zigotzky
2019-11-25  7:39               ` Christoph Hellwig
2019-11-25  7:39                 ` Christoph Hellwig
2019-11-25  7:39                 ` Christoph Hellwig
2019-11-25  9:32                 ` Mike Rapoport
2019-11-25  9:32                   ` Mike Rapoport
2019-11-25  9:32                   ` Mike Rapoport
2019-11-25  9:32                   ` Mike Rapoport
2019-11-25  9:32                   ` Mike Rapoport
2019-11-25 16:38                   ` Christian Zigotzky
2019-11-25 16:38                     ` Christian Zigotzky
2019-11-25 16:38                     ` Christian Zigotzky
2019-11-26 11:57                   ` Christian Zigotzky
2019-11-26 11:57                     ` Christian Zigotzky
2019-11-26 11:57                     ` Christian Zigotzky
2019-11-25 16:36                 ` Christian Zigotzky
2019-11-25 16:36                   ` Christian Zigotzky
2019-11-25 16:36                   ` Christian Zigotzky
2019-11-26 11:26                 ` Christian Zigotzky
2019-11-26 11:26                   ` Christian Zigotzky
2019-11-26 11:26                   ` Christian Zigotzky
2019-11-26 16:40                   ` Christoph Hellwig
2019-11-26 16:40                     ` Christoph Hellwig
2019-11-26 16:40                     ` Christoph Hellwig
2019-11-27  6:56                     ` Mike Rapoport
2019-11-27  6:56                       ` Mike Rapoport
2019-11-27  6:56                       ` Mike Rapoport
2019-11-27  6:56                       ` Mike Rapoport
2019-11-27  8:53                       ` Christoph Hellwig
2019-11-27  8:53                         ` Christoph Hellwig
2019-11-27  8:53                         ` Christoph Hellwig
2019-11-27 15:14                       ` Christian Zigotzky
2019-11-27 15:14                         ` Christian Zigotzky
2019-11-27 15:14                         ` Christian Zigotzky
2019-12-04  8:56                       ` Christoph Hellwig [this message]
2019-12-04  8:56                         ` Christoph Hellwig
2019-12-04  8:56                         ` Christoph Hellwig
2019-12-04 12:22                         ` Christian Zigotzky
2019-12-04 12:22                           ` Christian Zigotzky
2019-12-04 12:22                           ` Christian Zigotzky
2019-12-04 12:22                           ` Christian Zigotzky
2020-01-10  7:10                         ` Christian Zigotzky
2020-01-10  7:10                           ` Christian Zigotzky
2020-01-10  7:10                           ` Christian Zigotzky
2020-01-15 15:18                           ` Christoph Hellwig
2020-01-15 15:18                             ` Christoph Hellwig
2020-01-15 15:18                             ` Christoph Hellwig
  -- strict thread matches above, loose matches on Subject: below --
2019-10-29  4:49 Christian Zigotzky
2019-10-30  0:10 ` Christian Zigotzky

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20191204085634.GA25929@lst.de \
    --to=hch@lst.de \
    --cc=benh@kernel.crashing.org \
    --cc=chzigotzky@xenosoft.de \
    --cc=contact@a-eon.com \
    --cc=darren@stevens-zone.net \
    --cc=iommu@lists.linux-foundation.org \
    --cc=linux-arch@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=linuxppc-dev@lists.ozlabs.org \
    --cc=madskateman@gmail.com \
    --cc=nsaenzjulienne@suse.de \
    --cc=paulus@samba.org \
    --cc=robh+dt@kernel.org \
    --cc=robin.murphy@arm.com \
    --cc=rppt@linux.ibm.com \
    --cc=rtd2@xtra.co.nz \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.