Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 9 Jun 2017 18:40:44 +0300
From:      Andriy Gapon <avg@FreeBSD.org>
To:        Takahashi Yoshihiro <nyan@FreeBSD.org>, src-committers@FreeBSD.org, svn-src-all@FreeBSD.org, svn-src-head@FreeBSD.org
Subject:   Re: svn commit: r317591 - head/sys/dev/drm2/radeon
Message-ID:  <086f41db-c2a1-5367-6acb-59df190a43db@FreeBSD.org>
In-Reply-To: <03e500d5-76b1-fd38-04c6-90d85a2ddfa8@FreeBSD.org>
References:  <201704290947.v3T9lF0r075921@repo.freebsd.org> <03e500d5-76b1-fd38-04c6-90d85a2ddfa8@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help

[ping]

On 31/05/2017 08:22, Andriy Gapon wrote:
> On 29/04/2017 12:47, Takahashi Yoshihiro wrote:
>> Author: nyan
>> Date: Sat Apr 29 09:47:15 2017
>> New Revision: 317591
>> URL: https://svnweb.freebsd.org/changeset/base/317591
>>
>> Log:
>>   Add TUNABLE_INT to radeonkms driver parameters.
>>   They are required by PowerMac G5 DP.
>>   
>>   PR:		217852
>>   Submitted by:	Hiroo Ono
>>   MFC after:	1 week
> 
> I think that this solution that I had in my private tree is a little bit more
> powerful:
> https://people.freebsd.org/~avg/radeon_drv-knobs.diff
> 
> The advantage is that it is less intrusive and also provides read-only sysctl in
> addition to the tunables.
> Also, hw.drm.radeon seems like a better place for the knobs than just drm.radeon.
> 
> Do you mind if I commit this?
> 
>> Modified:
>>   head/sys/dev/drm2/radeon/radeon_drv.c
>>
>> Modified: head/sys/dev/drm2/radeon/radeon_drv.c
>> ==============================================================================
>> --- head/sys/dev/drm2/radeon/radeon_drv.c	Sat Apr 29 09:26:30 2017	(r317590)
>> +++ head/sys/dev/drm2/radeon/radeon_drv.c	Sat Apr 29 09:47:15 2017	(r317591)
>> @@ -127,54 +127,71 @@ int radeon_pcie_gen2 = -1;
>>  int radeon_msi = -1;
>>  int radeon_lockup_timeout = 10000;
>>  
>> +TUNABLE_INT("drm.radeon.no_wb", &radeon_no_wb);
>>  MODULE_PARM_DESC(no_wb, "Disable AGP writeback for scratch registers");
>>  module_param_named(no_wb, radeon_no_wb, int, 0444);
>>  
>> +TUNABLE_INT("drm.radeon.modeset", &radeon_modeset);
>>  MODULE_PARM_DESC(modeset, "Disable/Enable modesetting");
>>  module_param_named(modeset, radeon_modeset, int, 0400);
>>  
>> +TUNABLE_INT("drm.radeon.dynclks", &radeon_dynclks);
>>  MODULE_PARM_DESC(dynclks, "Disable/Enable dynamic clocks");
>>  module_param_named(dynclks, radeon_dynclks, int, 0444);
>>  
>> +TUNABLE_INT("drm.radeon.r4xx_atom", &radeon_r4xx_atom);
>>  MODULE_PARM_DESC(r4xx_atom, "Enable ATOMBIOS modesetting for R4xx");
>>  module_param_named(r4xx_atom, radeon_r4xx_atom, int, 0444);
>>  
>> +TUNABLE_INT("drm.radeon.vramlimit", &radeon_vram_limit);
>>  MODULE_PARM_DESC(vramlimit, "Restrict VRAM for testing");
>>  module_param_named(vramlimit, radeon_vram_limit, int, 0600);
>>  
>> +TUNABLE_INT("drm.radeon.agpmode", &radeon_agpmode);
>>  MODULE_PARM_DESC(agpmode, "AGP Mode (-1 == PCI)");
>>  module_param_named(agpmode, radeon_agpmode, int, 0444);
>>  
>> +TUNABLE_INT("drm.radeon.gartsize", &radeon_gart_size);
>>  MODULE_PARM_DESC(gartsize, "Size of PCIE/IGP gart to setup in megabytes (32, 64, etc)");
>>  module_param_named(gartsize, radeon_gart_size, int, 0600);
>>  
>> +TUNABLE_INT("drm.radeon.benchmark", &radeon_benchmarking);
>>  MODULE_PARM_DESC(benchmark, "Run benchmark");
>>  module_param_named(benchmark, radeon_benchmarking, int, 0444);
>>  
>> +TUNABLE_INT("drm.radeon.test", &radeon_testing);
>>  MODULE_PARM_DESC(test, "Run tests");
>>  module_param_named(test, radeon_testing, int, 0444);
>>  
>> +TUNABLE_INT("drm.radeon.connector_table", &radeon_connector_table);
>>  MODULE_PARM_DESC(connector_table, "Force connector table");
>>  module_param_named(connector_table, radeon_connector_table, int, 0444);
>>  
>> +TUNABLE_INT("drm.radeon.tv", &radeon_tv);
>>  MODULE_PARM_DESC(tv, "TV enable (0 = disable)");
>>  module_param_named(tv, radeon_tv, int, 0444);
>>  
>> +TUNABLE_INT("drm.radeon.audio", &radeon_audio);
>>  MODULE_PARM_DESC(audio, "Audio enable (1 = enable)");
>>  module_param_named(audio, radeon_audio, int, 0444);
>>  
>> +TUNABLE_INT("drm.radeon.disp_priority", &radeon_disp_priority);
>>  MODULE_PARM_DESC(disp_priority, "Display Priority (0 = auto, 1 = normal, 2 = high)");
>>  module_param_named(disp_priority, radeon_disp_priority, int, 0444);
>>  
>> +TUNABLE_INT("drm.radeon.hw_i2c", &radeon_hw_i2c);
>>  MODULE_PARM_DESC(hw_i2c, "hw i2c engine enable (0 = disable)");
>>  module_param_named(hw_i2c, radeon_hw_i2c, int, 0444);
>>  
>> +TUNABLE_INT("drm.radeon.pcie_gen2", &radeon_pcie_gen2);
>>  MODULE_PARM_DESC(pcie_gen2, "PCIE Gen2 mode (-1 = auto, 0 = disable, 1 = enable)");
>>  module_param_named(pcie_gen2, radeon_pcie_gen2, int, 0444);
>>  
>> +TUNABLE_INT("drm.radeon.msi", &radeon_msi);
>>  MODULE_PARM_DESC(msi, "MSI support (1 = enable, 0 = disable, -1 = auto)");
>>  module_param_named(msi, radeon_msi, int, 0444);
>>  
>> +TUNABLE_INT("drm.radeon.lockup_timeout", &radeon_lockup_timeout);
>>  MODULE_PARM_DESC(lockup_timeout, "GPU lockup timeout in ms (defaul 10000 = 10 seconds, 0 = disable)");
>>  module_param_named(lockup_timeout, radeon_lockup_timeout, int, 0444);
>>  
>>
> 
> 


-- 
Andriy Gapon



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?086f41db-c2a1-5367-6acb-59df190a43db>