Messages in this thread | | | Date | Wed, 30 Oct 2013 06:21:01 -0700 | From | Greg Kroah-Hartman <> | Subject | Re: [PATCH 2/4] staging/lustre/obdclass: read jobid from proc |
| |
On Wed, Oct 30, 2013 at 07:30:34PM +0800, Peng Tao wrote: > so that we can get rid of cfs_get_environ() that needs access_process_vm() that > is a core mm function and is not available on some architectures. > > Reviewed-by: Niu Yawei <yawei.niu@intel.com> > Signed-off-by: Peng Tao <bergwolf@gmail.com> > Signed-off-by: Andreas Dilger <andreas.dilger@intel.com> > --- > drivers/staging/lustre/lustre/obdclass/class_obd.c | 119 +++++++++++++++++++- > 1 file changed, 115 insertions(+), 4 deletions(-) > > diff --git a/drivers/staging/lustre/lustre/obdclass/class_obd.c b/drivers/staging/lustre/lustre/obdclass/class_obd.c > index b1024a6..99ce863 100644 > --- a/drivers/staging/lustre/lustre/obdclass/class_obd.c > +++ b/drivers/staging/lustre/lustre/obdclass/class_obd.c > @@ -96,8 +96,117 @@ EXPORT_SYMBOL(obd_dirty_transit_pages); > char obd_jobid_var[JOBSTATS_JOBID_VAR_MAX_LEN + 1] = JOBSTATS_DISABLE; > EXPORT_SYMBOL(obd_jobid_var); > > -/* Get jobid of current process by reading the environment variable > - * stored in between the "env_start" & "env_end" of task struct. > +static char *self_environ_file = "/proc/self/environ";
Heh, no, that's not ok at all.
This is a _huge_ sign that you are doing something wrong in your driver if you need something that isn't exported, or that you have to dig out of proc.
Sorry, I can't take this, please fix the underlying problems that would even think that you need access to the environment from within a kernel driver.
greg k-h
| |