Skip to contentSkip to conversation

BUG: tty: memory corruption through tty_release/tty_ldisc_release

PATCH In review Alexander Holler 4 participants 16 messages May 16, 2013 – Jun 26, 2013

Subsystems (3)
lore
Share & tools
lore
#0 lore raw

Hello,

after some pain because the "big step" (ecbbfd4) happened while the support for my AMD CPU was broken and thus git bisect hit a series of kernels which didn't boot, I've finally found the cause for a memory corruption: tty_ldisc_release().

What happens is the following:

tty_port is self-destructing, that means it destroys itself in tty_port.c:tty_port_destructor() when the last reference is gone. E.g. in case of rfcomm this happens with the call to tty->ops->close() in tty_io.c:tty_release().

The problem here is that tty_io.c:tty_release() calls tty_ldisc.c:tty_ldisc_release() which uses the tty_port to flush the ldisc work queues.

In the best case this hits a BUG() in cancel_work_sync() but often it just causes a memory corruption without a BUG() got hit before.

My quick fix (the diff below) is to remove the call to tty_ldisc_release() but I'm very doubtful that this is the correct approach, as I'm not very familiar with the tty subsystem and therefor I'm not sure if it is necessary to flush the ldisc queues. Another solution could be to move the call to tty_ldisc_release() to tty_port.c:tty_port_destructor().

If the patch finds friends, it could be enhanced to remove tty_ldisc.c:tty_ldisc_release() too, tty_release() is the only user of that function.

I've added the bluetooth people to cc, because rfcomm is one user of tty_release() which gets hit by this bug whenever a connected (rfcomm) remote device disappears.

Regards,

Alexander Holler

