aboutsummaryrefslogtreecommitdiffstats
path: root/lib/of-reconfig-notifier-error-inject.c
diff options
context:
space:
mode:
authorJason Gunthorpe <jgg@mellanox.com>2018-07-10 20:55:21 -0600
committerJason Gunthorpe <jgg@mellanox.com>2018-07-25 14:21:22 -0600
commitaba94548c9e49939fafc92bb406a7f8e7ed87643 (patch)
tree3d68355c7076520926746c98df13f8a4d69e58f6 /lib/of-reconfig-notifier-error-inject.c
parentIB/uverbs: Always propagate errors from rdma_alloc_commit_uobject() (diff)
downloadlinux-dev-aba94548c9e49939fafc92bb406a7f8e7ed87643.tar.xz
linux-dev-aba94548c9e49939fafc92bb406a7f8e7ed87643.zip
IB/uverbs: Move the FD uobj type struct file allocation to alloc_commit
Allocating the struct file during alloc_begin creates this strange asymmetry with IDR, where the FD has two krefs pointing at it during the pre-commit phase. In particular this makes the abort process for FD very strange and confusing. For instance abort currently calls the type's destroy_object twice, and the fops release once if abort is done. This is very counter intuitive. No fops should be called until alloc_commit succeeds, and destroy_object should only ever be called once. Moving the struct file allocation to the alloc_commit is now simple, as we already support failure of rdma_alloc_commit_uobject, with all the required rollback pieces. This creates an understandable symmetry with IDR and simplifies/fixes the abort handling for FD types. Signed-off-by: Jason Gunthorpe <jgg@mellanox.com>
Diffstat (limited to '')
0 files changed, 0 insertions, 0 deletions