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

[Greg Kroah-Hartman]

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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/