From owner-cvs-src@FreeBSD.ORG Wed Sep 10 16:01:42 2008 Return-Path: Delivered-To: cvs-src@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 6F25B106567B; Wed, 10 Sep 2008 16:01:42 +0000 (UTC) (envelope-from sam@freebsd.org) Received: from ebb.errno.com (ebb.errno.com [69.12.149.25]) by mx1.freebsd.org (Postfix) with ESMTP id 419DC8FC1C; Wed, 10 Sep 2008 16:01:42 +0000 (UTC) (envelope-from sam@freebsd.org) Received: from trouble.errno.com (trouble.errno.com [10.0.0.248]) (authenticated bits=0) by ebb.errno.com (8.13.6/8.12.6) with ESMTP id m8AG1fWi062737 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Wed, 10 Sep 2008 09:01:41 -0700 (PDT) (envelope-from sam@freebsd.org) Message-ID: <48C7EF65.3050204@freebsd.org> Date: Wed, 10 Sep 2008 09:01:41 -0700 From: Sam Leffler Organization: FreeBSD Project User-Agent: Thunderbird 2.0.0.9 (X11/20071125) MIME-Version: 1.0 To: Weongyo Jeong References: <200809100341.m8A3f5n4001038@repoman.freebsd.org> In-Reply-To: <200809100341.m8A3f5n4001038@repoman.freebsd.org> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-DCC--Metrics: ebb.errno.com; whitelist Cc: cvs-src@freebsd.org, src-committers@freebsd.org, cvs-all@freebsd.org Subject: Re: cvs commit: src/sys/dev/usb if_zyd.c if_zydreg.h X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 10 Sep 2008 16:01:42 -0000 Weongyo Jeong wrote: > weongyo 2008-09-10 03:40:51 UTC > > FreeBSD src repository > > Modified files: > sys/dev/usb if_zyd.c if_zydreg.h > Log: > SVN rev 182900 on 2008-09-10 03:40:51Z by weongyo > > rename flags and add a ZYD_FLAG_DETACHING flag to indicate we're > detaching that when the USB is pulled out forcibly during the driver is > running background scan, a page fault can be occurred even if we called > usb_rem_task() when detaching. It looks like a kind of races. > > If I understand the issue, it should be handled in the 802.11 state machine. The device should be clocked to the INIT state and as a result clear any outstanding tasks, timers, etc. The only reason you need to do something special is if the h/w is gone and you need to guard against accessing it. Sam