Re: [PATCH 2/2] drm/ttm: Use pr_debug for all output from ttm_bo_evict

2018-12-06 Thread Michel Dänzer
On 2018-12-06 5:46 p.m., Joe Perches wrote: > On Thu, 2018-12-06 at 17:39 +0800, Zhang, Jerry(Junwei) wrote: >> On 12/6/18 5:33 PM, Koenig, Christian wrote: >>> Am 06.12.18 um 10:09 schrieb Michel Dänzer: On 2018-12-06 3:43 a.m., Zhang, Jerry(Junwei) wrote: > On 12/6/18 12:56 AM, Michel Dä

Re: [PATCH 2/2] drm/ttm: Use pr_debug for all output from ttm_bo_evict

2018-12-06 Thread Joe Perches
On Thu, 2018-12-06 at 17:39 +0800, Zhang, Jerry(Junwei) wrote: > On 12/6/18 5:33 PM, Koenig, Christian wrote: > > Am 06.12.18 um 10:09 schrieb Michel Dänzer: > > > On 2018-12-06 3:43 a.m., Zhang, Jerry(Junwei) wrote: > > > > On 12/6/18 12:56 AM, Michel Dänzer wrote: > > > > > From: Michel Dänzer >

Re: [PATCH 2/2] drm/ttm: Use pr_debug for all output from ttm_bo_evict

2018-12-06 Thread Michel Dänzer
On 2018-12-06 10:49 a.m., Christian König wrote: > Am 06.12.18 um 10:39 schrieb Zhang, Jerry(Junwei): >> On 12/6/18 5:33 PM, Koenig, Christian wrote: >>> Am 06.12.18 um 10:09 schrieb Michel Dänzer: On 2018-12-06 3:43 a.m., Zhang, Jerry(Junwei) wrote: > On 12/6/18 12:56 AM, Michel Dänzer wr

Re: [PATCH 2/2] drm/ttm: Use pr_debug for all output from ttm_bo_evict

2018-12-06 Thread Christian König
Am 06.12.18 um 10:39 schrieb Zhang, Jerry(Junwei): On 12/6/18 5:33 PM, Koenig, Christian wrote: Am 06.12.18 um 10:09 schrieb Michel Dänzer: On 2018-12-06 3:43 a.m., Zhang, Jerry(Junwei) wrote: On 12/6/18 12:56 AM, Michel Dänzer wrote: From: Michel Dänzer All the output is related, so it sho

Re: [PATCH 2/2] drm/ttm: Use pr_debug for all output from ttm_bo_evict

2018-12-06 Thread Michel Dänzer
On 2018-12-06 10:38 a.m., Michel Dänzer wrote: > On 2018-12-06 10:33 a.m., Koenig, Christian wrote: >> Am 06.12.18 um 10:09 schrieb Michel Dänzer: >>> On 2018-12-06 3:43 a.m., Zhang, Jerry(Junwei) wrote: On 12/6/18 12:56 AM, Michel Dänzer wrote: > From: Michel Dänzer > > All the o

Re: [PATCH 2/2] drm/ttm: Use pr_debug for all output from ttm_bo_evict

2018-12-06 Thread Zhang, Jerry(Junwei)
On 12/6/18 5:33 PM, Koenig, Christian wrote: Am 06.12.18 um 10:09 schrieb Michel Dänzer: On 2018-12-06 3:43 a.m., Zhang, Jerry(Junwei) wrote: On 12/6/18 12:56 AM, Michel Dänzer wrote: From: Michel Dänzer All the output is related, so it should all be printed the same way. Some of it was usin

Re: [PATCH 2/2] drm/ttm: Use pr_debug for all output from ttm_bo_evict

2018-12-06 Thread Michel Dänzer
On 2018-12-06 10:33 a.m., Koenig, Christian wrote: > Am 06.12.18 um 10:09 schrieb Michel Dänzer: >> On 2018-12-06 3:43 a.m., Zhang, Jerry(Junwei) wrote: >>> On 12/6/18 12:56 AM, Michel Dänzer wrote: From: Michel Dänzer All the output is related, so it should all be printed the same

Re: [PATCH 2/2] drm/ttm: Use pr_debug for all output from ttm_bo_evict

2018-12-06 Thread Koenig, Christian
Am 06.12.18 um 10:09 schrieb Michel Dänzer: > On 2018-12-06 3:43 a.m., Zhang, Jerry(Junwei) wrote: >> On 12/6/18 12:56 AM, Michel Dänzer wrote: >>> From: Michel Dänzer >>> >>> All the output is related, so it should all be printed the same way. >>> Some of it was using pr_debug, but some of it app

Re: [PATCH 2/2] drm/ttm: Use pr_debug for all output from ttm_bo_evict

2018-12-06 Thread Michel Dänzer
On 2018-12-06 3:43 a.m., Zhang, Jerry(Junwei) wrote: > On 12/6/18 12:56 AM, Michel Dänzer wrote: >> From: Michel Dänzer >> >> All the output is related, so it should all be printed the same way. >> Some of it was using pr_debug, but some of it appeared in dmesg by >> default. The caller should han

Re: [PATCH 2/2] drm/ttm: Use pr_debug for all output from ttm_bo_evict

2018-12-05 Thread Zhang, Jerry(Junwei)
On 12/6/18 12:56 AM, Michel Dänzer wrote: From: Michel Dänzer All the output is related, so it should all be printed the same way. Some of it was using pr_debug, but some of it appeared in dmesg by default. The caller should handle failure, so there's no need to spam dmesg with potentially quit

[PATCH 2/2] drm/ttm: Use pr_debug for all output from ttm_bo_evict

2018-12-05 Thread Michel Dänzer
From: Michel Dänzer All the output is related, so it should all be printed the same way. Some of it was using pr_debug, but some of it appeared in dmesg by default. The caller should handle failure, so there's no need to spam dmesg with potentially quite a lot of output by default. Signed-off-by