[RFC] drm/dp: move hw_mutex up the call stack

2015-12-09 Thread Ville Syrjälä
On Wed, Dec 09, 2015 at 12:26:11PM -0500, Rob Clark wrote: > On Wed, Dec 9, 2015 at 11:25 AM, Ville Syrjälä > wrote: > > On Wed, Dec 09, 2015 at 05:51:43PM +0200, Jani Nikula wrote: > >> On Wed, 09 Dec 2015, Rob Clark wrote: > >> > On Fri, Oct 16, 2015 at 2:54 PM, Rob Clark > >> > wrote: > >>

[RFC] drm/dp: move hw_mutex up the call stack

2015-12-09 Thread Ville Syrjälä
On Wed, Dec 09, 2015 at 05:51:43PM +0200, Jani Nikula wrote: > On Wed, 09 Dec 2015, Rob Clark wrote: > > On Fri, Oct 16, 2015 at 2:54 PM, Rob Clark wrote: > >> 1) don't let other threads trying to bang on aux channel interrupt the > >> defer timeout/logic > >> 2) don't let other threads interrupt

[RFC] drm/dp: move hw_mutex up the call stack

2015-12-09 Thread Jani Nikula
On Wed, 09 Dec 2015, Jani Nikula wrote: > On Wed, 09 Dec 2015, Rob Clark wrote: >> On Fri, Oct 16, 2015 at 2:54 PM, Rob Clark wrote: >>> 1) don't let other threads trying to bang on aux channel interrupt the >>> defer timeout/logic >>> 2) don't let other threads interrupt the i2c over aux logic

[RFC] drm/dp: move hw_mutex up the call stack

2015-12-09 Thread Jani Nikula
On Wed, 09 Dec 2015, Rob Clark wrote: > On Fri, Oct 16, 2015 at 2:54 PM, Rob Clark wrote: >> 1) don't let other threads trying to bang on aux channel interrupt the >> defer timeout/logic >> 2) don't let other threads interrupt the i2c over aux logic >> >> --- >> This wasn't actually fixing things

[RFC] drm/dp: move hw_mutex up the call stack

2015-12-09 Thread Rob Clark
On Wed, Dec 9, 2015 at 2:37 PM, Ville Syrjälä wrote: > On Wed, Dec 09, 2015 at 12:26:11PM -0500, Rob Clark wrote: >> On Wed, Dec 9, 2015 at 11:25 AM, Ville Syrjälä >> wrote: >> > On Wed, Dec 09, 2015 at 05:51:43PM +0200, Jani Nikula wrote: >> >> On Wed, 09 Dec 2015, Rob Clark wrote: >> >> >

[RFC] drm/dp: move hw_mutex up the call stack

2015-12-09 Thread Rob Clark
On Wed, Dec 9, 2015 at 11:25 AM, Ville Syrjälä wrote: > On Wed, Dec 09, 2015 at 05:51:43PM +0200, Jani Nikula wrote: >> On Wed, 09 Dec 2015, Rob Clark wrote: >> > On Fri, Oct 16, 2015 at 2:54 PM, Rob Clark wrote: >> >> 1) don't let other threads trying to bang on aux channel interrupt the >> >

[RFC] drm/dp: move hw_mutex up the call stack

2015-12-09 Thread Rob Clark
On Wed, Dec 9, 2015 at 10:53 AM, Jani Nikula wrote: > On Wed, 09 Dec 2015, Jani Nikula wrote: >> On Wed, 09 Dec 2015, Rob Clark wrote: >>> On Fri, Oct 16, 2015 at 2:54 PM, Rob Clark wrote: 1) don't let other threads trying to bang on aux channel interrupt the defer timeout/logic

[RFC] drm/dp: move hw_mutex up the call stack

2015-12-09 Thread Rob Clark
On Fri, Oct 16, 2015 at 2:54 PM, Rob Clark wrote: > 1) don't let other threads trying to bang on aux channel interrupt the > defer timeout/logic > 2) don't let other threads interrupt the i2c over aux logic > > --- > This wasn't actually fixing things w/ problematic monitor, but seems > like gener

[RFC] drm/dp: move hw_mutex up the call stack

2015-10-16 Thread Rob Clark
1) don't let other threads trying to bang on aux channel interrupt the defer timeout/logic 2) don't let other threads interrupt the i2c over aux logic --- This wasn't actually fixing things w/ problematic monitor, but seems like generally a good idea. At least AFAIU you shouldn't allow the sequen