Re: [PATCH 2/2] drm/radeon/kms: Schedule host path read cache flush through the ring

2010-01-07 Thread Alex Deucher
On Wed, Jan 6, 2010 at 3:51 PM, Jerome Glisse wrote: > On Wed, Jan 06, 2010 at 01:57:08PM -0500, Alex Deucher wrote: >> On Wed, Jan 6, 2010 at 1:29 PM, Jerome Glisse wrote: >> > R300 family will hard lockup if host path read cache flush is >> > done through MMIO to HOST_PATH_CNTL. But scheduling

Re: [PATCH 2/2] drm/radeon/kms: Schedule host path read cache flush through the ring

2010-01-06 Thread Jerome Glisse
On Wed, Jan 06, 2010 at 01:57:08PM -0500, Alex Deucher wrote: > On Wed, Jan 6, 2010 at 1:29 PM, Jerome Glisse wrote: > > R300 family will hard lockup if host path read cache flush is > > done through MMIO to HOST_PATH_CNTL. But scheduling same flush > > through ring seems harmless. This patch remo

Re: [PATCH 2/2] drm/radeon/kms: Schedule host path read cache flush through the ring

2010-01-06 Thread Alex Deucher
On Wed, Jan 6, 2010 at 1:29 PM, Jerome Glisse wrote: > R300 family will hard lockup if host path read cache flush is > done through MMIO to HOST_PATH_CNTL. But scheduling same flush > through ring seems harmless. This patch remove the hdp_flush > callback and add a flush after each fence emission

[PATCH 2/2] drm/radeon/kms: Schedule host path read cache flush through the ring

2010-01-06 Thread Jerome Glisse
R300 family will hard lockup if host path read cache flush is done through MMIO to HOST_PATH_CNTL. But scheduling same flush through ring seems harmless. This patch remove the hdp_flush callback and add a flush after each fence emission which means a flush after each IB schedule. Thus we should hav