Re: [PATCH libdrm 0/4] Dynamicly disable suites and tets.

2017-11-13 Thread Andrey Grodzovsky
On 11/13/2017 10:27 AM, Christian König wrote: Am 13.11.2017 um 15:57 schrieb Andrey Grodzovsky: On 11/13/2017 07:39 AM, Christian König wrote: Am 13.11.2017 um 12:32 schrieb Michel Dänzer: On 12/11/17 10:35 AM, Christian König wrote: A few comments on the code: +/* Validate bo size is b

Re: [PATCH libdrm 0/4] Dynamicly disable suites and tets.

2017-11-13 Thread Christian König
Am 13.11.2017 um 15:57 schrieb Andrey Grodzovsky: On 11/13/2017 07:39 AM, Christian König wrote: Am 13.11.2017 um 12:32 schrieb Michel Dänzer: On 12/11/17 10:35 AM, Christian König wrote: A few comments on the code: +/* Validate bo size is bit bigger then the request domain */ +static inlin

Re: [PATCH libdrm 0/4] Dynamicly disable suites and tets.

2017-11-13 Thread Andrey Grodzovsky
On 11/13/2017 07:39 AM, Christian König wrote: Am 13.11.2017 um 12:32 schrieb Michel Dänzer: On 12/11/17 10:35 AM, Christian König wrote: A few comments on the code: +/* Validate bo size is bit bigger then the request domain */ +static inline bool amdgpu_bo_validate_bo_size(struct amdgpu_dev

Re: [PATCH libdrm 0/4] Dynamicly disable suites and tets.

2017-11-13 Thread Christian König
Am 13.11.2017 um 12:32 schrieb Michel Dänzer: On 12/11/17 10:35 AM, Christian König wrote: A few comments on the code: +/* Validate bo size is bit bigger then the request domain */ +static inline bool amdgpu_bo_validate_bo_size(struct amdgpu_device *adev, +                      unsigned long s

Re: [PATCH libdrm 0/4] Dynamicly disable suites and tets.

2017-11-13 Thread Michel Dänzer
On 12/11/17 10:35 AM, Christian König wrote: > A few comments on the code: > >> +/* Validate bo size is bit bigger then the request domain */ >> +static inline bool amdgpu_bo_validate_bo_size(struct amdgpu_device >> *adev, >> +                      unsigned long size, u32 domain) > Drop the inline

Re: [PATCH libdrm 0/4] Dynamicly disable suites and tets.

2017-11-12 Thread Christian König
A few comments on the code: +/* Validate bo size is bit bigger then the request domain */ +static inline bool amdgpu_bo_validate_bo_size(struct amdgpu_device *adev, +                      unsigned long size, u32 domain) Drop the inline keyword and the second _bo_ in the name here. +{ +    str

Re: [PATCH libdrm 0/4] Dynamicly disable suites and tets.

2017-11-10 Thread Andrey Grodzovsky
On 11/10/2017 10:48 AM, Christian König wrote: Am 10.11.2017 um 16:36 schrieb Andrey Grodzovsky: On 11/10/2017 07:17 AM, Christian König wrote: Series is Acked-by: Christian König . Please note that I think your OOM killer test shows quite a bug we currently have in the kernel driver. A

Re: [PATCH libdrm 0/4] Dynamicly disable suites and tets.

2017-11-10 Thread Christian König
Am 10.11.2017 um 16:36 schrieb Andrey Grodzovsky: On 11/10/2017 07:17 AM, Christian König wrote: Series is Acked-by: Christian König . Please note that I think your OOM killer test shows quite a bug we currently have in the kernel driver. A single allocation of 1TB shouldn't trigger the OO

Re: [PATCH libdrm 0/4] Dynamicly disable suites and tets.

2017-11-10 Thread Andrey Grodzovsky
On 11/10/2017 07:17 AM, Christian König wrote: Series is Acked-by: Christian König . Please note that I think your OOM killer test shows quite a bug we currently have in the kernel driver. A single allocation of 1TB shouldn't trigger the OOM killer, but rather be reacted immediately. May

Re: [PATCH libdrm 0/4] Dynamicly disable suites and tets.

2017-11-10 Thread Christian König
Am 10.11.2017 um 13:17 schrieb Christian König: Series is Acked-by: Christian König . Please note that I think your OOM killer test shows quite a bug we currently have in the kernel driver. A single allocation of 1TB shouldn't trigger the OOM killer, but rather be reacted immediately. Typo

Re: [PATCH libdrm 0/4] Dynamicly disable suites and tets.

2017-11-10 Thread Christian König
Series is Acked-by: Christian König . Please note that I think your OOM killer test shows quite a bug we currently have in the kernel driver. A single allocation of 1TB shouldn't trigger the OOM killer, but rather be reacted immediately. Instead I expected that we need to do multiple 1GB al

[PATCH libdrm 0/4] Dynamicly disable suites and tets.

2017-11-09 Thread Andrey Grodzovsky
THe following patch series intoroduce dynamic tests dusabling/enabling in amdgpu tester using Cunit API. Today test suits that don't apply to specific HW just return success w/o executing while single tests that can't be executed properly are commented out. Suits are diasbled based on hooks the