This patch transitions the execbuf engine selection away from using the
ring nomenclature - though we still refer to the user's incoming
selector as their user_ring_id.
Signed-off-by: Chris Wilson
---
drivers/gpu/drm/i915/i915_drv.h| 4 ++--
drivers/gpu/drm/i915/i915_gem.c
The state stored in this struct is not only the information about the
buffer object, but the ring used to communicate with the hardware. Using
buffer here is overly specific and, for me at least, conflates with the
notion of buffer objects themselves.
s/struct intel_ringbuffer/struct intel_ring/
s
On ke, 2016-07-20 at 14:12 +0100, Chris Wilson wrote:
> Since requests can no longer be generated as a side-effect of
> intel_ring_begin(), we know that the seqno will be unchanged during
> ring-emission. This predicatablity then means we do not have to check
> for the seqno wrapping around whilst
On ke, 2016-07-20 at 14:12 +0100, Chris Wilson wrote:
> Now that emitting requests is identical between legacy and execlists, we
> can use the same function to build up the ring for submitting to either
> engine. (With the exception of i915_switch_contexts(), but in time that
> will also be handled
On 2016 Jan 26 (Tue) at 21:51:10 +0100 (+0100), Ren? Rebe wrote:
:
:On Jan 26, 2016, at 19:03, Andreas Radke wrote:
:
:> Am Wed, 6 May 2015 22:02:57 +0200
:> schrieb Julien Cristau :
:>
:>> On Sun, Dec 21, 2014 at 14:47:58 +, Chris Wilson wrote:
:>>
:>>> Snapshot 2.99.917 (2014-12-21)
:>>> =
Em Wed, 20 Jul 2016 20:35:09 +0200
Markus Heiser escreveu:
> Am 20.07.2016 um 14:20 schrieb Mauro Carvalho Chehab
> :
>
> > Em Tue, 19 Jul 2016 14:36:50 +0200
> > Daniel Vetter escreveu:
> >
> >> On Tue, Jul 19, 2016 at 01:42:55PM +0200, Daniel Vetter wrote:
> >>> These are the leftovers
Am 20.07.2016 um 13:27 schrieb Daniel Vetter :
> On Wed, Jul 20, 2016 at 12:55 PM, Markus Heiser
> wrote:
>> Hi Daniel, hi Mauro,
>>
>> Am 19.07.2016 um 17:32 schrieb Daniel Vetter :
>>
>>> On Tue, Jul 19, 2016 at 5:25 PM, Daniel Vetter
>>> wrote:
On Tue, Jul 19, 2016 at 4:59 PM, Markus
Am 20.07.2016 um 14:20 schrieb Mauro Carvalho Chehab :
> Em Tue, 19 Jul 2016 14:36:50 +0200
> Daniel Vetter escreveu:
>
>> On Tue, Jul 19, 2016 at 01:42:55PM +0200, Daniel Vetter wrote:
>>> These are the leftovers I could only track down using keep_warnings =
>>> True. For some of them we might
Em Wed, 20 Jul 2016 14:29:01 +0200
Markus Heiser escreveu:
> Am 20.07.2016 um 13:27 schrieb Daniel Vetter :
>
> > On Wed, Jul 20, 2016 at 12:55 PM, Markus Heiser
> > wrote:
> >> Hi Daniel, hi Mauro,
> >>
> >> Am 19.07.2016 um 17:32 schrieb Daniel Vetter :
> >>
> >>> On Tue, Jul 19, 2016 a
Hi Daniel, hi Mauro,
Am 19.07.2016 um 17:32 schrieb Daniel Vetter :
> On Tue, Jul 19, 2016 at 5:25 PM, Daniel Vetter wrote:
>> On Tue, Jul 19, 2016 at 4:59 PM, Markus Heiser
>> wrote:
>>>
>>> Am 19.07.2016 um 13:42 schrieb Daniel Vetter :
>>>
Unfortunately warnings generated after parsin
Em Tue, 19 Jul 2016 14:36:50 +0200
Daniel Vetter escreveu:
> On Tue, Jul 19, 2016 at 01:42:55PM +0200, Daniel Vetter wrote:
> > These are the leftovers I could only track down using keep_warnings =
> > True. For some of them we might want to update our style guide on how
> > to reference structur
Am 19.07.2016 um 13:42 schrieb Daniel Vetter :
> Unfortunately warnings generated after parsing in sphinx can end up
> with entirely bogus files and line numbers as sources. Strangely for
> outright errors this is not a problem. Trying to convert warnings to
> errors also doesn't fix it.
>
> The
On ke, 2016-07-20 at 14:12 +0100, Chris Wilson wrote:
> As GEN6+ is now a simple variant on the basic breadcrumbs + tail write,
> reuse the common code.
>
Code motion could always be separate patch for easier review of what
functions were changed.
Reviewed-by: Joonas Lahtinen
> Signed-off-by:
On Fri, Jul 22, 2016 at 11:15:59AM +0300, Joonas Lahtinen wrote:
> On ke, 2016-07-20 at 14:12 +0100, Chris Wilson wrote:
> > Rather than pass in the num_dwords that the caller wishes to use after
> > the signal command packet, split the breadcrumb emission into two phases
> > and have both the sign
On Fri, Jul 22, 2016 at 11:03:19AM +0300, Joonas Lahtinen wrote:
> On ke, 2016-07-20 at 14:12 +0100, Chris Wilson wrote:
> > @@ -1904,8 +1898,10 @@ logical_ring_default_vfuncs(struct intel_engine_cs
> > *engine)
> > {
> > /* Default vfuncs which can be overriden by each engine. */
> > eng
On ke, 2016-07-20 at 14:12 +0100, Chris Wilson wrote:
> Rather than pass in the num_dwords that the caller wishes to use after
> the signal command packet, split the breadcrumb emission into two phases
> and have both the signal and breadcrumb individiually acquire space on
> the ring. This makes t
Hi Dave,
Suddenly everyone shows up with their trivial patch series!
- piles of if (!ptr) check removals from Markus Elfring
- more of_node_put fixes from Peter Chen
- make fbdev support really optional in all drivers (except vmwgfx),
somehow this fell through the cracks when we did all the hard
On ke, 2016-07-20 at 14:12 +0100, Chris Wilson wrote:
> Move request submission from emit_request into its own common vfunc
> from i915_add_request().
>
> Signed-off-by: Chris Wilson
> ---
> drivers/gpu/drm/i915/i915_gem_request.c| 7 +++
> drivers/gpu/drm/i915/i915_guc_submission.c |
On Thu, Jul 21, 2016 at 07:57:39AM +0100, Chris Wilson wrote:
> As the interrupt wakeup counter only increments when we have a waiter,
> before testing to see if that counter is unchanged we have to first
> check that we do expect it to change (i.e. we have a waiter).
>
> Signed-off-by: Chris Wils
== Series Details ==
Series: drm/i915/skl: Finally fix watermarks (rev2)
URL : https://patchwork.freedesktop.org/series/10108/
State : success
== Summary ==
Series 10108v2 drm/i915/skl: Finally fix watermarks
http://patchwork.freedesktop.org/api/1.0/series/10108/revisions/2/mbox
fi-hsw-i7-47
101 - 120 of 120 matches
Mail list logo