Re: [PATCH 0/4] drm-intel-collector - review request

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

 



Thanks Paulo.

Thanks for the suggestion Ben, will do this on next round!

On Tue, Oct 8, 2013 at 10:57 AM, Paulo Zanoni <przanoni@xxxxxxxxx> wrote:
> 2013/10/7 Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx>:
>>
>> This is another drm-intel-collector push for review:
>> http://cgit.freedesktop.org/~vivijim/drm-intel/log/?h=drm-intel-collector
>>
>> Here goes the list in order for better reviewers assignment:
>>
>> Patch     drm/i915: check that the i965g/gm 4G limit is really obeyed - Reviewer: Damien Lespiau <damien.lespiau@xxxxxxxxx>
>> Patch     drm/i915: Asynchronously perform the set-base for a simple modeset - Reviewer: I would prefer Ville review this since he had concerns about sorround code. switch to fbcon is much faster with this patch so I let it here and will try to review by myself if I heard nothing else until next update.
>> Patch     drm/i915: implement another plane WM workaround for HSW - Reviewer: If no body else volunteer I'll do this review until next update.
>> Patch     drm/i915: Use the real cpu max frequency for ring scaling - Reviewed by Chris.
>>
>> Overall Process:
>>
>> drm-intel-collector - review request
>>  1. Daniel pushs drm-intel-testing (every 2 weeks)
>>  2. I rebase drm-intel-collector onto drm-intel-testing
>>  3. Add Reviewer: tag with voluntered reviewers. If you don't believe you should be assigned on a particular patch please don't get mad just tell you wont review or volunteer someone else.
>>  4. I resubmit remaining patches for review without picking any new (drm-intel-collector - review request)
>>
>> drm-intel-collector - updated
>>  5. One week later I collect all simple (1-2) patches that wasn't yet reviewed and not queued by Daniel from one testing update until another.
>>  6. Request automated QA's PRTS automated i-g-t tests comparing drm-intel-testing x drm-intel-collector
>>  7. If tests are ok I send the update notification or the patches as a series to intel-gfx mailing list for better tracking and to be reviewed. (drm-intel-collector - updated)
>>  8. Let me know volunteers for review new patches and also let me know if I've picked any patch that I shouldn't.
>>
>> There are some reasons that some patches can be left behind:
>> 1. Your patch didn't applied cleanly and I couldn't easily solve the conflicts.
>> 2. Kernel didn't compiled with your patch.
>> 3. I simply missed it. If you believe this is the case please warn me.
>>
>> Please help me to get these patches reviewed and queued by Daniel.
>>
>> Also, please let me know if you have further ideas how to improve this process.
>>
>> Thanks in advance,
>> Rodrigo.
>>
>>
>> Ben Widawsky (1):
>>   drm/i915: Use the real cpu max frequency for ring scaling
>>
>> Chris Wilson (1):
>>   drm/i915: Asynchronously perform the set-base for a simple modeset
>>
>> Daniel Vetter (1):
>>   drm/i915: check that the i965g/gm 4G limit is really obeyed
>>
>> Paulo Zanoni (1):
>>   drm/i915: implement another plane WM workaround for HSW
>
> Please just discard this one, it was replaced by other patches from
> Ville, Jani and Paulo.
>
>
>
>>
>>  drivers/gpu/drm/i915/i915_gem.c      |  3 +++
>>  drivers/gpu/drm/i915/intel_display.c | 20 ++++++++++++++++----
>>  drivers/gpu/drm/i915/intel_pm.c      | 17 +++++++++++------
>>  3 files changed, 30 insertions(+), 10 deletions(-)
>>
>> --
>> 1.8.1.4
>>
>> _______________________________________________
>> Intel-gfx mailing list
>> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
>> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
>
>
>
> --
> Paulo Zanoni



-- 
Rodrigo Vivi
Blog: http://blog.vivi.eng.br
_______________________________________________
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