Re: Is it OK to export symbols 'getname' and 'putname'?

2015-04-22 Thread Boqun Feng
On Tue, Apr 21, 2015 at 10:50:15PM -0700, Christoph Hellwig wrote: > On Mon, Apr 20, 2015 at 05:55:07PM +0200, Jan Kara wrote: > > > Is that a good idea to export these symbols, given that lustre may be > > > the only user? > > Yes, it is a good idea. > > It was if lustre was in core code and

Re: Is it OK to export symbols 'getname' and 'putname'?

2015-04-22 Thread Boqun Feng
On Tue, Apr 21, 2015 at 10:50:15PM -0700, Christoph Hellwig wrote: On Mon, Apr 20, 2015 at 05:55:07PM +0200, Jan Kara wrote: Is that a good idea to export these symbols, given that lustre may be the only user? Yes, it is a good idea. It was if lustre was in core code and these

Re: Is it OK to export symbols 'getname' and 'putname'?

2015-04-21 Thread Christoph Hellwig
On Mon, Apr 20, 2015 at 05:55:07PM +0200, Jan Kara wrote: > > Is that a good idea to export these symbols, given that lustre may be > > the only user? > Yes, it is a good idea. It was if lustre was in core code and these idiotic ioctls were something we had to live with. But lustre is in

Re: Is it OK to export symbols 'getname' and 'putname'?

2015-04-21 Thread Boqun Feng
Hi, On Mon, Apr 20, 2015 at 05:55:07PM +0200, Jan Kara wrote: > On Fri 17-04-15 20:35:30, Boqun Feng wrote: > > Hi Al, > > > > > > I'm trying to clean that part of code you mentioned, and I found I have > > to export the symbols 'getname' and 'putname' as follow to replace that > > __getname()

Re: Is it OK to export symbols 'getname' and 'putname'?

2015-04-21 Thread Boqun Feng
Hi, On Mon, Apr 20, 2015 at 05:55:07PM +0200, Jan Kara wrote: On Fri 17-04-15 20:35:30, Boqun Feng wrote: Hi Al, I'm trying to clean that part of code you mentioned, and I found I have to export the symbols 'getname' and 'putname' as follow to replace that __getname() caller:

Re: Is it OK to export symbols 'getname' and 'putname'?

2015-04-21 Thread Christoph Hellwig
On Mon, Apr 20, 2015 at 05:55:07PM +0200, Jan Kara wrote: Is that a good idea to export these symbols, given that lustre may be the only user? Yes, it is a good idea. It was if lustre was in core code and these idiotic ioctls were something we had to live with. But lustre is in staging

Re: Is it OK to export symbols 'getname' and 'putname'?

2015-04-20 Thread Jan Kara
On Fri 17-04-15 20:35:30, Boqun Feng wrote: > Hi Al, > > On Sun, Apr 12, 2015 at 02:13:18AM +0100, Al Viro wrote: > > > > BTW, looking at the __getname() callers... Lustre one sure as hell looks > > bogus: > > char *tmp = __getname(); > > > > if (!tmp) > >

Re: Is it OK to export symbols 'getname' and 'putname'?

2015-04-20 Thread Jan Kara
On Fri 17-04-15 20:35:30, Boqun Feng wrote: Hi Al, On Sun, Apr 12, 2015 at 02:13:18AM +0100, Al Viro wrote: BTW, looking at the __getname() callers... Lustre one sure as hell looks bogus: char *tmp = __getname(); if (!tmp) return

Re: Is it OK to export symbols 'getname' and 'putname'?

2015-04-17 Thread Boqun Feng
On Fri, Apr 17, 2015 at 08:35:30PM +0800, Boqun Feng wrote: > Hi Al, > > On Sun, Apr 12, 2015 at 02:13:18AM +0100, Al Viro wrote: > > > > BTW, looking at the __getname() callers... Lustre one sure as hell looks > > bogus: > > char *tmp = __getname(); > > > > if (!tmp) > >

Is it OK to export symbols 'getname' and 'putname'?

2015-04-17 Thread Boqun Feng
Hi Al, On Sun, Apr 12, 2015 at 02:13:18AM +0100, Al Viro wrote: > > BTW, looking at the __getname() callers... Lustre one sure as hell looks > bogus: > char *tmp = __getname(); > > if (!tmp) > return ERR_PTR(-ENOMEM); > > len = strncpy_from_user(tmp,

Is it OK to export symbols 'getname' and 'putname'?

2015-04-17 Thread Boqun Feng
Hi Al, On Sun, Apr 12, 2015 at 02:13:18AM +0100, Al Viro wrote: BTW, looking at the __getname() callers... Lustre one sure as hell looks bogus: char *tmp = __getname(); if (!tmp) return ERR_PTR(-ENOMEM); len = strncpy_from_user(tmp, filename,

Re: Is it OK to export symbols 'getname' and 'putname'?

2015-04-17 Thread Boqun Feng
On Fri, Apr 17, 2015 at 08:35:30PM +0800, Boqun Feng wrote: Hi Al, On Sun, Apr 12, 2015 at 02:13:18AM +0100, Al Viro wrote: BTW, looking at the __getname() callers... Lustre one sure as hell looks bogus: char *tmp = __getname(); if (!tmp) return