From owner-freebsd-current@FreeBSD.ORG Fri Jan 23 19:12:25 2009 Return-Path: Delivered-To: current@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id C3DFE106564A; Fri, 23 Jan 2009 19:12:25 +0000 (UTC) (envelope-from maksim.yevmenkin@gmail.com) Received: from rv-out-0506.google.com (rv-out-0506.google.com [209.85.198.225]) by mx1.freebsd.org (Postfix) with ESMTP id 9449C8FC1E; Fri, 23 Jan 2009 19:12:25 +0000 (UTC) (envelope-from maksim.yevmenkin@gmail.com) Received: by rv-out-0506.google.com with SMTP id b25so4955990rvf.43 for ; Fri, 23 Jan 2009 11:12:25 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:received:date:message-id:subject :from:to:cc:content-type:content-transfer-encoding; bh=7eHRUhigtyV5gmsb7iT4hMaH+k55+IMX8Roey37Fv9k=; b=SCCd1H+tABkfV8Vj1hlJH4wgDmqHAMFliEHSiyd8VtdW/iI+oGpWkltdUiBm04ouEj 3lPV340wZtHf3f7ZueIqDdDxxJ5Q7hQ0rCHN53W+CXTgcET0wFBKNsr+SxVz1TQUhpBy 6t/JJEeyolrgmItMMETPcTRPv0XOwe3TK0V8Q= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:date:message-id:subject:from:to:cc:content-type :content-transfer-encoding; b=SthOrd4ARPvvZu0Vvsh1moE2PBFaCvy3s/TReBj5tCBdKuQ3yrSe8YvGAbAocPE3My aATSu8Iv+ZyH7n4jKC9HU+2KxLfUgP9lkzju/eugJcltGWsbJYn2U+0qbxT3pFfrJvpp vqUU269pL3oLHpgJMjDKJLmJbAUkWzzEWUMI4= MIME-Version: 1.0 Received: by 10.140.164.1 with SMTP id m1mr1372032rve.223.1232737945322; Fri, 23 Jan 2009 11:12:25 -0800 (PST) Date: Fri, 23 Jan 2009 11:12:25 -0800 Message-ID: From: Maksim Yevmenkin To: Lars Engels Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: current@freebsd.org, Hans Petter Selasky Subject: Re: [PATCH] for ng_ubt2 stalled transfers X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 23 Jan 2009 19:12:26 -0000 Lars, > Maksim has made a lot of changes to the Bluetooth driver. Maybe he can look > into it? > > This looks like a NULL-pointer access to me. > > Maksim: > > In the ubt_detach routine, make sure that: > > 0) No further commands are accepted from the Netgraph stack. > 1) You drain the task queue! > 2) You drain all USB transfers: "usb2_transfer_drain()" called unlocked. > 3) Set the "sc_node" to NULL. please try attached patch for ng_ubt2 that hopefully addresses some of the issues with stalled transfers. stalled transfers do not happen on my machine, so i can not test it. i briefly kicked the tires and made sure other things seems to work fine for me. thanks, max