On 02/08/2019 03:04 PM, Paul Moore wrote:
On Fri, Feb 8, 2019 at 2:58 PM Don Dutile <ddutile@xxxxxxxxxx> wrote:
On 02/01/2019 11:09 AM, Paul Moore wrote:
On Fri, Feb 1, 2019 at 9:16 AM Daniel Jurgens <danielj@xxxxxxxxxxxx> wrote:
On 2/1/2019 7:57 AM, Paul Moore wrote:
On Tue, Jan 29, 2019 at 4:13 PM Paul Moore <paul@xxxxxxxxxxxxxx> wrote:
On Tue, Jan 29, 2019 at 3:58 PM Daniel Jurgens <danielj@xxxxxxxxxxxx> wrote:
On 1/29/2019 2:51 PM, Paul Moore wrote:
Okay, so let's attempt the change above where we just do the access
check directly. Although I'm a little concerned that without a
reproducer we might not end up fixing the problem we're trying to fix.
Is anyone in touch with the person who originally reported the
problem? It would be great if we could get that person to verify the
change ...
I decided to go with maintaining a list in IB core. The notifier call is done under rcu_read_lock vs spin_lock_irq for register/unregister, so we shouldn't have any problems in that case. So only registering once basically achieves the same thing as taking it all out. I'm testing it now. I'll send it for internal review today assuming it checks out. Hopefully Leon can get it posted tomorrow, I know Don has some schedule pressure here.
Okay sounds good. We're still at -rc4 so as long as we can get
something posted this week, or early next, I see no reason why it
can't make the upcoming merge window.
I'm guessing Don's schedule pressure is more a RH deadline, and not an
upstream constraint.
I just wanted to check in and see how this was progressing? I didn't
see anything in my inbox, but perhaps I missed it ...
It's passed internal review. Leon should send it soon (Sunday at the soonest, IL has Friday-Saturday weekends), he may be waiting for a regression run to finish.
Great, thanks for the update.
update? we're approaching the 2nd Sunday after 2/1/2019, when this email was sent ...
Leon sent out an updated patchset on February 2nd, and I sent a
reply/ack for the SELinux relevant patch a few days later on the 6th.
Archive link below, but it looks like you were CC'd ... ?
* https://lore.kernel.org/selinux/20190202090945.4106-1-leon@xxxxxxxxxx
Thanks for ptr. I was looking for a posting from Daniel; forgot Leon was doing it.
I should have search for email from you! ;-)
Finally found it... lost in a several-thousand email stampede here. :-p
I thought I had seen something... and I viewed it while dealing with a massive head cold...
lousy meds! :-p
I guess the maintainers haven't pulled it into an rc branch yet, though,
since a pull/merge doesn't show it up in (rdma-linux's) for-rc or for-next
(or any other jgg or dl-for-* branch I dug around in).
/me needs c-p-able commit-id(s), so it's the '2nd Sunday' for me...
