Re: [PATCH] staging/lustre: Replace jobid acquiring with per node setting

[Greg Kroah-Hartman <gregkh@linuxfoundation.org>]

On Sat, May 03, 2014 at 09:20:06PM -0400, Oleg Drokin wrote:
> 
> On May 3, 2014, at 7:29 PM, Greg Kroah-Hartman wrote:
> 
> > On Sun, Apr 27, 2014 at 10:21:52PM -0400, Oleg Drokin wrote:
> >> Insted of meddling directly in process environment variables
> >> (which is also not possible on certain platforms due to not exported
> >> symbols), create jobid_name proc file to represent this info
> >> (to be filled by job scheduler epilogue).
> > That looks better, but what are you going to do about getting rid of all
> > of these proc files?  Just move them all at once?  I really don't like
> > the idea of adding new ones, and want a plan for what you all are going
> > to do here moving forward.
> 
> What proc files are to referring to, if I may ask?

All of them :)

> I don't think I saw complaints about proc files, the complaints I saw were mostly about
> reading env variables directly and the like so that was the focus of this patch.
> Did I miss some side discussion? Any pointers?

No, no side discussion, the proc files need to be removed / fixed before
the code can be merged to the "proper" part of the kernel tree.

thanks,

greg k-h