Re: [PATCH 1/2] vfs: export symbol 'getname' and 'putname'

2015-04-23 Thread Drokin, Oleg
On Apr 22, 2015, at 3:34 AM, Christoph Hellwig wrote: > On Wed, Apr 22, 2015 at 06:49:08AM +, Drokin, Oleg wrote: >> I know this is not going to be a popular opinion with you, but sometimes >> opening a file >> is just too expensive. 1 RPC roudntrip to open a file and then another one >> to

Re: [PATCH 1/2] vfs: export symbol 'getname' and 'putname'

2015-04-22 Thread Christoph Hellwig
On Wed, Apr 22, 2015 at 06:49:08AM +, Drokin, Oleg wrote: > I know this is not going to be a popular opinion with you, but sometimes > opening a file > is just too expensive. 1 RPC roudntrip to open a file and then another one to > close it. Use O_PATH to avoid this. -- To unsubscribe from t

Re: [PATCH 1/2] vfs: export symbol 'getname' and 'putname'

2015-04-21 Thread Drokin, Oleg
On Apr 22, 2015, at 2:31 AM, Christoph Hellwig wrote: > On Wed, Apr 22, 2015 at 06:27:11AM +, Drokin, Oleg wrote: >>> Nak on exporting symbols for broken staging code. Please get rid of >>> the ioctls looking up path names in horrible ways in the lustre code. >> >> For a reference, is there

Re: [PATCH 1/2] vfs: export symbol 'getname' and 'putname'

2015-04-21 Thread Drokin, Oleg
On Apr 22, 2015, at 2:31 AM, Greg Kroah-Hartman wrote: > On Tue, Apr 21, 2015 at 10:53:11PM -0700, Christoph Hellwig wrote: >> Nak on exporting symbols for broken staging code. Please get rid of >> the ioctls looking up path names in horrible ways in the lustre code. > > I agree with Christoph,

Re: [PATCH 1/2] vfs: export symbol 'getname' and 'putname'

2015-04-21 Thread Greg Kroah-Hartman
On Tue, Apr 21, 2015 at 10:53:11PM -0700, Christoph Hellwig wrote: > Nak on exporting symbols for broken staging code. Please get rid of > the ioctls looking up path names in horrible ways in the lustre code. I agree with Christoph, we shouldn't be doing this. Let's fix lustre "properly", which

Re: [PATCH 1/2] vfs: export symbol 'getname' and 'putname'

2015-04-21 Thread Christoph Hellwig
On Wed, Apr 22, 2015 at 06:27:11AM +, Drokin, Oleg wrote: > > Nak on exporting symbols for broken staging code. Please get rid of > > the ioctls looking up path names in horrible ways in the lustre code. > > For a reference, is there a good example of a non-horrible way to look up a > pathna

Re: [PATCH 1/2] vfs: export symbol 'getname' and 'putname'

2015-04-21 Thread Drokin, Oleg
On Apr 22, 2015, at 1:53 AM, Christoph Hellwig wrote: > Nak on exporting symbols for broken staging code. Please get rid of > the ioctls looking up path names in horrible ways in the lustre code. For a reference, is there a good example of a non-horrible way to look up a pathname? Thanks. Bye

Re: [PATCH 1/2] vfs: export symbol 'getname' and 'putname'

2015-04-21 Thread Christoph Hellwig
Nak on exporting symbols for broken staging code. Please get rid of the ioctls looking up path names in horrible ways in the lustre code. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http:/