Re: [PATCH] gem_pin: Be explicit about GGTT needs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Nov 7, 2013 at 7:11 PM, Ben Widawsky <ben@xxxxxxxxxxxx> wrote:
>> > @@ -63,7 +65,7 @@ static void exec(int fd, uint32_t handle, uint32_t offset)
>> >     gem_exec[0].relocs_ptr = (uintptr_t) gem_reloc;
>> >     gem_exec[0].alignment = 0;
>> >     gem_exec[0].offset = 0;
>> > -   gem_exec[0].flags = 0;
>> > +   gem_exec[0].flags = LOCAL__EXEC_OBJ_NEEDS_GTT;
>>
>> This only really works with the aliasing ppgtt stuff on gen6, I'd just
>> skip the test -it's not really useful with real ppgtt.
>> -Daniel
>
> It is really useful with real ppgtt. Please rethink your assertion.

The test links up the global gtt used by the current pin ioctl with
the ppgtt. That's not useful, except when they alias. If we want
soft-pinning, then we need a new ioctl mode to return the right
address from the right address space. Which means a new (sub)test.
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux