Re: [PATCH 0/5] fs, xfs refcount conversions

2017-04-20 Thread Kees Cook
On Thu, Apr 20, 2017 at 3:04 PM, Darrick J. Wong wrote: > On Thu, Apr 20, 2017 at 10:24:04AM -0700, Darrick J. Wong wrote: >> On Thu, Apr 20, 2017 at 08:11:41AM +, Reshetova, Elena wrote: >> > >> > >> > > v3: >> > > * fixed header file inclusion >> > >> > I don't

Re: [PATCH 0/5] fs, xfs refcount conversions

2017-04-20 Thread Kees Cook
On Thu, Apr 20, 2017 at 3:04 PM, Darrick J. Wong wrote: > On Thu, Apr 20, 2017 at 10:24:04AM -0700, Darrick J. Wong wrote: >> On Thu, Apr 20, 2017 at 08:11:41AM +, Reshetova, Elena wrote: >> > >> > >> > > v3: >> > > * fixed header file inclusion >> > >> > I don't think I have heard anything

Re: [PATCH 0/5] fs, xfs refcount conversions

2017-04-20 Thread Darrick J. Wong
On Thu, Apr 20, 2017 at 10:24:04AM -0700, Darrick J. Wong wrote: > On Thu, Apr 20, 2017 at 08:11:41AM +, Reshetova, Elena wrote: > > > > > > > v3: > > > * fixed header file inclusion > > > > I don't think I have heard anything back on this v3 patch set. > > Is there still smth here to

Re: [PATCH 0/5] fs, xfs refcount conversions

2017-04-20 Thread Darrick J. Wong
On Thu, Apr 20, 2017 at 10:24:04AM -0700, Darrick J. Wong wrote: > On Thu, Apr 20, 2017 at 08:11:41AM +, Reshetova, Elena wrote: > > > > > > > v3: > > > * fixed header file inclusion > > > > I don't think I have heard anything back on this v3 patch set. > > Is there still smth here to

Re: [PATCH 0/5] fs, xfs refcount conversions

2017-04-20 Thread Darrick J. Wong
On Thu, Apr 20, 2017 at 08:11:41AM +, Reshetova, Elena wrote: > > > > v3: > > * fixed header file inclusion > > I don't think I have heard anything back on this v3 patch set. > Is there still smth here to fix or could you take the changes in? Generally I think it looks ok; it's running

Re: [PATCH 0/5] fs, xfs refcount conversions

2017-04-20 Thread Darrick J. Wong
On Thu, Apr 20, 2017 at 08:11:41AM +, Reshetova, Elena wrote: > > > > v3: > > * fixed header file inclusion > > I don't think I have heard anything back on this v3 patch set. > Is there still smth here to fix or could you take the changes in? Generally I think it looks ok; it's running

RE: [PATCH 0/5] fs, xfs refcount conversions

2017-04-20 Thread Reshetova, Elena
> v3: > * fixed header file inclusion I don't think I have heard anything back on this v3 patch set. Is there still smth here to fix or could you take the changes in? Best Regards, Elena. > > Now when new refcount_t type and API are finally merged > (see include/linux/refcount.h), the

RE: [PATCH 0/5] fs, xfs refcount conversions

2017-04-20 Thread Reshetova, Elena
> v3: > * fixed header file inclusion I don't think I have heard anything back on this v3 patch set. Is there still smth here to fix or could you take the changes in? Best Regards, Elena. > > Now when new refcount_t type and API are finally merged > (see include/linux/refcount.h), the

[PATCH 0/5] fs, xfs refcount conversions

2017-03-08 Thread Elena Reshetova
v3: * fixed header file inclusion Now when new refcount_t type and API are finally merged (see include/linux/refcount.h), the following patches convert various refcounters in the xfs susystem from atomic_t to refcount_t. By doing this we prevent intentional or accidental underflows or overflows

[PATCH 0/5] fs, xfs refcount conversions

2017-03-08 Thread Elena Reshetova
v3: * fixed header file inclusion Now when new refcount_t type and API are finally merged (see include/linux/refcount.h), the following patches convert various refcounters in the xfs susystem from atomic_t to refcount_t. By doing this we prevent intentional or accidental underflows or overflows