Re: [PATCH 4/7] ovl: add infrastructure for intercepting file ops

2016-11-24 Thread Amir Goldstein
On Thu, Nov 24, 2016 at 4:08 PM, Amir Goldstein wrote: > On Thu, Nov 24, 2016 at 3:51 PM, Miklos Szeredi wrote: >> On Thu, Nov 24, 2016 at 2:12 PM, Amir Goldstein wrote: >>> On Thu, Nov 24, 2016 at 2:03 PM, Miklos Szeredi

Re: [PATCH 4/7] ovl: add infrastructure for intercepting file ops

2016-11-24 Thread Amir Goldstein
On Thu, Nov 24, 2016 at 4:08 PM, Amir Goldstein wrote: > On Thu, Nov 24, 2016 at 3:51 PM, Miklos Szeredi wrote: >> On Thu, Nov 24, 2016 at 2:12 PM, Amir Goldstein wrote: >>> On Thu, Nov 24, 2016 at 2:03 PM, Miklos Szeredi wrote: On Thu, Nov 24, 2016 at 12:52 PM, Amir Goldstein

Re: [PATCH 4/7] ovl: add infrastructure for intercepting file ops

2016-11-24 Thread Amir Goldstein
On Thu, Nov 24, 2016 at 3:51 PM, Miklos Szeredi wrote: > On Thu, Nov 24, 2016 at 2:12 PM, Amir Goldstein wrote: >> On Thu, Nov 24, 2016 at 2:03 PM, Miklos Szeredi wrote: >>> On Thu, Nov 24, 2016 at 12:52 PM, Amir Goldstein

Re: [PATCH 4/7] ovl: add infrastructure for intercepting file ops

2016-11-24 Thread Amir Goldstein
On Thu, Nov 24, 2016 at 3:51 PM, Miklos Szeredi wrote: > On Thu, Nov 24, 2016 at 2:12 PM, Amir Goldstein wrote: >> On Thu, Nov 24, 2016 at 2:03 PM, Miklos Szeredi wrote: >>> On Thu, Nov 24, 2016 at 12:52 PM, Amir Goldstein wrote: On Thu, Nov 24, 2016 at 12:55 PM, Miklos Szeredi

Re: [PATCH 4/7] ovl: add infrastructure for intercepting file ops

2016-11-24 Thread Miklos Szeredi
On Thu, Nov 24, 2016 at 2:12 PM, Amir Goldstein wrote: > On Thu, Nov 24, 2016 at 2:03 PM, Miklos Szeredi wrote: >> On Thu, Nov 24, 2016 at 12:52 PM, Amir Goldstein wrote: >>> On Thu, Nov 24, 2016 at 12:55 PM, Miklos Szeredi

Re: [PATCH 4/7] ovl: add infrastructure for intercepting file ops

2016-11-24 Thread Miklos Szeredi
On Thu, Nov 24, 2016 at 2:12 PM, Amir Goldstein wrote: > On Thu, Nov 24, 2016 at 2:03 PM, Miklos Szeredi wrote: >> On Thu, Nov 24, 2016 at 12:52 PM, Amir Goldstein wrote: >>> On Thu, Nov 24, 2016 at 12:55 PM, Miklos Szeredi >>> wrote: >> + /* +* These

Re: [PATCH 4/7] ovl: add infrastructure for intercepting file ops

2016-11-24 Thread Amir Goldstein
On Thu, Nov 24, 2016 at 2:03 PM, Miklos Szeredi wrote: > On Thu, Nov 24, 2016 at 12:52 PM, Amir Goldstein wrote: >> On Thu, Nov 24, 2016 at 12:55 PM, Miklos Szeredi wrote: > >>> + /* >>> +* These should be

Re: [PATCH 4/7] ovl: add infrastructure for intercepting file ops

2016-11-24 Thread Amir Goldstein
On Thu, Nov 24, 2016 at 2:03 PM, Miklos Szeredi wrote: > On Thu, Nov 24, 2016 at 12:52 PM, Amir Goldstein wrote: >> On Thu, Nov 24, 2016 at 12:55 PM, Miklos Szeredi wrote: > >>> + /* >>> +* These should be intercepted, but they are very unlikely >>> to be >>> +

Re: [PATCH 4/7] ovl: add infrastructure for intercepting file ops

2016-11-24 Thread Miklos Szeredi
On Thu, Nov 24, 2016 at 12:52 PM, Amir Goldstein wrote: > On Thu, Nov 24, 2016 at 12:55 PM, Miklos Szeredi wrote: >> + /* >> +* These should be intercepted, but they are very unlikely >> to be >> +* a

Re: [PATCH 4/7] ovl: add infrastructure for intercepting file ops

2016-11-24 Thread Miklos Szeredi
On Thu, Nov 24, 2016 at 12:52 PM, Amir Goldstein wrote: > On Thu, Nov 24, 2016 at 12:55 PM, Miklos Szeredi wrote: >> + /* >> +* These should be intercepted, but they are very unlikely >> to be >> +* a problem in practice. Leave them alone for now.

Re: [PATCH 4/7] ovl: add infrastructure for intercepting file ops

2016-11-24 Thread Amir Goldstein
On Thu, Nov 24, 2016 at 12:55 PM, Miklos Szeredi wrote: > Overlayfs needs to intercept a few file operations in order to properly > handle the following corner case: > > - file X is in overlayfs; > > - X resides on a lower (read-only) layer > the lower file is L; >

Re: [PATCH 4/7] ovl: add infrastructure for intercepting file ops

2016-11-24 Thread Amir Goldstein
On Thu, Nov 24, 2016 at 12:55 PM, Miklos Szeredi wrote: > Overlayfs needs to intercept a few file operations in order to properly > handle the following corner case: > > - file X is in overlayfs; > > - X resides on a lower (read-only) layer > the lower file is L; > > - X is opened with