From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paul Moore Subject: Re: [RFC PATCH 4/7] audit: wake up threads if queue switched from limited to unlimited Date: Thu, 05 Nov 2015 19:05:05 -0500 Message-ID: <17437322.gt1f2pcy05@sifl> References: <8a0bd940090b5c4cc19869c6126bdcc85e3be940.1445539473.git.rgb@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Return-path: In-Reply-To: <8a0bd940090b5c4cc19869c6126bdcc85e3be940.1445539473.git.rgb@redhat.com> Sender: linux-kernel-owner@vger.kernel.org To: Richard Guy Briggs Cc: linux-audit@redhat.com, linux-kernel@vger.kernel.org, v.rathor@gmail.com List-Id: linux-audit@redhat.com On Thursday, October 22, 2015 02:53:17 PM Richard Guy Briggs wrote: > If the audit_backlog_limit is changed from a limited value to an > unlimited value (zero) while the queue was overflowed, wake up the > audit_backlog_wait queue to allow those processes to continue. > > Signed-off-by: Richard Guy Briggs > --- > kernel/audit.c | 3 ++- > 1 files changed, 2 insertions(+), 1 deletions(-) Looks like the right thing to do, merged to audit#next-queue. > diff --git a/kernel/audit.c b/kernel/audit.c > index 384a1a1..02a5ec0 100644 > --- a/kernel/audit.c > +++ b/kernel/audit.c > @@ -523,7 +523,8 @@ static int kauditd_thread(void *dummy) > skb = skb_dequeue(&audit_skb_queue); > > if (skb) { > - if (skb_queue_len(&audit_skb_queue) <= audit_backlog_limit) > + if (!audit_backlog_limit || > + (skb_queue_len(&audit_skb_queue) <= audit_backlog_limit)) > wake_up(&audit_backlog_wait); > if (audit_pid) > kauditd_send_skb(skb); -- paul moore www.paul-moore.com