Hi Thierry,
I love your patch! Yet something to improve:
[auto build test ERROR on linus/master]
[also build test ERROR on v5.4-rc5 next-20191031]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to spec
https://bugs.freedesktop.org/show_bug.cgi?id=112201
--- Comment #3 from Ilia Mirkin ---
(In reply to stratus from comment #2)
> So this is not causing any problems, I suppose it's controlling the hardware
> by reading/writing to memory addresses provided by the kernel, and valgrind
> is giving a
https://bugs.freedesktop.org/show_bug.cgi?id=112201
--- Comment #2 from stra...@tuta.io ---
So this is not causing any problems, I suppose it's controlling the hardware by
reading/writing to memory addresses provided by the kernel, and valgrind is
giving a false positive, thank you for your rapid
https://bugs.freedesktop.org/show_bug.cgi?id=112201
--- Comment #1 from Ilia Mirkin ---
This is expected. valgrind does not understand how the ioctl works, so it
thinks that uninitialized memory is being read. But that's not an actual issue.
--
You are receiving this mail because:
You are the a
https://bugs.freedesktop.org/show_bug.cgi?id=112201
Bug ID: 112201
Summary: Syscall param ioctl(generic) points to uninitialised
byte(s)
Product: xorg
Version: unspecified
Hardware: x86-64 (AMD64)
OS: Linux
From: Thierry Reding
When the GPU powergate is controlled by a generic power domain provider,
the reset will automatically be asserted and deasserted as part of the
power-ungating procedure.
On some Jetson TX2 boards, doing an additional assert and deassert of
the GPU outside of the power-ungate
From: Thierry Reding
When Nouveau is instantiated on top of a platform device, the dev->pdev
field will be NULL and calling pci_disable_device() will crash. Move the
PCI disabling code to the PCI specific driver removal code.
Reviewed-by: Lyude Paul
Signed-off-by: Thierry Reding
---
drivers/g
From: Thierry Reding
The GPUs found on Tegra SoCs have registers that can be used to read the
WPR configuration. Use these registers instead of reaching into the
memory controller's register space to read the same information.
Signed-off-by: Thierry Reding
---
.../drm/nouveau/nvkm/subdev/secbo
From: Thierry Reding
This dummy implementation is useful to avoid a dependency on the
IOMMU_API Kconfig symbol in drivers that can optionally use the IOMMU
API.
In order to fully use this, also move the struct iommu_fwspec definition
out of the IOMMU_API protected region.
Suggested-by: Ben Dook
From: Thierry Reding
Hi Ben,
here's a revised subset of the patches I had sent out a couple of weeks
ago. I've reworked the BAR2 accesses in the way that you had suggested,
which at least for GP10B turned out to be fairly trivial to do. I have
not looked in detail at this for GV11B yet, but a cu
From: Thierry Reding
gp10b uses the new engine enumeration mechanism introduced in the Pascal
architecture. As a result, the copy engine, which used to be at index 2
for prior Tegra GPU instantiations, has now moved to index 0. Fix up the
index and also use the gp100 variant of the copy engine cl
From: Thierry Reding
There are extra registers that need to be programmed to make the level 2
cache work on GP10B, such as the stream ID register that is used when an
SMMU is used to translate memory addresses.
Signed-off-by: Thierry Reding
---
Changes in v2:
- remove IOMMU_API protection to in
From: Thierry Reding
If the GPU clock has not had a rate set, initialize it to the maximum
clock rate to make sure it does run.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/nouveau/nvkm/engine/device/tegra.c | 12
1 file changed, 12 insertions(+)
diff --git a/drivers/gpu/drm
From: Thierry Reding
There is no BAR2 on GP10B and there is no need to map through BAR2
because all memory is shared between the GPU and the CPU. Add a custom
implementation of the fault sub-device that uses nvkm_memory_addr()
instead of nvkm_memory_bar2() to return the address of a pinned fault
From: Thierry Reding
When this field was added in commit 5702ee24182f ("ACPI/IORT: Check ATS
capability in root complex nodes"), the kerneldoc comment wasn't updated
at the same time.
Signed-off-by: Thierry Reding
---
include/linux/iommu.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/in
https://bugs.freedesktop.org/show_bug.cgi?id=100228
--- Comment #44 from Andrey Melentyev ---
Got some time to test runpm_fixes branch by Karol Herbst:
https://github.com/karolherbst/linux/commits/runpm_fixes
Applying the two latest commits bbb0b9a16c86fc54fe296df73000da3fba4e91b6 and
749a9c843f
From: Colin Ian King
Don't populate const arrays on the stack but instead make them
static. Makes the object code smaller by 6 bytes.
Before:
textdata bss dec hex filename
1030759692 0 112767 1b87f nvkm/subdev/bios/init.o
After:
textdata bss dec
https://bugs.freedesktop.org/show_bug.cgi?id=112197
Bug ID: 112197
Summary: Black screen after resume
Product: xorg
Version: unspecified
Hardware: x86-64 (AMD64)
OS: Linux (All)
Status: NEW
Severity: criti
18 matches
Mail list logo