(The format of the patch below is likely broken, but I don't care, as it is just a RFC.)

From 4a1d24e5334668f6cd82937d574cd75b5e372a75 Mon Sep 17 00:00:00 2001
From: Alexander Holler <[email protected]>
Date: Thu, 16 May 2013 08:21:22 +0200
Subject: [PATCH] tty: don't call tty_ldisc_release() because tty_port likely
disappeared before

tty_port destroys itself when the last reference is gone, therefor tty_ldisc_release() can't be called afterwards.

---
 drivers/tty/tty_io.c | 4 ----
 1 file changed, 4 deletions(-)
drivers/tty/tty_io.c +0 -4
index 6464029..9c9ad04 100644
@@ -1850,10 +1850,6 @@ int tty_release(struct inode *inode, struct file *filp)
 #ifdef TTY_DEBUG_HANGUP
 	printk(KERN_DEBUG "%s: %s: final close\n", __func__, tty_name(tty, buf));
 #endif
-	/*
-	 * Ask the line discipline code to release its structures
-	 */
-	tty_ldisc_release(tty, o_tty);
 
 	/* Wait for pending work before tty destruction commmences */
 	tty_flush_works(tty);
Signature · 1 line
1.8.1.4
#1 lore raw

Am 16.05.2013 08:45, schrieb Alexander Holler:

Quoted 8 lines

tty_port is self-destructing, that means it destroys itself in tty_port.c:tty_port_destructor() when the last reference is gone. E.g. in case of rfcomm this happens with the call to tty->ops->close() in tty_io.c:tty_release().

The problem here is that tty_io.c:tty_release() calls tty_ldisc.c:tty_ldisc_release() which uses the tty_port to flush the ldisc work queues.

As I've got a bit nervous if ptys are affected, I've played a bit around using the following patch:

--------------

drivers/tty/tty_io.c +1 -0
index 84ec4ca..340440b 100644
@@ -1790,6 +1790,7 @@ int tty_release(struct inode *inode, struct file *filp)
        if (!tty_closing || (o_tty && !o_tty_closing))
                return 0;
 
+pr_info("AHO: tty_release tty_port 0x%p\n", tty->port);
 #ifdef TTY_DEBUG_HANGUP
        printk(KERN_DEBUG "%s: freeing tty structure...\n", __func__);
 #endif
drivers/tty/tty_port.c +1 -0
index b7ff59d..346e30b 100644
@@ -139,6 +139,7 @@ EXPORT_SYMBOL(tty_port_destroy);
 static void tty_port_destructor(struct kref *kref)
 {
        struct tty_port *port = container_of(kref, struct tty_port, kref);
+pr_info("AHO: tty_port_destructor 0x%p\n", port);
        if (port->xmit_buf)
                free_page((unsigned long)port->xmit_buf);
        tty_port_destroy(port);

--------------

Happily it looks like ptys are not affected, the destructor seems to be always called after tty_release() and not before. But I'm not sure if I tried all possible ways (from userland).

Regards,

Alexander Holler

#2 lore raw
In reply to Alexander Holler
Quoted 11 lines · Alexander Holler

Hello,

after some pain because the "big step" (ecbbfd4) happened while the support for my AMD CPU was broken and thus git bisect hit a series of kernels which didn't boot, I've finally found the cause for a memory corruption: tty_ldisc_release().

What happens is the following:

tty_port is self-destructing, that means it destroys itself in tty_port.c:tty_port_destructor() when the last reference is gone. E.g. in case of rfcomm this happens with the call to tty->ops->close() in tty_io.c:tty_release().

The problem here is that tty_io.c:tty_release() calls tty_ldisc.c:tty_ldisc_release() which uses the tty_port to flush the ldisc work queues.

In the best case this hits a BUG() in cancel_work_sync() but often it just causes a memory corruption without a BUG() got hit before.

Hi Alexander,

Actually, the problem is that tty->ops->close() shouldn't be the last kref on the port.

It doesn't look to me like device removal is being handled properly.

Regards,
Peter Hurley

#3 lore raw

Am 16.05.2013 15:47, schrieb Peter Hurley:

Quoted 22 lines · Alexander Holler
Quoted 14 lines

Hello,

after some pain because the "big step" (ecbbfd4) happened while the support for my AMD CPU was broken and thus git bisect hit a series of kernels which didn't boot, I've finally found the cause for a memory
corruption: tty_ldisc_release().

What happens is the following:

tty_port is self-destructing, that means it destroys itself in tty_port.c:tty_port_destructor() when the last reference is gone. E.g. in case of rfcomm this happens with the call to tty->ops->close() in tty_io.c:tty_release().

Quoted 6 lines

The problem here is that tty_io.c:tty_release() calls tty_ldisc.c:tty_ldisc_release() which uses the tty_port to flush the ldisc work queues.

In the best case this hits a BUG() in cancel_work_sync() but often it just causes a memory corruption without a BUG() got hit before.

Quoted 8 lines

Hi Alexander,

Actually, the problem is that tty->ops->close() shouldn't be the last kref on the port.

It doesn't look to me like device removal is being handled properly.

Maybe, but if so, that should be documented (and ideally prevented). Especially since it seemed to have been worked before tty_ports got introduced.

But I can't add much more to this discussion, as I'm rather a novice in regard to the tty subsystem. I even don't know much about the task sharing between tty, tty_port and tty_ldisc, except the stuff I found out because I got hit by that bug and therefor have read some of the sources.

Regards,

Alexander Holler

#4 lore raw
Quoted 25 lines · Alexander Holler

Am 16.05.2013 15:47, schrieb Peter Hurley:

Quoted 24 lines · Alexander Holler
Quoted 14 lines

Hello,

after some pain because the "big step" (ecbbfd4) happened while the support for my AMD CPU was broken and thus git bisect hit a series of kernels which didn't boot, I've finally found the cause for a memory
corruption: tty_ldisc_release().

What happens is the following:

tty_port is self-destructing, that means it destroys itself in tty_port.c:tty_port_destructor() when the last reference is gone. E.g. in case of rfcomm this happens with the call to tty->ops->close() in tty_io.c:tty_release().

Quoted 6 lines

The problem here is that tty_io.c:tty_release() calls tty_ldisc.c:tty_ldisc_release() which uses the tty_port to flush the ldisc work queues.

In the best case this hits a BUG() in cancel_work_sync() but often it just causes a memory corruption without a BUG() got hit before.

Hi Alexander,

Quoted 8 lines
Quoted 6 lines

Actually, the problem is that tty->ops->close() shouldn't be the last kref on the port.

It doesn't look to me like device removal is being handled properly.

Maybe, but if so, that should be documented (and ideally prevented).

The tty_port documentation is trapped in the place as _all_ the bluetooth documentation :)

And the tty layer can't really _prevent_ the tty driver from mishandling the port kref.

Quoted 1 line

Especially since it seemed to have been worked before tty_ports got introduced.

Well, at the time tty_port was introduced to RFCOMM, there was nothing to tear-down in tty_port. Now that tty_port owns the flip buffers and must do proper tear-down, the problem has surfaced.

Quoted 1 line

But I can't add much more to this discussion, as I'm rather a novice in regard to the tty subsystem. I even don't know much about the task sharing between tty, tty_port and tty_ldisc, except the stuff I found out because I got hit by that bug and therefor have read some of the sources.

Ok. Could you paste the BUG() and steps to reproduce? I have a plan to fix it but I'd like to review what you have first.

Regards,
Peter Hurley

#5 lore raw

Am 16.05.2013 23:53, schrieb Peter Hurley:

Quoted 16 lines

And the tty layer can't really _prevent_ the tty driver from mishandling the port kref.

Quoted 2 lines

Especially since it seemed to have been worked before tty_ports got introduced.

Well, at the time tty_port was introduced to RFCOMM, there was nothing to tear-down in tty_port. Now that tty_port owns the flip buffers and must do proper tear-down, the problem has surfaced.

Quoted 5 lines

But I can't add much more to this discussion, as I'm rather a novice in regard to the tty subsystem. I even don't know much about the task sharing between tty, tty_port and tty_ldisc, except the stuff I found out because I got hit by that bug and therefor have read some of the sources.

Quoted 3 lines

Ok. Could you paste the BUG() and steps to reproduce? I have a plan to fix it but I'd like to review what you have first.

As described before, it ends up with memory corruption because freed memory is used, so if a BUG() happens, it doesn't help much. E.g. with kernel 3.9.2 I never have seen a bug, just a rebooting machine (sometimes minutes after the real bug happened).

To reproduce it, call rfcomm connect /dev/rfcommN and after the connection to the remote device happened, power down the remote device and wait 20s (the timeout until a connection drop will be discovered). Furthermore I would suggest to use commit ecbbfd4, because of the above mentioned problem. With that you might have luck and see a BUG like this:

May 16 00:06:18 laptopahvpn kernel: [   51.238969] ------------[ cut
here ]------------
May 16 00:06:18 laptopahvpn kernel: [   51.241754] kernel BUG at
kernel/workqueue.c:609!
May 16 00:06:18 laptopahvpn kernel: [    5.603591] error attempted to
write to tty [0x          (null)] = NULL
May 16 00:06:18 laptopahvpn kernel: [   51.244131] invalid opcode: 0000
[#1] SMP
May 16 00:06:18 laptopahvpn kernel: [   51.249491] Modules linked in:

sch_sfq cdc_acm msr nfs lockd sunrpc rfcomm bnep iptable_nat nf_na t_ipv4 nf_nat nf_conntrack_ipv4 nf_defrag_ipv4 xt_recent xt_conntrack nf_conntrack iptable_filter xt_LOG xt_limit ip6table_filter ip6_ta bles ipv6 btusb bluetooth snd_hda_codec_hdmi coretemp kvm_intel snd_hda_codec_realtek arc4 kvm crc32c_intel iwldvm ghash_clmulni_intel mac80211 aesni_intel aes_x86_64 ablk_helper cryptd samsung_laptop xts lrw gf128mul iwlwifi microcode cfg80211 xhci_hcd rfkill snd_hda_intel snd_hda_codec snd_hwdep snd_pcm ehci_hcd snd_page_alloc snd_timer snd usbcore soundcore lpc_ich usb_common mfd_core joydev

May 16 00:06:18 laptopahvpn kernel: [   51.261073] CPU 1
May 16 00:06:18 laptopahvpn kernel: [   51.261106] Pid: 2449, comm:
rfcomm Not tainted 3.7.0-rc2-00023-gecbbfd4-dirty #208 SAMSUNG
ELECTRONICS CO., LTD. 900X3C/900X3D/900X4C/900X4D/SAMSUNG_NP1234567890
May 16 00:06:18 laptopahvpn kernel: [   51.266958] RIP:
0010:[<ffffffff810492fe>]  [<ffffffff810492fe>] get_work_gcwq+0x5e/0x60
May 16 00:06:18 laptopahvpn kernel: [   51.270064] RSP:
0018:ffff88020f253da0  EFLAGS: 00010016
May 16 00:06:18 laptopahvpn kernel: [   51.273155] RAX: ffffffff81931380
RBX: ffff880214fee400 RCX: 0000000000000024
May 16 00:06:18 laptopahvpn kernel: [   51.276270] RDX: 007fffc4010a7f73
RSI: 0000000000000000 RDI: ffff880214fee400
May 16 00:06:18 laptopahvpn kernel: [   51.279333] RBP: 0000000000000000
R08: 000000000000000a R09: 000000000000181c
May 16 00:06:18 laptopahvpn kernel: [   51.282319] R10: 0000000000000000
R11: 000000000000181b R12: 0000000000000000
May 16 00:06:18 laptopahvpn kernel: [   51.285286] R13: 0000000000000004
R14: ffff880210863000 R15: 0000000000000000
May 16 00:06:18 laptopahvpn kernel: [   51.288265] FS:
00007f8bd6e94700(0000) GS:ffff88021f280000(0000) knlGS:0000000000000000
May 16 00:06:18 laptopahvpn kernel: [   51.291283] CS:  0010 DS: 0000
ES: 0000 CR0: 0000000080050033
May 16 00:06:18 laptopahvpn kernel: [   51.294328] CR2: 00007fc249111e60
CR3: 000000020f1d3000 CR4: 00000000001407e0
May 16 00:06:18 laptopahvpn kernel: [   51.297415] DR0: 0000000000000000
DR1: 0000000000000000 DR2: 0000000000000000
May 16 00:06:18 laptopahvpn kernel: [   51.300506] DR3: 0000000000000000
DR6: 00000000ffff0ff0 DR7: 0000000000000400
May 16 00:06:18 laptopahvpn kernel: [   51.303555] Process rfcomm (pid:
2449, threadinfo ffff88020f252000, task ffff880210bbee80)
May 16 00:06:18 laptopahvpn kernel: [   51.306638] Stack:
May 16 00:06:18 laptopahvpn kernel: [   51.309704]  ffffffff8104a471
0000000000014040 0000000000000296 ffff880210863000
May 16 00:06:18 laptopahvpn kernel: [   51.312850]  0000000000000000
0000000000000001 ffffffff81258188 0000000000000000
May 16 00:06:18 laptopahvpn kernel: [   51.315998]  ffffffff812591b4
0000000000013fc0 ffff880215278700 0000000000000000
May 16 00:06:18 laptopahvpn kernel: [   51.319139] Call Trace:
May 16 00:06:18 laptopahvpn kernel: [   51.322236]  [<ffffffff8104a471>]
? __cancel_work_timer+0x31/0xa0
May 16 00:06:18 laptopahvpn kernel: [   51.325398]  [<ffffffff81258188>]
? tty_ldisc_halt+0x18/0x20
May 16 00:06:18 laptopahvpn kernel: [   51.328551]  [<ffffffff812591b4>]
? tty_ldisc_release+0x34/0x110
May 16 00:06:18 laptopahvpn kernel: [   51.331719]  [<ffffffff81251dbc>]
? tty_release+0x4ac/0x520
May 16 00:06:18 laptopahvpn kernel: [   51.334873]  [<ffffffff810f2161>]
? __fput+0xe1/0x230
May 16 00:06:18 laptopahvpn kernel: [   51.338030]  [<ffffffff8104d75f>]
? task_work_run+0x8f/0xd0
May 16 00:06:18 laptopahvpn kernel: [   51.341208]  [<ffffffff81002919>]
? do_notify_resume+0x69/0xc0
May 16 00:06:18 laptopahvpn kernel: [   51.344383]  [<ffffffff8104d649>]
? task_work_add+0x49/0x60
May 16 00:06:18 laptopahvpn kernel: [   51.347578]  [<ffffffff81422e1a>]
? int_signal+0x12/0x17
May 16 00:06:18 laptopahvpn kernel: [   51.350777] Code: d5 a0 d3 85 81
c3 0f 1f 80 00 00 00 00 31 c0 66 0f 1f 44 00 00 f3 c3 66 0f 1f 44 00 00
30 c0 48 8b 00 48 8b 00 c3 83 fa 04 74 ea <0f> 0b e8 9b ff ff ff ba 05
00 00 00 48 85 c0 74 03 8b 50 04 89
May 16 00:06:18 laptopahvpn kernel: [   51.358380] RIP
[<ffffffff810492fe>] get_work_gcwq+0x5e/0x60
May 16 00:06:18 laptopahvpn kernel: [   51.362070]  RSP <ffff88020f253da0>
May 16 00:06:18 laptopahvpn kernel: [   51.365766] ---[ end trace

f2ccc5bea5182396 ]---

But only fixing the problem with rewriting rfcomm/tty.c but without any explanations about the expected lifetime of tty_port doesn't help much. As proved the switch to tty_port has some pitfalls and even people with a deeper insight into the new tty layer entered them.

E.g. the fact that tty_port is self-destructing suggests the conclusion that the problem isn't in rfcomm, but in tty_release() (that's why I placed the wrong workaround there).

So without at least some small clarification about the expected lifetime of tty_port, it's likely someone else will enter the same pit (which unfortunately isn't seen that easy and a BUG() doesn't have to happen). In include/linux/tty.h is just

"The tty port has a different lifetime to the tty so must be kept apart."

As it isn't specified that tty_port has to live as long as tty, I would (again) conclude it could have a shorter livetime than tty. Maybe someone can clarify that statement there.

I assume I would be able to fix the problem in rfcomm myself, if someone would offer me an explanation about the expected lifetime of tty_port and some confirmation, that the call of tty_ldisc_release() in tty_release() isn't the real problem.

E.g. why isn't that call to tty_ldisc_release() in tty_port_destructor() or in tty_port_destroy()? If it would be there the problem (and one pitfall) would be gone too. struct tty_port seems to have a pointer to tty (even two, tty and itty), so calling tty_ldisc_release() in tty_port_destroy() looks possible.

Regards,

Alexander Holler

#6 lore raw

PATCH 1/1 tty: make sure a BUG is hit if tty_port will be destroyed before tty

tty depends on tty_port until tty_release() was called. Make sure a BUG will be hit, if tty_port will be destroyed before tty.

---

Currently things are changing fast in the tty subsystem, therefor I don't know if the patch should be applied to kernel 3.10 too because the reference to tty_port in tty_ldisc_halt() is gone in 3.10-rc1.

So it might be a patch only for the stable kernels since commit ecbbfd4 (kernels 3.8 and 3.9).

 drivers/tty/tty_port.c | 2 ++
 1 file changed, 2 insertions(+)
drivers/tty/tty_port.c +2 -0
index 121aeb9..a40c52b 100644
@@ -140,6 +140,8 @@ EXPORT_SYMBOL(tty_port_destroy);
 static void tty_port_destructor(struct kref *kref)
 {
 	struct tty_port *port = container_of(kref, struct tty_port, kref);
+	/* tty_port has to live until tty_release() was called. */
+	BUG_ON(port->itty);
 	if (port->xmit_buf)
 		free_page((unsigned long)port->xmit_buf);
 	tty_port_destroy(port);
Signature · 1 line
1.8.1.4
#7 lore raw

Re: [PATCH] tty: make sure a BUG is hit if tty_port will be destroyed before tty

Quoted 2 lines · Alexander Holler

tty depends on tty_port until tty_release() was called. Make sure a BUG will be hit, if tty_port will be destroyed before tty.

So you want to ensure that we crash a machine? No, please never add BUG() statements to the kernel, unless something _really_ bad is going to happen if we don't call it. I never want to stop a machine from running, do you?

I can't take this as-is, why not just fix the root problem?

thanks,

greg k-h

#8 lore raw

Re: [PATCH] tty: make sure a BUG is hit if tty_port will be destroyed before tty

Am 17.05.2013 17:31, schrieb Greg Kroah-Hartman:

Quoted 5 lines · Alexander Holler
Quoted 2 lines

tty depends on tty_port until tty_release() was called. Make sure a BUG will be hit, if tty_port will be destroyed before tty.

So you want to ensure that we crash a machine? No, please never add

Exactly. Let me quote myself:

Quoted 8 lines
Quoted 4 lines

As described before, it ends up with memory corruption because freed memory is used, so if a BUG() happens, it doesn't help much. E.g. with kernel 3.9.2 I never have seen a bug, just a rebooting machine (sometimes minutes after the real bug happened).

BUG() statements to the kernel, unless something _really_ bad is going to happen if we don't call it. I never want to stop a machine from running, do you?

Yes. I'm not sure how you define _really_ bad, but a memory corruption with undefined result is exactly how I would define such.

And in the case of rfcomm, the box doesn't stop, at least not here. Just the process is killed together with an easy to identfiy oops. And the BUG_ON() prevents that memory will become corrupted and the machine is still usable afterwards. If that isn't a use case for BUG_ON(), I really don't know what else would be a use case for it.

Quoted 1 line

I can't take this as-is, why not just fix the root problem?

First I'm still not sure about the root problem and awaiting some response to my mail before that patch. As noted in the mail with the patch, 3.10-rc1 looks different, so the it might already be fixed there, even if rfcomm doesn't handle the tty as it (now in 3.8 and 3.9) should be (I haven't tested 3.10-rc1 up to now).

Second, if I would fix the bug in rfcomm, as Peter suggested, I still would not know if the same problem doesn't appear in any other user of ttys too, so even if I would fix rfcomm, I still would want that BUG_ON() to make sure I don't get a memory corruption whenever another similiar bug is hit.

Regards,

Alexander Holler

#9 lore raw

Re: [PATCH] tty: make sure a BUG is hit if tty_port will be destroyed before tty

Quoted 14 lines · Alexander Holler

Am 17.05.2013 17:31, schrieb Greg Kroah-Hartman:

Quoted 5 lines · Alexander Holler
Quoted 2 lines

tty depends on tty_port until tty_release() was called. Make sure a BUG will be hit, if tty_port will be destroyed before tty.

So you want to ensure that we crash a machine? No, please never add

Exactly. Let me quote myself:

Quoted 4 lines
Quoted 4 lines

As described before, it ends up with memory corruption because freed memory is used, so if a BUG() happens, it doesn't help much. E.g. with kernel 3.9.2 I never have seen a bug, just a rebooting machine (sometimes minutes after the real bug happened).

Quoted 5 lines
Quoted 3 lines

BUG() statements to the kernel, unless something _really_ bad is going to happen if we don't call it. I never want to stop a machine from running, do you?

Yes. I'm not sure how you define _really_ bad, but a memory corruption with undefined result is exactly how I would define such.

First, I like the idea of a diagnostic here. But I'm with Greg on this; BUG() is overkill. Just because the specific path which you found only kills the process doesn't mean that other callers might not prompt machine halt.

The memory corruption happens as a result of the tty_port being freed by tty_port_destructor(). So a suitable diagnostic is to detect the condition, WARN, and return without actually performing the destroy, yes?

Quoted 5 lines

And in the case of rfcomm, the box doesn't stop, at least not here. Just the process is killed together with an easy to identfiy oops. And the BUG_ON() prevents that memory will become corrupted and the machine is still usable afterwards. If that isn't a use case for BUG_ON(), I really don't know what else would be a use case for it.

Quoted 1 line

I can't take this as-is, why not just fix the root problem?

First I'm still not sure about the root problem and awaiting some response to my mail before that patch. As noted in the mail with the patch, 3.10-rc1 looks different, so the it might already be fixed there, even if rfcomm doesn't handle the tty as it (now in 3.8 and 3.9) should be (I haven't tested 3.10-rc1 up to now).

This problem exists from the commit identified up through current mainline. It has not been fixed yet. I'll specifically respond to what I believe is the correct solution in that thread.

Quoted 1 line

Second, if I would fix the bug in rfcomm, as Peter suggested, I still would not know if the same problem doesn't appear in any other user of ttys too, so even if I would fix rfcomm, I still would want that BUG_ON() to make sure I don't get a memory corruption whenever another similiar bug is hit.

Only a handful of tty drivers use the ref-counted destroy method of tty_port cleanup (the other method being calling tty_port_destroy() directly). They are:

drivers/tty/pty.c
drivers/mmc/card/sdio_uart.c
drivers/idsn/capi/capi.c
drivers/usb/class/cdc-acm.c
drivers/tty/n_gsm.c
drivers/tty/hvc/hvc_console.c
drivers/tty/hvc/hvcs.c
net/bluetooth/rfcomm/tty.c

Of these, I know and have reviewed the first two drivers; their usage is correct.

While reviewing these additional drivers, maybe we should review whether it makes sense to
* require and document the tty_port to live at least

    to ops->cleanup() (currently the default)

* allow tty_port lifetime to be completely independent of a tty's lifetime
* remove ref-counting from tty_port

Regards,
Peter Hurley

#10 lore raw

Re: [PATCH] tty: make sure a BUG is hit if tty_port will be destroyed before tty

Am 17.05.2013 20:06, schrieb Peter Hurley:

Quoted 23 lines · Alexander Holler
Quoted 15 lines

Am 17.05.2013 17:31, schrieb Greg Kroah-Hartman:

Quoted 5 lines · Alexander Holler
Quoted 2 lines

tty depends on tty_port until tty_release() was called. Make sure a BUG will be hit, if tty_port will be destroyed before tty.

So you want to ensure that we crash a machine? No, please never add

Exactly. Let me quote myself:

Quoted 2 lines
Quoted 2 lines

As described before, it ends up with memory corruption because freed memory is used, so if a BUG() happens, it doesn't help much. E.g.

with

Quoted 2 lines
Quoted 2 lines

kernel 3.9.2 I never have seen a bug, just a rebooting machine (sometimes minutes after the real bug happened).

Quoted 6 lines
Quoted 3 lines

BUG() statements to the kernel, unless something _really_ bad is going to happen if we don't call it. I never want to stop a machine from running, do you?

Yes. I'm not sure how you define _really_ bad, but a memory corruption with undefined result is exactly how I would define such.

Quoted 14 lines

First, I like the idea of a diagnostic here. But I'm with Greg on this; BUG() is overkill. Just because the specific path which you found only kills the process doesn't mean that other callers might not prompt machine halt.

The memory corruption happens as a result of the tty_port being freed by tty_port_destructor(). So a suitable diagnostic is to detect the condition, WARN, and return without actually performing the destroy, yes?

Quoted 5 lines

And in the case of rfcomm, the box doesn't stop, at least not here. Just the process is killed together with an easy to identfiy oops. And the BUG_ON() prevents that memory will become corrupted and the machine is still usable afterwards. If that isn't a use case for BUG_ON(), I really don't know what else would be a use case for it.

Sorry, I didn't express it such, that this can't be misunderstood. Without that BUG_ON() in my proposed patch, my boxes always died afterwards. And that with a lot of different results before. Sometimes nothing happened and the machine just rebooted, sometimes I've just seen a warn_slowpath before the machine stopped/rebooted, sometimes I've just got the BUG I posted in a previous mail, and often I've seen many OOPSes before the machine rebooted. But in every case, the machine died unexpectly.

The case that the machine didn't die, but just the process, only happens when my proposed patch is applied, which prevents the memory corruption.

Regards,

Alexander Holler

#11 lore raw

Re: [PATCH] tty: make sure a BUG is hit if tty_port will be destroyed before tty

Am 17.05.2013 21:22, schrieb Alexander Holler:

Quoted 2 lines

The case that the machine didn't die, but just the process, only happens when my proposed patch is applied, which prevents the memory corruption.

In short, the proposed BUG_ON() prevents the memory corruption because it is hit before something bad can happen. The result is that just the process in question will be killed (and a tty is not released), but only that BUG_ON() prevents that something _really_ bad happens.

I hope I could describe it now clearly. ;)

Regards,

Alexander

#12 lore raw

Re: [PATCH] tty: make sure a BUG is hit if tty_port will be destroyed before tty

Quoted 11 lines · Alexander Holler

Am 17.05.2013 21:22, schrieb Alexander Holler:

Quoted 2 lines

The case that the machine didn't die, but just the process, only happens when my proposed patch is applied, which prevents the memory corruption.

In short, the proposed BUG_ON() prevents the memory corruption because it is hit before something bad can happen. The result is that just the process in question will be killed (and a tty is not released), but only that BUG_ON() prevents that something _really_ bad happens.

I hope I could describe it now clearly. ;)

Your descriptions have been clear and I understood your meaning. However, I think you may have misunderstood my suggestion.

Would you please test the patch below?

--- >% ---
Subject: [PATCH] tty: Prevent tty_port destruction if tty not released

If the tty driver mistakenly drops the last port reference before the tty has been released, issue a diagnostic and abort the port destruction.

This will leak memory and may zombify the port, but should otherwise keep the machine in runnable state.

---
 drivers/tty/tty_port.c | 4 ++++
 1 file changed, 4 insertions(+)
drivers/tty/tty_port.c +4 -0
index 6d9e0b2..a4f4fa9 100644
@@ -140,6 +140,10 @@ EXPORT_SYMBOL(tty_port_destroy);
 static void tty_port_destructor(struct kref *kref)
 {
 	struct tty_port *port = container_of(kref, struct tty_port, kref);
+
+	/* check if last port ref was dropped before tty release */
+	if (WARN_ON(port->itty))
+		return;
 	if (port->xmit_buf)
 		free_page((unsigned long)port->xmit_buf);
 	tty_port_destroy(port);
Signature · 1 line
1.8.1.2
#13 lore raw

Re: [PATCH] tty: make sure a BUG is hit if tty_port will be destroyed before tty

Am 18.05.2013 00:51, schrieb Peter Hurley:

Quoted 38 lines · Alexander Holler
Quoted 11 lines

Am 17.05.2013 21:22, schrieb Alexander Holler:

Quoted 2 lines

The case that the machine didn't die, but just the process, only happens when my proposed patch is applied, which prevents the memory corruption.

In short, the proposed BUG_ON() prevents the memory corruption because it is hit before something bad can happen. The result is that just the process in question will be killed (and a tty is not released), but only that BUG_ON() prevents that something _really_ bad happens.

I hope I could describe it now clearly. ;)

Your descriptions have been clear and I understood your meaning. However, I think you may have misunderstood my suggestion.

Would you please test the patch below?

--- >% ---
Subject: [PATCH] tty: Prevent tty_port destruction if tty not released

If the tty driver mistakenly drops the last port reference before the tty has been released, issue a diagnostic and abort the port destruction.

This will leak memory and may zombify the port, but should otherwise keep the machine in runnable state.

---
  drivers/tty/tty_port.c | 4 ++++
  1 file changed, 4 insertions(+)
drivers/tty/tty_port.c +0 -0
index 6d9e0b2..a4f4fa9 100644

drivers/tty/tty_port.c EXPORT_SYMBOL(tty_port_destroy); · line 149

Quoted 10 lines
drivers/tty/tty_port.c +4 -0
@@ -140,6 +140,10 @@ EXPORT_SYMBOL(tty_port_destroy);
  static void tty_port_destructor(struct kref *kref)
  {
      struct tty_port *port = container_of(kref, struct tty_port, kref);
+
+    /* check if last port ref was dropped before tty release */
+    if (WARN_ON(port->itty))
+        return;
      if (port->xmit_buf)
          free_page((unsigned long)port->xmit_buf);
      tty_port_destroy(port);

I don't have to test this, I see what will happen. Sorry, but I'm exhausted and need a break dealing with lkml and maintainers.

Regards,

Alexander Holler

#14 lore raw
In reply to Alexander Holler
Quoted 19 lines · Alexander Holler

Am 16.05.2013 23:53, schrieb Peter Hurley:

Quoted 9 lines

And the tty layer can't really _prevent_ the tty driver from mishandling the port kref.

Quoted 2 lines

Especially since it seemed to have been worked before tty_ports got introduced.

Well, at the time tty_port was introduced to RFCOMM, there was nothing to tear-down in tty_port. Now that tty_port owns the flip buffers and must do proper tear-down, the problem has surfaced.

Quoted 8 lines
Quoted 5 lines

But I can't add much more to this discussion, as I'm rather a novice in regard to the tty subsystem. I even don't know much about the task sharing between tty, tty_port and tty_ldisc, except the stuff I found out because I got hit by that bug and therefor have read some of the sources.

Ok. Could you paste the BUG() and steps to reproduce? I have a plan to fix it but I'd like to review what you have first.

Quoted 8 lines

As described before, it ends up with memory corruption because freed memory is used, so if a BUG() happens, it doesn't help much. E.g. with kernel 3.9.2 I never have seen a bug, just a rebooting machine (sometimes minutes after the real bug happened).

To reproduce it, call rfcomm connect /dev/rfcommN and after the connection to the remote device happened, power down the remote device and wait 20s (the timeout until a connection drop will be discovered).

I expect this behaviour depends on the remote device. Does the device close the RFCOMM session cleanly ? Perhaps an out of range test would be better so that the connection drops.

Quoted 113 lines

Furthermore I would suggest to use commit ecbbfd4, because of the above mentioned problem. With that you might have luck and see a BUG like this:

May 16 00:06:18 laptopahvpn kernel: [   51.238969] ------------[ cut
here ]------------
May 16 00:06:18 laptopahvpn kernel: [   51.241754] kernel BUG at
kernel/workqueue.c:609!
May 16 00:06:18 laptopahvpn kernel: [    5.603591] error attempted to
write to tty [0x          (null)] = NULL
May 16 00:06:18 laptopahvpn kernel: [   51.244131] invalid opcode: 0000
[#1] SMP
May 16 00:06:18 laptopahvpn kernel: [   51.249491] Modules linked in:

sch_sfq cdc_acm msr nfs lockd sunrpc rfcomm bnep iptable_nat nf_na t_ipv4 nf_nat nf_conntrack_ipv4 nf_defrag_ipv4 xt_recent xt_conntrack nf_conntrack iptable_filter xt_LOG xt_limit ip6table_filter ip6_ta bles ipv6 btusb bluetooth snd_hda_codec_hdmi coretemp kvm_intel snd_hda_codec_realtek arc4 kvm crc32c_intel iwldvm ghash_clmulni_intel mac80211 aesni_intel aes_x86_64 ablk_helper cryptd samsung_laptop xts lrw gf128mul iwlwifi microcode cfg80211 xhci_hcd rfkill snd_hda_intel snd_hda_codec snd_hwdep snd_pcm ehci_hcd snd_page_alloc snd_timer snd usbcore soundcore lpc_ich usb_common mfd_core joydev

May 16 00:06:18 laptopahvpn kernel: [   51.261073] CPU 1
May 16 00:06:18 laptopahvpn kernel: [   51.261106] Pid: 2449, comm:
rfcomm Not tainted 3.7.0-rc2-00023-gecbbfd4-dirty #208 SAMSUNG
ELECTRONICS CO., LTD. 900X3C/900X3D/900X4C/900X4D/SAMSUNG_NP1234567890
May 16 00:06:18 laptopahvpn kernel: [   51.266958] RIP:
0010:[<ffffffff810492fe>]  [<ffffffff810492fe>] get_work_gcwq+0x5e/0x60
May 16 00:06:18 laptopahvpn kernel: [   51.270064] RSP:
0018:ffff88020f253da0  EFLAGS: 00010016
May 16 00:06:18 laptopahvpn kernel: [   51.273155] RAX: ffffffff81931380
RBX: ffff880214fee400 RCX: 0000000000000024
May 16 00:06:18 laptopahvpn kernel: [   51.276270] RDX: 007fffc4010a7f73
RSI: 0000000000000000 RDI: ffff880214fee400
May 16 00:06:18 laptopahvpn kernel: [   51.279333] RBP: 0000000000000000
R08: 000000000000000a R09: 000000000000181c
May 16 00:06:18 laptopahvpn kernel: [   51.282319] R10: 0000000000000000
R11: 000000000000181b R12: 0000000000000000
May 16 00:06:18 laptopahvpn kernel: [   51.285286] R13: 0000000000000004
R14: ffff880210863000 R15: 0000000000000000
May 16 00:06:18 laptopahvpn kernel: [   51.288265] FS:
00007f8bd6e94700(0000) GS:ffff88021f280000(0000) knlGS:0000000000000000
May 16 00:06:18 laptopahvpn kernel: [   51.291283] CS:  0010 DS: 0000
ES: 0000 CR0: 0000000080050033
May 16 00:06:18 laptopahvpn kernel: [   51.294328] CR2: 00007fc249111e60
CR3: 000000020f1d3000 CR4: 00000000001407e0
May 16 00:06:18 laptopahvpn kernel: [   51.297415] DR0: 0000000000000000
DR1: 0000000000000000 DR2: 0000000000000000
May 16 00:06:18 laptopahvpn kernel: [   51.300506] DR3: 0000000000000000
DR6: 00000000ffff0ff0 DR7: 0000000000000400
May 16 00:06:18 laptopahvpn kernel: [   51.303555] Process rfcomm (pid:
2449, threadinfo ffff88020f252000, task ffff880210bbee80)
May 16 00:06:18 laptopahvpn kernel: [   51.306638] Stack:
May 16 00:06:18 laptopahvpn kernel: [   51.309704]  ffffffff8104a471
0000000000014040 0000000000000296 ffff880210863000
May 16 00:06:18 laptopahvpn kernel: [   51.312850]  0000000000000000
0000000000000001 ffffffff81258188 0000000000000000
May 16 00:06:18 laptopahvpn kernel: [   51.315998]  ffffffff812591b4
0000000000013fc0 ffff880215278700 0000000000000000
May 16 00:06:18 laptopahvpn kernel: [   51.319139] Call Trace:
May 16 00:06:18 laptopahvpn kernel: [   51.322236]  [<ffffffff8104a471>]
? __cancel_work_timer+0x31/0xa0
May 16 00:06:18 laptopahvpn kernel: [   51.325398]  [<ffffffff81258188>]
? tty_ldisc_halt+0x18/0x20
May 16 00:06:18 laptopahvpn kernel: [   51.328551]  [<ffffffff812591b4>]
? tty_ldisc_release+0x34/0x110
May 16 00:06:18 laptopahvpn kernel: [   51.331719]  [<ffffffff81251dbc>]
? tty_release+0x4ac/0x520
May 16 00:06:18 laptopahvpn kernel: [   51.334873]  [<ffffffff810f2161>]
? __fput+0xe1/0x230
May 16 00:06:18 laptopahvpn kernel: [   51.338030]  [<ffffffff8104d75f>]
? task_work_run+0x8f/0xd0
May 16 00:06:18 laptopahvpn kernel: [   51.341208]  [<ffffffff81002919>]
? do_notify_resume+0x69/0xc0
May 16 00:06:18 laptopahvpn kernel: [   51.344383]  [<ffffffff8104d649>]
? task_work_add+0x49/0x60
May 16 00:06:18 laptopahvpn kernel: [   51.347578]  [<ffffffff81422e1a>]
? int_signal+0x12/0x17
May 16 00:06:18 laptopahvpn kernel: [   51.350777] Code: d5 a0 d3 85 81
c3 0f 1f 80 00 00 00 00 31 c0 66 0f 1f 44 00 00 f3 c3 66 0f 1f 44 00 00
30 c0 48 8b 00 48 8b 00 c3 83 fa 04 74 ea <0f> 0b e8 9b ff ff ff ba 05
00 00 00 48 85 c0 74 03 8b 50 04 89
May 16 00:06:18 laptopahvpn kernel: [   51.358380] RIP
[<ffffffff810492fe>] get_work_gcwq+0x5e/0x60
May 16 00:06:18 laptopahvpn kernel: [   51.362070]  RSP <ffff88020f253da0>
May 16 00:06:18 laptopahvpn kernel: [   51.365766] ---[ end trace

f2ccc5bea5182396 ]---

But only fixing the problem with rewriting rfcomm/tty.c but without any explanations about the expected lifetime of tty_port doesn't help much. As proved the switch to tty_port has some pitfalls and even people with a deeper insight into the new tty layer entered them.

E.g. the fact that tty_port is self-destructing suggests the conclusion that the problem isn't in rfcomm, but in tty_release() (that's why I placed the wrong workaround there).

So without at least some small clarification about the expected lifetime of tty_port, it's likely someone else will enter the same pit (which unfortunately isn't seen that easy and a BUG() doesn't have to happen). In include/linux/tty.h is just

"The tty port has a different lifetime to the tty so must be kept apart."

As it isn't specified that tty_port has to live as long as tty, I would (again) conclude it could have a shorter livetime than tty. Maybe someone can clarify that statement there.

I assume I would be able to fix the problem in rfcomm myself, if someone would offer me an explanation about the expected lifetime of tty_port and some confirmation, that the call of tty_ldisc_release() in tty_release() isn't the real problem.

Quoted 6 lines

E.g. why isn't that call to tty_ldisc_release() in tty_port_destructor() or in tty_port_destroy()? If it would be there the problem (and one pitfall) would be gone too. struct tty_port seems to have a pointer to tty (even two, tty and itty), so calling tty_ldisc_release() in tty_port_destroy() looks possible.

I have experienced this crash on Mageia 3 kernel 3.8.13 on my laptop. I was using SLIP over a RFCOMM connection between my laptop and an ARM board. I was using ssh sessions on my laptop over Ethernet to control Bluetooth on the ARM board. If my KDE session dies, then the ssh sessions die which kills any slattach and rfcomm programs on my ARM target.

I had a failure that killed my KDE session on my external monitor, I did "service dm restart" to bring up the KDE login on my monitor but I closed the lid on my laptop which put everything to sleep. When I opened the lid on the laptop, the kernel panicked. /var/log/syslog showed a similar crash as reported in your E-mail. See below at end of this E-mail.

I suspect the dying KDE session disconnected SLIP and RFCOMM on both my laptop and my ARM target board.

Is there a bugzilla bug report that is tracking this issue ? If I can reproduce this issue then I would like to add more information to a bug report.

Note that I added some RFCOMM session disconnection fixes to Linux 3.10 commits (8 off) 24fd642ccb24c8b5732d7d7b5e98277507860b2a to fea7b02fbf73adb2e746f00ed279a782de7e74e4. This may or may not help this crash.

For completeness my crash bracktrace looks like the following. I think this kernel contains commit ecbbfd4.

Jun 25 12:52:32 localhost kernel: PM: Finishing wakeup.
Jun 25 12:52:32 localhost kernel: ------------[ cut here ]------------
Jun 25 12:52:32 localhost kernel: kernel BUG at kernel/workqueue.c:610!
Jun 25 12:52:32 localhost kernel: Restarting tasks ...
Jun 25 12:52:32 localhost kernel: invalid opcode: 0000 [#1] SMP
Jun 25 12:52:32 localhost kernel: Modules linked in: slip slhc lp tun ipt_IFWLOG ipt_psd cls_basic cls_flow cls_fw cls_u32 sch_tbf sch_prio sch_htb sch_hfsc sch_ingress sch_sfq bridge stp llc xt_CHECKSUM ipt_rpfilter ip6t_rpfilter xt_statistic xt_CT xt_realm xt_LOG xt_connlimit xt_addrtype ip_set_hash_ip xt_comment xt_recent xt_nat ipt_ULOG ipt_REJECT ipt_MASQUERADE ipt_ECN ipt_CLUSTERIP ipt_ah xt_set ip_set nf_nat_tftp xt_time nf_nat_snmp_basic xt_TCPMSS nf_conntrack_snmp nf_nat_sip xt_sctp xt_policy nf_nat_pptp nf_nat_proto_gre ip6t_REJECT nf_conntrack_ipv6 nf_nat_irc ip6table_raw nf_nat_h323 ip6table_mangle nf_nat_ftp nf_nat_amanda ts_kmp nf_conntrack_amanda nf_conntrack_sane nf_conntrack_tftp nf_conntrack_sip nf_conntrack_proto_udplite nf_conntrack_proto_sctp nf_conntrack_pptp nf_conntrack_proto_gre nf_conntrack_netlink nf_conntrack_netbios_ns nf_conntrack_broadcast nf_conntrack_irc nf_conntrack_h323 nf_conntrack_ftp xt_TPROXY nf_defrag_ipv6 nf_tproxy_core xt_tcpmss xt_pkttype xt_physdev xt_owner xt_NFQUEUE xt_NFLOG nfnetlink_log xt_multiport xt_mark xt_mac xt_limit xt_length xt_iprange xt_helper xt_hashlimit xt_DSCP xt_dscp xt_dccp xt_conntrack xt_connmark xt_CLASSIFY xt_AUDIT xt_tcpudp xt_state iptable_raw iptable_nat nf_nat_ipv4 nf_nat nf_conntrack_ipv4 nf_defrag_ipv4 nf_conntrack iptable_mangle nfnetlink ip6table_filter ip6_tables lockd iptable_filter ip_tables x_tables vmnet(O) fuse vsock(O) vmci(O) vmmon(O) af_packet rndis_host cdc_ether usbnet mii rfcomm bnep ipv6 uvcvideo videobuf2_vmalloc videobuf2_memops videobuf2_core videodev btusb rt2800usb rt2x00usb rt2800lib rt2x00lib bluetooth crc_ccitt media pl2303 usbserial iTCO_wdt coretemp kvm_intel iTCO_vendor_support snd_hda_codec_idt arc4 iwldvm snd_hda_intel mac80211 snd_hda_codec snd_hwdep snd_pcm snd_page_alloc snd_timer snd kvm ppdev parport_pc mei lpc_ich i2c_i801 soundcore parport dell_laptop thermal battery dcdbas ghash_clmulni_intel ac yenta_socket dell_wmi sparse_keymap joydev iwlwifi e1000e pcmcia_rsrc pcmcia_core cfg80211 rfkill cpufreq_ondemand cpufreq_cons
Jun 25 12:52:52 localhost kernel: ervative cpufreq_powersave acpi_cpufreq mperf processor evdev nvram i8k sunrpc autofs4 hid_generic usbhid hid dm_crypt mmc_block sdhci_pci xhci_hcd ehci_pci ehci_hcd crc32c_intel usbcore aesni_intel xts sdhci aes_x86_64 lrw gf128mul ablk_helper cryptd mmc_core sr_mod usb_common nouveau i915 mxm_wmi wmi ttm video button i2c_algo_bit drm_kms_helper drm i2c_core dm_mod [last unloaded: microcode]
Jun 25 12:52:32 localhost kernel: CPU 0 <4>[15002.946347] Pid: 1946, comm: rfcomm Tainted: P           O 3.8.13-desktop-1.mga3 #1 Dell Inc. Latitude E6430/0H3MT5
Jun 25 12:52:32 localhost kernel: RIP: 0010:[<ffffffff81071ee1>]  [<ffffffff81071ee1>] get_work_gcwq+0x61/0x70
Jun 25 12:52:32 localhost kernel: RSP: 0018:ffff8801a4385d58 EFLAGS: 00010006
Jun 25 12:52:32 localhost kernel: RAX: ffffffff81c02a80 RBX: ffff8801dba86600 RCX: ffffffffffffffc8
Jun 25 12:52:32 localhost kernel: RDX: 007fffc400edd422 RSI: 0000000000000000 RDI: ffff8801dba86600
Jun 25 12:52:32 localhost kernel: RBP: ffff8801a4385d58 R08: 0000000000000000 R09: ffff88022e1fafa0
Jun 25 12:52:32 localhost kernel: R10: 0000000000000040 R11: 0000000000000000 R12: 0000000000000000
Jun 25 12:52:32 localhost kernel: R13: 0000000000000000 R14: 0000000000000000 R15: ffff8801ad1a1000
Jun 25 12:52:32 localhost kernel: FS:  00007fa709d21700(0000) GS:ffff88022e200000(0000) knlGS:0000000000000000
Jun 25 12:52:32 localhost kernel: CS:  0010 DS: 0000 ES: 0000 CR0: 0000000080050033
Jun 25 12:52:32 localhost kernel: CR2: 00007fa08e462cb0 CR3: 00000001acbca000 CR4: 00000000001407f0
Jun 25 12:52:32 localhost kernel: DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000
Jun 25 12:52:32 localhost kernel: DR3: 0000000000000000 DR6: 00000000ffff0ff0 DR7: 0000000000000400
Jun 25 12:52:32 localhost kernel: done.
Jun 25 12:52:32 localhost kernel: Process rfcomm (pid: 1946, threadinfo ffff8801a4384000, task ffff880223082da0)
Jun 25 12:52:32 localhost kernel: Stack:
Jun 25 12:52:32 localhost kernel: video LNXVIDEO:00: Restoring backlight state
Jun 25 12:52:32 localhost kernel:  ffff8801a4385d98 ffffffff81074a7b<6>[15002.946424] video LNXVIDEO:01: Restoring backlight state
Jun 25 12:52:32 localhost kernel:  ffff8801dba86724 0000000000000282
Jun 25 12:52:32 localhost kernel:  ffff8801a41c1b00 ffff8801ad1a1000 0000000000000000 0000000000000000
Jun 25 12:52:32 localhost kernel:  ffff8801a4385da8 ffffffff81074b20 ffff8801a4385db8 ffffffff81403cbd
Jun 25 12:52:32 localhost kernel: Call Trace:
Jun 25 12:52:32 localhost kernel:  [<ffffffff81074a7b>] __cancel_work_timer+0x3b/0xb0
Jun 25 12:52:32 localhost kernel:  [<ffffffff81074b20>] cancel_work_sync+0x10/0x20
Jun 25 12:52:32 localhost kernel:  [<ffffffff81403cbd>] tty_ldisc_halt+0x1d/0x30
Jun 25 12:52:32 localhost kernel:  [<ffffffff81404b17>] tty_ldisc_release+0x17/0x90
Jun 25 12:52:32 localhost kernel:  [<ffffffff813fd270>] tty_release+0x470/0x580
Jun 25 12:52:32 localhost kernel:  [<ffffffff8101440e>] ? do_signal+0xbe/0x8e0
Jun 25 12:52:32 localhost kernel:  [<ffffffff8117938c>] __fput+0xec/0x240
Jun 25 12:52:32 localhost kernel:  [<ffffffff811794ee>] ____fput+0xe/0x10
Jun 25 12:52:32 localhost kernel:  [<ffffffff81076e07>] task_work_run+0xa7/0xe0
Jun 25 12:52:32 localhost kernel:  [<ffffffff81014ca1>] do_notify_resume+0x71/0xb0
Jun 25 12:52:32 localhost kernel:  [<ffffffff815e12aa>] int_signal+0x12/0x17
Jun 25 12:52:32 localhost kernel: Code: e2 00 00 48 03 04 d5 00 bc ab 81 5d c3 0f 1f 44 00 00 30 c0 48 8b 00 5d 48 8b 00 c3 66 0f 1f 44 00 00 31 c0 5d c3 83 fa 40 74 e0 <0f> 0b 66 66 66 66 2e 0f 1f 84 00 00 00 00 00 0f 1f 44 00 00 55
Jun 25 12:52:32 localhost kernel: RIP  [<ffffffff81071ee1>] get_work_gcwq+0x61/0x70
Jun 25 12:52:32 localhost kernel:  RSP <ffff8801a4385d58>
Jun 25 12:52:32 localhost kernel: ---[ end trace f219a9b765a4acc9 ]---
Jun 25 12:52:52 localhost vmnetBridge: Removing interface eth0 index:2
Jun 25 12:52:52 localhost vmnetBridge: RTM_NEWLINK: name:eth0 index:2 flags:0x00011043
Jun 25 12:52:52 localhost vmnetBridge: Adding interface eth0 index:2

Thanks,

Regards,
Dean Jenkins
Mentor Graphics

#15 lore raw

Am 25.06.2013 16:18, schrieb Dean Jenkins:

Quoted 8 lines · Alexander Holler
Quoted 4 lines

To reproduce it, call rfcomm connect /dev/rfcommN and after the connection to the remote device happened, power down the remote device and wait 20s (the timeout until a connection drop will be discovered).

I expect this behaviour depends on the remote device. Does the device close the RFCOMM session cleanly ? Perhaps an out of range test would be better so that the connection drops.

Yes. With power down I meant a hard power down, such that the remote doesn't has the chance to close the session cleanly. It's easier to do than putting it out of range, at least if the remote isn't a battery-powered device. I should have described that more clearly.

Quoted 3 lines

Is there a bugzilla bug report that is tracking this issue ? If I can reproduce this issue then I would like to add more information to a bug report.

I haven't written a bugzilla bug report and won't write one.

Furthermore I've already found and described the problem, so more crash reports should not be necessary.

And they don't make much sense because they happen because of a disastreous memory corruption, which means the BUGs can include almost everything (hopefully nothing which eats your disk contents).

Regards,

Alexander Holler