From: Andy Shevchenko <andy.shevchenko@gmail.com>
To: "Christian König" <deathsimple@vodafone.de>
Cc: "linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org>,
Platform Driver <platform-driver-x86@vger.kernel.org>,
Bjorn Helgaas <helgaas@kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: Resizeable PCI BAR support v5
Date: Sun, 7 May 2017 13:50:07 +0300 [thread overview]
Message-ID: <CAHp75VcrKTirCH6gHj1DoLzz5JhfzpvVb34M9e5-OQOjO4Mymw@mail.gmail.com> (raw)
In-Reply-To: <1493890270-1188-1-git-send-email-deathsimple@vodafone.de>
On Thu, May 4, 2017 at 12:31 PM, Christian K=C3=B6nig
<deathsimple@vodafone.de> wrote:
> Hi everyone,
>
> this is the fifth incarnation of this set of patches. It enables device
> drivers to resize and most likely also relocate the PCI BAR of devices
> they manage to allow the CPU to access all of the device local memory at =
once.
>
> This is very useful for GFX device drivers where the default PCI BAR is o=
nly
> about 256MB in size for compatibility reasons, but the device easily have
> multiple gigabyte of local memory.
>
> Noteable changed compared to v4:
> 1. Patch #1 is new and a minor cleanup to the PCI code.
> 2. Some more helpers added to patch #2
> 3. We now print a note to syslog before releasing resources.
> 4. Add defines for registers and bits used to add new root hub window.
>
> I've addressed mostly every review comment I've got so far and it looks l=
ike we
> are getting closer to landing this. Any more suggestions/hints how we cou=
ld
> improve the patchset would be very welcome.
>
Thanks for an update.
I'm going to comment individual patches.
Just for the future, please use versioning in the patches themselves
(hint: -vX to git-format-patch).
--=20
With Best Regards,
Andy Shevchenko
next prev parent reply other threads:[~2017-05-07 10:50 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-05-04 9:31 Resizeable PCI BAR support v5 Christian König
2017-05-04 9:31 ` [PATCH 1/5] PCI: add a define for the PCI resource type mask Christian König
2017-05-07 10:48 ` Andy Shevchenko
2017-05-04 9:31 ` [PATCH 2/5] PCI: add resizeable BAR infrastructure v5 Christian König
2017-05-07 10:52 ` Andy Shevchenko
2017-05-04 9:31 ` [PATCH 3/5] PCI: add functionality for resizing resources v4 Christian König
2017-05-07 10:58 ` Andy Shevchenko
2017-05-04 9:31 ` [PATCH 4/5] x86/PCI: Enable a 64bit BAR on AMD Family 15h (Models 30h-3fh) Processors v3 Christian König
2017-05-07 11:03 ` Andy Shevchenko
2017-05-04 9:31 ` [PATCH 5/5] drm/amdgpu: resize VRAM BAR for CPU access v2 Christian König
2017-05-07 10:50 ` Andy Shevchenko [this message]
-- strict thread matches above, loose matches on Subject: below --
2017-06-09 8:59 Resizeable PCI BAR support V5 Christian König
2017-06-14 18:52 ` Bjorn Helgaas
2017-06-29 23:51 ` Dieter Nützel
2017-06-30 12:55 ` Christian König
2017-08-06 22:30 ` Dieter Nützel
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=CAHp75VcrKTirCH6gHj1DoLzz5JhfzpvVb34M9e5-OQOjO4Mymw@mail.gmail.com \
--to=andy.shevchenko@gmail.com \
--cc=deathsimple@vodafone.de \
--cc=helgaas@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pci@vger.kernel.org \
--cc=platform-driver-x86@vger.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).