ucom_free Fatal trap on shutdown / module unload

[Lundberg, Johannes]

As I mention in earlier mail, it is when I comment out following lines it
works

# For full debugger support use (turn off in stable branch):
options         DDB                     # Support DDB.
options         GDB                     # Support remote GDB.
options         DEADLKRES               # Enable the deadlock resolver
options         INVARIANTS              # Enable calls of extra sanity
checking
options         INVARIANT_SUPPORT       # Extra sanity checks of internal
structures, required by INVARIANTS
options         WITNESS                 # Enable checks to detect deadlocks
and cycles
options         WITNESS_SKIPSPIN        # Don't run witness on spinlocks
for speed
options         MALLOC_DEBUG_MAXZONES=8 # Separate malloc(9) zones


I have tried different levels of debugging and I have narrowed it down to
INVARIANTS.

That is

# For full debugger support use (turn off in stable branch):
#options         DDB                     # Support DDB.
#options         GDB                     # Support remote GDB.
#options         DEADLKRES               # Enable the deadlock resolver
#options         INVARIANTS              # Enable calls of extra sanity
checking
#options         INVARIANT_SUPPORT       # Extra sanity checks of internal
structures, required by INVARIANTS
#options         WITNESS                 # Enable checks to detect
deadlocks and cycles
#options         WITNESS_SKIPSPIN        # Don't run witness on spinlocks
for speed
#options         MALLOC_DEBUG_MAXZONES=8 # Separate malloc(9) zones

works but the following crashes when unloading uhso.

# For full debugger support use (turn off in stable branch):
#options         DDB                     # Support DDB.
#options         GDB                     # Support remote GDB.
#options         DEADLKRES               # Enable the deadlock resolver
options         INVARIANTS              # Enable calls of extra sanity
checking
options         INVARIANT_SUPPORT       # Extra sanity checks of internal
structures, required by INVARIANTS
#options         WITNESS                 # Enable checks to detect
deadlocks and cycles
#options         WITNESS_SKIPSPIN        # Don't run witness on spinlocks
for speed
#options         MALLOC_DEBUG_MAXZONES=8 # Separate malloc(9) zones




--
Johannes Lundberg
BRILLIANTSERVICE CO., LTD.


On Sat, Jun 28, 2014 at 11:56 PM, Hans Petter Selasky <hps at selasky.org>
wrote:

> On 06/28/14 08:59, Lundberg, Johannes wrote:
>
>> The problem exists already from Revision *239180*
>> <http://svnweb.freebsd.org/base?view=revision&revision=239180> where ucom
>>
>> was introduced.. Also observe that it only crashes when full debugging is
>> enabled. No problems with debugging off (release settings in kernel conf).
>>
>>
> Interesting.
>
> You mean USB_DEBUG is defined or hw.usb.ucom.debug is set?
>
> --HPS
>
>

-- 
=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
秘密保持について：この電子メールは、名宛人に送信したものであり、秘匿特権の対象となる情報を含んでいます。
もし、名宛人以外の方が受信された場合、このメールの破棄、およびこのメールに関する一切の開示、
複写、配布、その他の利用、または記載内容に基づくいかなる行動もされないようお願い申し上げます。
---
CONFIDENTIALITY NOTE: The information in this email is confidential
and intended solely for the addressee.
Disclosure, copying, distribution or any other action of use of this
email by person other than intended recipient, is prohibited.
If you are not the intended recipient and have received this email in
error, please destroy the original message.