Hi Juri,

As Amit explains, the coming weeks will be quite busy but i will
reorder the patchset as soon as i find available time.

Regards,
Vincent

On 7 August 2014 16:57, Juri Lelli <juri.le...@arm.com> wrote:
> Hi Amit,
>
> On 06/08/14 12:35, Amit Kucheria wrote:
>> Hi Juri,
>>
>> On Wed, Aug 6, 2014 at 3:38 PM, Juri Lelli <juri.le...@arm.com> wrote:
>>> Hi Vincent,
>>>
>>> I'm actually adding Chris to Cc: as he is also part of the
>>> original discussion about rt-app.
>>>
>>> On 05/08/14 22:45, Vincent Guittot wrote:
>>>> Hi Juri,
>>>>
>>>> Here is the pull request for the changes that create a workload generator 
>>>> tool
>>>>
>>>> Regards,
>>>>
>>>> The following changes since commit 
>>>> 17be4548c4260b80be623e0e1317e98a770dea7a:
>>>>
>>>>   copyright added (2014-04-11 09:28:27 +0200)
>>>>
>>>> are available in the git repository at:
>>>>
>>>>   git://git.linaro.org/power/rt-app.git master
>>>>
>>>> for you to fetch changes up to 8cbaad65d00e3c64c4f941139bc11f4d07822474:
>>>>
>>>>   add a web browsing use case (2014-08-01 15:43:47 +0200)
>>>
>>> Thanks for sharing changes in your repository, but IMHO it is
>>> somewhat difficult to review them in this form, as behavioural
>>> changes are intermixed with small fixes. Also, it would be good
>>> if we can integrate changes keeping git history clean. Ideally
>>> we could introduce functional changes followed by (or together
>>> with) examples showing what feature the change introduces. I
>>> know that this is time consuming, so I'm actually asking if you
>>> think you have some time to work on rearranging things.
>>
>> Vincent is on vacations until the end of the month. After that, I
>> expect he'll be busy with Linaro Connect preparations.
>>
>> Yes, the timing of all this sucks given that we want to announce the
>> tools this week to give 2 weeks of lead time before Kernel Summit.
>>
>>> Having a proper history could also ease review. I'd say we could
>>> try two different methods. I created a GitHub organization that
>>> now hosts the rt-app repo: https://github.com/scheduler-tools/rt-app.
>>> I also created a branch for the original version of the tool so that
>>> we can integrate changes on master.
>>>
>>>  Method 1) We use GitHub pull request feature to discuss changes and
>>>            finally integrate them.
>>>
>>>  Method 2) You post the patchset (based on master) on linaro-dev
>>>            mailing list, discussion happens on the mailing list
>>>            (everybody can participate), after discussion I apply
>>>            patches resulting from discussion on master.
>>>
>>> I'd personally prefer method 2 as it is simpler and can probably get
>>> contributions from a wider audience. What other thinks?
>>
>> I prefer Method 2 myself but it is going to be hard to get anything
>> refactored until late September.
>>
>
> So, I'd say we wait for the actual review process to happen when
> feasible anyway.
>
> Thanks,
>
> - Juri
>
>>> We also said that the name of the tool could be changed, any opinions
>>> on this? Something like workload-gen or wload-gen ?
>>
>> IMHO, the name change is just a nice to have and not critical to the
>> success of this project.
>>
>> Regards,
>> Amit
>>
>

_______________________________________________
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev

Reply via email to