From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alexey.Brodkin@synopsys.com (Alexey Brodkin) Date: Fri, 10 Jun 2016 13:23:22 +0000 Subject: [PATCH 03/27] drm/arc: Actually bother with handling atomic events. In-Reply-To: References: <1465388359-8070-1-git-send-email-daniel.vetter@ffwll.ch> <1465388359-8070-3-git-send-email-daniel.vetter@ffwll.ch> <20160608143004.GE3363@phenom.ffwll.local> <1465469639.3203.57.camel@synopsys.com> <20160609122631.GO3363@phenom.ffwll.local> <1465476465.3203.66.camel@synopsys.com> <20160609132327.GQ3363@phenom.ffwll.local> <1465478829.3203.68.camel@synopsys.com> <20160609135222.GS3363@phenom.ffwll.local> <1465482496.3203.73.camel@synopsys.com> List-ID: Message-ID: <1465564954.2942.40.camel@synopsys.com> To: linux-snps-arc@lists.infradead.org Hi Daniel, On Thu, 2016-06-09@16:37 +0200, Daniel Vetter wrote: > On Thu, Jun 9, 2016@4:31 PM, Daniel Vetter wrote: > > > > On Thu, Jun 9, 2016 at 4:29 PM, Alexey Brodkin > > wrote: > > > > > > Hi Daniel, > > > > > > On Thu, 2016-06-09@15:52 +0200, Daniel Vetter wrote: > > > > > > > > > > > > The fake implementation is fundamentally racy, and I don't want to write > > > > helpers which can't be used correctly. Anyway I think without this patch > > > > (or something similar) arcpgu will stall badly with the new nonblocking > > > > helpers, because arcpgu didn't bother at all to implement nonblocking. Can > > > > you pls ack this, or even better, test the entire patch series? The > > > > helpers themselves should work, but in all 5 drivers tested thus far they > > > > discovered some bugs. > > > Sure I will happily test this series. > > > The only question then is what should I use as a proper base? > > It should apply on drm-next from Dave. > > And indeed it won't work at all because arcpgu doesn't call > drm_crtc_handle_vblank anywhere. So you need to add your patch to > enable vblank interrupts somewhere. Note that as long as you leave > max_vblank_counter as 0, the only bits you need is drm_vblank_init and > drm_crtc_handle_vblanke() from the irq handler. So is there any sense in testing that series if vblank interrupt is not yet supported (I'm looking forward to implementing it sometime soon but definitely I'm not there yet)? -Alexey From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alexey Brodkin Subject: Re: [PATCH 03/27] drm/arc: Actually bother with handling atomic events. Date: Fri, 10 Jun 2016 13:23:22 +0000 Message-ID: <1465564954.2942.40.camel@synopsys.com> References: <1465388359-8070-1-git-send-email-daniel.vetter@ffwll.ch> <1465388359-8070-3-git-send-email-daniel.vetter@ffwll.ch> <20160608143004.GE3363@phenom.ffwll.local> <1465469639.3203.57.camel@synopsys.com> <20160609122631.GO3363@phenom.ffwll.local> <1465476465.3203.66.camel@synopsys.com> <20160609132327.GQ3363@phenom.ffwll.local> <1465478829.3203.68.camel@synopsys.com> <20160609135222.GS3363@phenom.ffwll.local> <1465482496.3203.73.camel@synopsys.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Content-Language: en-US Content-ID: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-snps-arc" Errors-To: linux-snps-arc-bounces+gla-linux-snps-arc=m.gmane.org@lists.infradead.org To: "daniel@ffwll.ch" Cc: "daniel.vetter@intel.com" , "linux-snps-arc@lists.infradead.org" , "maarten.lankhorst@linux.intel.com" , "dri-devel@lists.freedesktop.org" List-Id: dri-devel@lists.freedesktop.org Hi Daniel, On Thu, 2016-06-09 at 16:37 +0200, Daniel Vetter wrote: > On Thu, Jun 9, 2016 at 4:31 PM, Daniel Vetter wrote: > > > > On Thu, Jun 9, 2016 at 4:29 PM, Alexey Brodkin > > wrote: > > > > > > Hi Daniel, > > > > > > On Thu, 2016-06-09 at 15:52 +0200, Daniel Vetter wrote: > > > > > > > > > > > > The fake implementation is fundamentally racy, and I don't want to write > > > > helpers which can't be used correctly. Anyway I think without this patch > > > > (or something similar) arcpgu will stall badly with the new nonblocking > > > > helpers, because arcpgu didn't bother at all to implement nonblocking. Can > > > > you pls ack this, or even better, test the entire patch series? The > > > > helpers themselves should work, but in all 5 drivers tested thus far they > > > > discovered some bugs. > > > Sure I will happily test this series. > > > The only question then is what should I use as a proper base? > > It should apply on drm-next from Dave. > > And indeed it won't work at all because arcpgu doesn't call > drm_crtc_handle_vblank anywhere. So you need to add your patch to > enable vblank interrupts somewhere. Note that as long as you leave > max_vblank_counter as 0, the only bits you need is drm_vblank_init and > drm_crtc_handle_vblanke() from the irq handler. So is there any sense in testing that series if vblank interrupt is not yet supported (I'm looking forward to implementing it sometime soon but definitely I'm not there yet)? -Alexey