Martin, I am self nacking this patch for now. On 02/01/2019 09:01 PM, Mike Christie wrote: > genlmsg_multicast_allns used to return -ESRCH even if the message was > successfully sent to a listener. With commit: > > commit cb9f7a9a5c96a773bbc9c70660dc600cfff82f82 > Author: Nicolas Dichtel <nicolas.dichtel@xxxxxxxxx> > Date: Tue Feb 6 14:48:32 2018 +0100 > > netlink: ensure to loop over all netns in genlmsg_multicast_allns() > > it now will return success if the message was sent to a listener. With > that patch, tcmu can now immediately fail if -ESRCH is returned because > we know there will be no reply. > > Signed-off-by: Mike Christie <mchristi@xxxxxxxxxx> > > --- > drivers/target/target_core_user.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/drivers/target/target_core_user.c b/drivers/target/target_core_user.c > index 5831e0e..dccc13c 100644 > --- a/drivers/target/target_core_user.c > +++ b/drivers/target/target_core_user.c > @@ -1794,9 +1794,6 @@ static int tcmu_netlink_event_send(struct tcmu_dev *udev, > > ret = genlmsg_multicast_allns(&tcmu_genl_family, skb, 0, > TCMU_MCGRP_CONFIG, GFP_KERNEL); > - /* We don't care if no one is listening */ > - if (ret == -ESRCH) > - ret = 0; > if (!ret) > ret = tcmu_wait_genl_cmd_reply(udev); > return ret; >
