On Wed, Apr 30, 2008 at 10:03 AM, David Miller <davem@davemloft.net> wrote:Just some comments: Analogous to that of the football team, everyone has an impt role to play. And u better let go of the ball as fast as u can, otherwise u are going to tire yourself out easily. So, in a development team, if u think there is some unequal distribution of workload, make noise. Or think of some means to do automatic loading of workload - specifically in the area of change review. (At other times, it is not easily to pass the load around.....eg, if the bug happened only on your machines and not on others?) 1. Generally, the more people reviewed the work, the higher chances the piece of work is ok. 2. If more variation of real-testing is done, the better. "variation" here means testing by users of different background skills, different applications running, and most impt - is the base kernel version where the patch is applied and tested. etc. 3. Based on the two numbers above alone, we can immediately have some measure of confidence of the patch - correct? 4. So if we can put all these in a web page - the patches itself, the reviewers/testers that have worked on it. When someone comes in and review, review counter increase by one. Or tester counter increased by one after testing. And I supposed everyone will attempt to cover those that are lesser covered by others - automatic loading of workload done in a distributed manner. Avoid having to fill in too much information though...u will discourage taking up the work, and let the participant spent more precious time on reviewing instead. So prior to consolidation of sources, just by looking at the numbers, u can see how successful the consolidation will be. If it is lesser tested, then avoid including it for consoldating...... Please comments...... -- Regards, Peter Teoh --
| jjohansen | [AppArmor 39/45] AppArmor: Profile loading and manipulation, pathname matching |
| Vladislav Bolkhovitin | Re: Integration of SCST in the mainstream Linux kernel |
| Heiko Carstens | Re: -mm merge plans for 2.6.23 -- sys_fallocate |
| Andrew Morton | 2.6.23-rc6-mm1 |
git: | |
| Gerrit Renker | [PATCH 27/37] dccp: Integration of dynamic feature activation - part 2 (server side) |
| Evgeniy Polyakov | Re: [BUG] New Kernel Bugs |
| Jarek Poplawski | [PATCH] pkt_sched: Destroy gen estimators under rtnl_lock(). |
| David Miller | [GIT]: Networking |
