MIME-Version: 1.0
In-Reply-To: <SN1PR12MB03524B310A5AD6FE3E67E3DEC80F0@SN1PR12MB0352.namprd12.prod.outlook.com>
References: <20171213232122.5ztmwume5kylh4mz@dtor-ws> <SN1PR12MB03526E031CF654E9AB85310AC80A0@SN1PR12MB0352.namprd12.prod.outlook.com>
 <CAKdAkRTufgwzfuoxnGFTDWOt+b3APy93_VEP41r+=JHa24AjiQ@mail.gmail.com>
 <SN1PR12MB0352579973FE602FE2C741D6C80A0@SN1PR12MB0352.namprd12.prod.outlook.com>
 <CAKdAkRTUEJqDk6Dp13Y5i1fU_ojAT8e7_DWVSqDh7uahYPHo+Q@mail.gmail.com>
 <SN1PR12MB0352729FF070C3384E5C0261C80A0@SN1PR12MB0352.namprd12.prod.outlook.com>
 <SN1PR12MB0352EE3210E82569520D8BCAC80B0@SN1PR12MB0352.namprd12.prod.outlook.com>
 <CAKdAkRRMFVBGvXJnU0EsQV3j4WRUjPKK6vdG=w5Ft3ruHaObBw@mail.gmail.com>
 <SN1PR12MB035218DF41A1F08791A3E89EC80B0@SN1PR12MB0352.namprd12.prod.outlook.com>
 <SN1PR12MB035283C93DA7894CD4F46AA6C80E0@SN1PR12MB0352.namprd12.prod.outlook.com>
 <SN1PR12MB0352162B7267B09489988E34C80E0@SN1PR12MB0352.namprd12.prod.outlook.com>
 <SN1PR12MB03524B310A5AD6FE3E67E3DEC80F0@SN1PR12MB0352.namprd12.prod.outlook.com>
From: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Date: Wed, 20 Dec 2017 23:13:33 -0600
Message-ID: <CAKdAkRRTHVL0iKOLeUjh8+q=Ych4Xjuj5CB4eEGu-ZKeBJthCQ@mail.gmail.com>
Subject: Re: [PATCH] kobject: fix suppressing modalias in uevents delivered
 over netlink
To: Casey Leedom <leedom@chelsio.com>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Eric Dumazet <eric.dumazet@gmail.com>,
        Eric Dumazet <edumazet@google.com>, Tariq Toukan <tariqt@mellanox.com>,
        "David S . Miller" <davem@davemloft.net>,
        Komali Katari <komali@chelsio.com>,
        "Luis R. Rodriguez" <mcgrof@kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1503
Lines: 30

On Tue, Dec 19, 2017 at 3:20 PM, Casey Leedom <leedom@chelsio.com> wrote:
>   By the way, Komali went back and tried 4.14.7 and it does exhibit the
> issue which she originally reported.  As noted before, the origin of the new
> behavior was tracked down to kernel.org:1455cf8 ...  Also as noted, Dmitry's
> patch does not solve the problem.

OK, so there is 2 separate issues. First is that the "zapping" of
modaliases was broken by Eric's changes and we do need the patch above
to fix it so we do not end up with modules being essentially
not-unloadable on systems with old udev rules, as since trying to
unload module would result in unbinding from a device, and udev would
see unbind uevent and immediately reload the module in question. Greg,
can you please schedule my patch in for 4.15?

The 2nd issue is that Casey is seeing cxgb4vf driver being loaded.
This happens because vfio-pci driver binds to the device, which causes
KOBJ_BIND to get emitted, which causes all modules that match the PCI
device ID that was bound to load [again].

While we could fix the issue by putting another bandaid and zapping
the modalias on KOBJ_BIND as well, I am not sure we need to do that,
as I do not see a harm in having cxgb4vf module loaded. Rather than
patching the kernel I'd recommend Casey simply update the udev scripts
to only load module on KOBJ_ADD event:
https://github.com/systemd/systemd/commit/9b32afa9f241fe8febc0a754850f1e7331caf6e3#diff-0e123aabb4420b8b95088699179f3416

Thanks.

-- 
Dmitry