linux-pci.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Bjorn Helgaas <helgaas@kernel.org>
To: Hans de Goede <hdegoede@redhat.com>
Cc: "Rafael J . Wysocki" <rafael@kernel.org>,
	"Mika Westerberg" <mika.westerberg@linux.intel.com>,
	"Krzysztof Wilczyński" <kw@linux.com>,
	"Bjorn Helgaas" <bhelgaas@google.com>,
	"Myron Stowe" <myron.stowe@redhat.com>,
	"Juha-Pekka Heikkila" <juhapekka.heikkila@gmail.com>,
	"Thomas Gleixner" <tglx@linutronix.de>,
	"Ingo Molnar" <mingo@redhat.com>,
	"Borislav Petkov" <bp@alien8.de>,
	"H . Peter Anvin" <hpa@zytor.com>,
	"Benoit Grégoire" <benoitg@coeus.ca>,
	"Hui Wang" <hui.wang@canonical.com>,
	linux-acpi@vger.kernel.org, linux-pci@vger.kernel.org,
	x86@kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v7 1/1] x86/PCI: Ignore E820 reservations for bridge windows on newer systems
Date: Thu, 19 May 2022 09:49:23 -0500	[thread overview]
Message-ID: <20220519144923.GA22233@bhelgaas> (raw)
In-Reply-To: <c49dfdf1-c1e8-a9d2-0f31-f190d7b6631f@redhat.com>

On Thu, May 19, 2022 at 04:29:43PM +0200, Hans de Goede wrote:
> Hi,
> 
> On 5/19/22 16:14, Bjorn Helgaas wrote:
> > On Thu, May 19, 2022 at 04:01:48PM +0200, Hans de Goede wrote:
> > 
> >> Ok, I'll go and prepare a v9 and I will submit that later today.
> > 
> > Would it be practical to split into three patches?
> > 
> >   1) Add command-line args
> >   2) Add DMI quirks
> >   3) Add date check
> > 
> > It seems easier to assimilate and document in smaller pieces, if
> > that's possible.
> 
> Ack, will do. Note this will cause quite a bit of copy/paste
> in the commit msg to explain why these changes are necessary.

OK, if the repetition gets excessive I can squash them back
together.  Hopefully the main explanation can go in the first patch,
the second can just mention the fact that these machines need the
exception, and the third can focus on the plan for the future.

  reply	other threads:[~2022-05-19 14:49 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-05-05 15:20 [PATCH v7 0/1] x86/PCI: Ignore E820 reservations for bridge windows on newer systems Hans de Goede
2022-05-05 15:20 ` [PATCH v7 1/1] " Hans de Goede
2022-05-06 16:51   ` Bjorn Helgaas
2022-05-07 10:09     ` Hans de Goede
2022-05-07 15:31       ` Bjorn Helgaas
2022-05-09 17:33         ` Hans de Goede
2022-05-09 18:21           ` Hans de Goede
2022-05-09 19:36           ` Bjorn Helgaas
2022-05-12 19:15             ` Hans de Goede
2022-05-18 22:07           ` Bjorn Helgaas
2022-05-19 14:01             ` Hans de Goede
2022-05-19 14:14               ` Bjorn Helgaas
2022-05-19 14:29                 ` Hans de Goede
2022-05-19 14:49                   ` Bjorn Helgaas [this message]
2022-05-19 15:06                     ` Hans de Goede

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=20220519144923.GA22233@bhelgaas \
    --to=helgaas@kernel.org \
    --cc=benoitg@coeus.ca \
    --cc=bhelgaas@google.com \
    --cc=bp@alien8.de \
    --cc=hdegoede@redhat.com \
    --cc=hpa@zytor.com \
    --cc=hui.wang@canonical.com \
    --cc=juhapekka.heikkila@gmail.com \
    --cc=kw@linux.com \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pci@vger.kernel.org \
    --cc=mika.westerberg@linux.intel.com \
    --cc=mingo@redhat.com \
    --cc=myron.stowe@redhat.com \
    --cc=rafael@kernel.org \
    --cc=tglx@linutronix.de \
    --cc=x86@kernel.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).