Re: [Nouveau] [PATCH v2 2/4] gpio: fail if gpu external power is missing

2019-07-15 Thread Ben Skeggs
On Mon, 15 Jul 2019 at 22:26, Ilia Mirkin wrote: > > Please add a config override to skip this, since we'll invariably get > it wrong for some setup, and should be able to provide users with > workarounds while the issue is being worked out. Yeah, this makes me nervous as well. In the very least,

Re: [Nouveau] Questions on Maxwell 2nd Gen Compute Kernels/Shaders

2019-07-15 Thread Ilia Mirkin
On Mon, Jul 15, 2019 at 2:34 PM Fernando Sahmkow wrote: > > So we have been busy implementing the compute engine lately but we have > discovered a few issues with Compute Shaders. I hope you guys can answer some > questions. > > 1st How do I determine the size of Compute Shaders/Kernel Local Mem

[Nouveau] Questions on Maxwell 2nd Gen Compute Kernels/Shaders

2019-07-15 Thread Fernando Sahmkow
So we have been busy implementing the compute engine lately but we have discovered a few issues with Compute Shaders. I hope you guys can answer some questions. 1st How do I determine the size of Compute Shaders/Kernel Local Memory ? In Pipeline shaders the size is included in the header but Compu

[Nouveau] [PATCH v2 5/6] drm/nouveau: utilize subconnector property for DP

2019-07-15 Thread Oleg Vasilev
Since DP-specific information is stored in driver's structures, every driver needs to implement subconnector property by itself. Signed-off-by: Oleg Vasilev Cc: nouveau@lists.freedesktop.org --- drivers/gpu/drm/nouveau/nouveau_connector.c | 13 + drivers/gpu/drm/nouveau/nouveau_dp.c

Re: [Nouveau] [PATCH v2 2/4] gpio: fail if gpu external power is missing

2019-07-15 Thread Ilia Mirkin
Please add a config override to skip this, since we'll invariably get it wrong for some setup, and should be able to provide users with workarounds while the issue is being worked out. On Mon, Jul 15, 2019 at 5:43 AM Mark Menzynski wrote: > > Currently, nouveau doesn't check if GPU is missing pow

Re: [Nouveau] [PATCH v2 5/6] drm/nouveau: utilize subconnector property for DP

2019-07-15 Thread Emil Velikov
Hi Oleg, On 2019/07/15, Oleg Vasilev wrote: > Since DP-specific information is stored in driver's structures, every > driver needs to implement subconnector property by itself. > > Signed-off-by: Oleg Vasilev > Cc: nouveau@lists.freedesktop.org > --- > drivers/gpu/drm/nouveau/nouveau_connector.

[Nouveau] [PATCH v2 0/4] Refuse to load if power cables are not connected

2019-07-15 Thread Mark Menzynski
Trello: https://trello.com/c/admzDRvd/50-reduce-performance-refuse-to-boot-if-not-all-the-power-connectors-are-plugged Fixed commit messages. Mark Menzynski (4): bios/gpio: sort gpios by values gpio: fail if gpu external power is missing gpio: check the gpio function 16 in the power check

[Nouveau] [PATCH v2 4/4] gpio: check function 76 in the power check as well

2019-07-15 Thread Mark Menzynski
Added GPIO is "Power Alert". It's uncertain if this GPIO is set on GPU initialization or only if a change is detected by the GPU at runtime. This GPIO can be found on Tesla and sometimes on Fermi GPUs. Untested, wrote according to documentation. Signed-off-by: Mark Menzynski --- drm/nouveau/in

[Nouveau] [PATCH v2 3/4] gpio: check the gpio function 16 in the power check as well

2019-07-15 Thread Mark Menzynski
Added GPIO is "Thermal and External Power Detect". It's uncertain if this GPIO is set on GPU initialization or only if a change is detected by the GPU at runtime. This GPIO can be found in Rankine and Curie and rarely on Tesla GPUs VBIOS. Untested, wrote according to documentation. Signed-off-by

[Nouveau] [PATCH v2 2/4] gpio: fail if gpu external power is missing

2019-07-15 Thread Mark Menzynski
Currently, nouveau doesn't check if GPU is missing power. This patch makes nouveau fail when this happens on latest GPUs. It checks GPIO function 121 (External Power Emergency), which should detect power problems on GPU initialization. Tested on TU104, GP106 and GF100. Signed-off-by: Mark Menzyn

[Nouveau] [PATCH v2 1/4] bios/gpio: sort gpios by values

2019-07-15 Thread Mark Menzynski
One gpio was in wrong place, moved it for better readability. Signed-off-by: Mark Menzynski --- drm/nouveau/include/nvkm/subdev/bios/gpio.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drm/nouveau/include/nvkm/subdev/bios/gpio.h b/drm/nouveau/include/nvkm/subdev/bios/gpio