Messages in this thread | | | Date | Tue, 24 Sep 2013 20:58:02 -0400 | From | Tejun Heo <> | Subject | Re: [PATCH] kernel/groups.c: consider about NULL for 'group_info' in all related extern functions |
| |
Hello, Chen.
On Wed, Sep 25, 2013 at 08:49:31AM +0800, Chen Gang wrote: > > Can you please demonstrate such failure? You can tell kernel to > > execute a given binary instead of init with "init=" param. > > OK, I will/should try, today (although I have to spend my additional > time resource on it. :-( ).
Please note that that's not necessarily "additional" resource that you spend. It's more of something necessary to justify the changes you're suggesting. It's true that not all bug fixes / improvements require explicit demonstration but I'm not very convinced about your analysis partly because I'm not too familiar with the code path but also because the code has been stable with years and you seem pretty new to the area.
> Hmm... in fact, in my opinion, interface (especially content system > call) need make itself consistency, although at present, it can not > cause issue.
Sure, it should be consistent but I'm not sure what you're perceiving as inconsistent is actually inconsistent. Anyways, let's please have something deomnstratable. We can think more about the interface niggles afterwards.
Thanks.
-- tejun
| |