Hi Vincent, On 26/08/14 12:11, Vincent Guittot wrote: > 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. >
No problem. Thanks, - Juri > 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
