[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20141220035148.GA512@madcap2.tricolour.ca>
Date: Fri, 19 Dec 2014 22:51:48 -0500
From: Richard Guy Briggs <rgb@...hat.com>
To: Valdis.Kletnieks@...edu
Cc: linux-audit@...hat.com, linux-kernel@...r.kernel.org,
selinux@...ho.nsa.gov, sgrubb@...hat.com, eparis@...isplace.org,
pmoore@...hat.com
Subject: Re: [PATCH] audit: use supplied gfp_mask from audit_buffer in
kauditd_send_multicast_skb
On 14/12/19, Valdis.Kletnieks@...edu wrote:
> On Thu, 18 Dec 2014 23:09:27 -0500, Richard Guy Briggs said:
> > Eric Paris explains: Since kauditd_send_multicast_skb() gets called in
> > audit_log_end(), which can come from any context (aka even a sleeping context)
> > GFP_KERNEL can't be used. Since the audit_buffer knows what context it should
> > use, pass that down and use that.
> >
> > See: https://lkml.org/lkml/2014/12/16/542
>
> > Reported-by: Valdis Kletnieks <Valdis.Kletnieks@...edu>
> > Signed-off-by: Richard Guy Briggs <rgb@...hat.com>
> > ---
> > kernel/audit.c | 8 ++++----
> > 1 files changed, 4 insertions(+), 4 deletions(-)
> >
> > diff --git a/kernel/audit.c b/kernel/audit.c
> > index 7b83c55..ce484fb 100644
>
> I was reliably triggering 3-4 BUGs an hour, and with this patch applied I've
> gone 9 hours without seeing one.
>
> So feel free to add a Tested-By: when it goes out.
Good stuff. Thanks for testing this Valdis!
- RGB
--
Richard Guy Briggs <rbriggs@...hat.com>
Senior Software Engineer, Kernel Security, AMER ENG Base Operating Systems, Red Hat
Remote, Ottawa, Canada
Voice: +1.647.777.2635, Internal: (81) 32635, Alt: +1.613.693.0684x3545
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists