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