> @@ -325,20 +296,12 @@ iser_create_fastreg_desc(struct iser_device *device, > if (!desc) > return ERR_PTR(-ENOMEM); > > - ret = iser_alloc_reg_res(device, pd, &desc->rsc, size); > + ret = iser_alloc_reg_res(device, pd, &desc->rsc, size, pi_enable); > if (ret) > goto reg_res_alloc_failure; > > - if (pi_enable) { > - ret = iser_alloc_pi_ctx(device, pd, desc, size); > - if (ret) > - goto pi_ctx_alloc_failure; > - } > - Is there any reason to keep iser_create_fastreg_desc and iser_alloc_reg_res separate after this?