Re: [PATCH v13 12/51] vfs: Cache richacl in struct inode

From: Andreas Gruenbacher
Date: Wed Nov 04 2015 - 17:10:53 EST


Andreas,

On Wed, Nov 4, 2015 at 3:03 AM, Andreas Dilger <adilger@xxxxxxxxx> wrote:
>> @@ -33,7 +33,7 @@ richacl_alloc(int count, gfp_t gfp)
>> struct richacl *acl = kzalloc(size, gfp);
>>
>> if (acl) {
>> - atomic_set(&acl->a_refcount, 1);
>> + atomic_set(&acl->a_base.ba_refcount, 1);
>> acl->a_count = count;
>> }
>> return acl;
>> @@ -52,7 +52,7 @@ richacl_clone(const struct richacl *acl, gfp_t gfp)
>>
>> if (dup) {
>> memcpy(dup, acl, size);
>> - atomic_set(&dup->a_refcount, 1);
>> + atomic_set(&dup->a_base.ba_refcount, 1);
>
> These two calls should be base_acl_init().

Yes. This should all be fixed in the next snapshot.

Thanks,
Andreas
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/