linux-kernel - Re: [RFC] audit: avoid soft lockup in audit_log

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20130828185436.279bc6dd@redhat.com>
Date:	Wed, 28 Aug 2013 18:54:36 -0400
From:	Luiz Capitulino <lcapitulino@...hat.com>
To:	Andrew Morton <akpm@...ux-foundation.org>
Cc:	linux-kernel@...r.kernel.org, oleg@...hat.com, eparis@...hat.com,
	rgb@...hat.com
Subject: Re: [RFC] audit: avoid soft lockup in audit_log_start()

On Wed, 28 Aug 2013 15:33:45 -0700
Andrew Morton <akpm@...ux-foundation.org> wrote:

> On Wed, 28 Aug 2013 18:21:14 -0400 Luiz Capitulino <lcapitulino@...hat.com> wrote:
> 
> > I'm getting the following soft lockup:
> > 
> > CPU: 6 PID: 2278 Comm: killall5 Tainted: GF            3.11.0-rc7+ #1
> > Hardware name: Bochs Bochs, BIOS Bochs 01/01/2011
> >  0000000000000099 ffff88011fd83de8 ffffffff815324df 0000000000002800
> >  ffffffff817d48f9 ffff88011fd83e68 ffffffff8152e669 ffff88011fd83e68
> >  ffffffff00000008 ffff88011fd83e78 ffff88011fd83e18 0000004081dac040
> > Call Trace:
> >  <IRQ>  [<ffffffff815324df>] dump_stack+0x46/0x58
> >  [<ffffffff8152e669>] panic+0xbb/0x1c4
> >  [<ffffffff810d03c3>] watchdog_timer_fn+0x163/0x170
> >  [<ffffffff8106c691>] __run_hrtimer+0x81/0x1c0
> >  [<ffffffff810d0260>] ? watchdog+0x30/0x30
> >  [<ffffffff8106cea7>] hrtimer_interrupt+0x107/0x240
> >  [<ffffffff8102f61b>] local_apic_timer_interrupt+0x3b/0x60
> >  [<ffffffff81542465>] smp_apic_timer_interrupt+0x45/0x60
> >  [<ffffffff8154124a>] apic_timer_interrupt+0x6a/0x70
> >  <EOI>  [<ffffffff810c2f5f>] ? audit_log_start+0xbf/0x430
> >  [<ffffffff810c2fe7>] ? audit_log_start+0x147/0x430
> >  [<ffffffff81079030>] ? try_to_wake_up+0x2a0/0x2a0
> >  [<ffffffff810c86be>] audit_log_exit+0x6ae/0xc30
> >  [<ffffffff81188662>] ? __alloc_fd+0x42/0x100
> >  [<ffffffff810c98e7>] __audit_syscall_exit+0x257/0x2b0
> >  [<ffffffff81540794>] sysret_audit+0x17/0x21
> > 
> > The reproducer is somewhat unusual:
> > 
> >  1. Install RHEL6.5 (maybe a similar older user-space will do)
> >  2. Boot the just installed system
> >  3. In this first boot you'll meet the firstboot script, which
> >     will do some setup and (depending on your answers) it will
> >     reboot the machine
> >  4. During that first reboot the system hangs while terminating
> >     all processes:
> > 
> >        Sending all processes the TERM signal...
> > 
> >     It's when the soft lockup above happens. And yes, I managed
> >     to get this with latest upstream kernel (HEAD fa8218def1b1)
> > 
> > I'm reproducing it on a VM, but the first report was on bare-metal.
> > 
> > This is what is happening:
> > 
> >  1. audit_log_start() is called
> >  2. As we have SKBs waiting in audit_skb_queue and all conditions
> >     evaluate to true, we sleep in wait_for_auditd()
> >  3. Go to 2, until sleep_time gets negative and audit_log_start()
> >     just busy-waits
> > 
> > Now, *why* this is happening is a mistery to me. I tried debugging
> > it, but all I could find is that at some point the kauditd thread
> > never wakes up after having called schedule(). I even tried waking
> > it up before calling wait_for_auditd(), but it didn't.
> 
> Odd.  If kauditd_thread() is stuck in schedule() in state
> TASK_INTERRUPTIBLE then a wake_up(kauditd_task) should get it running
> again, unless scheduler-related data structures are screwed up.
> 
> Are you really sure that kauditd is stuck in schedule() and doesn't
> come out?

No, that's a guess. Inferred from:

1. I tried calling wake_up_interruptible(&kauditd_wait); right
   before wait_for_auditd(). Nothing changes

2. I added this debug printks:

  diff --git a/kernel/audit.c b/kernel/audit.c
  index 91e53d0..27448ad 100644
  --- a/kernel/audit.c
  +++ b/kernel/audit.c
  @@ -458,11 +458,14 @@ static int kauditd_thread(void *dummy)
   		set_current_state(TASK_INTERRUPTIBLE);
   		add_wait_queue(&kauditd_wait, &wait);
   
  +		pr_emerg_ratelimited("*** sleeping\n");
  +
   		if (!skb_queue_len(&audit_skb_queue)) {
   			try_to_freeze();
   			schedule();
   		}
   
  +		pr_emerg_ratelimited("*** waking up\n");
   		__set_current_state(TASK_RUNNING);
   		remove_wait_queue(&kauditd_wait, &wait);
   	}

 I get several pairs of sleeping/waking up strings right before the
 system begins to shut down. Then it stops (even though we do
 have SKBs queued)

Now, the kauditd thread does exists when the soft lockup happens,
I could verify that with crash.

What's interesting is that I can only get it in that first reboot
done by the firstboot script in RHEL6. I didn't manage to get it
in any other way.

I'm open to any debugging tips, but be gentle because I'm not
very familiar with that stuff.
--
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/