From owner-p4-projects@FreeBSD.ORG Wed Nov 4 12:11:01 2009 Return-Path: Delivered-To: p4-projects@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 32767) id A742C10656A3; Wed, 4 Nov 2009 12:11:01 +0000 (UTC) Delivered-To: perforce@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 6B5A41065697 for ; Wed, 4 Nov 2009 12:11:01 +0000 (UTC) (envelope-from mav@freebsd.org) Received: from repoman.freebsd.org (repoman.freebsd.org [IPv6:2001:4f8:fff6::29]) by mx1.freebsd.org (Postfix) with ESMTP id 58B0C8FC21 for ; Wed, 4 Nov 2009 12:11:01 +0000 (UTC) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.14.3/8.14.3) with ESMTP id nA4CB19L016460 for ; Wed, 4 Nov 2009 12:11:01 GMT (envelope-from mav@freebsd.org) Received: (from perforce@localhost) by repoman.freebsd.org (8.14.3/8.14.3/Submit) id nA4CB1HT016458 for perforce@freebsd.org; Wed, 4 Nov 2009 12:11:01 GMT (envelope-from mav@freebsd.org) Date: Wed, 4 Nov 2009 12:11:01 GMT Message-Id: <200911041211.nA4CB1HT016458@repoman.freebsd.org> X-Authentication-Warning: repoman.freebsd.org: perforce set sender to mav@freebsd.org using -f From: Alexander Motin To: Perforce Change Reviews Precedence: bulk Cc: Subject: PERFORCE change 170157 for review X-BeenThere: p4-projects@freebsd.org X-Mailman-Version: 2.1.5 List-Id: p4 projects tree changes List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 04 Nov 2009 12:11:02 -0000 http://p4web.freebsd.org/chv.cgi?CH=170157 Change 170157 by mav@mav_mavtest on 2009/11/04 12:10:45 If several controllers complete requests same time, call swi_sched() only once. Affected files ... .. //depot/projects/scottl-camlock/src/sys/cam/cam_xpt.c#120 edit Differences ... ==== //depot/projects/scottl-camlock/src/sys/cam/cam_xpt.c#120 (text+ko) ==== @@ -4238,6 +4238,7 @@ xpt_done(union ccb *done_ccb) { struct cam_sim *sim; + int first; CAM_DEBUG(done_ccb->ccb_h.path, CAM_DEBUG_TRACE, ("xpt_done\n")); if ((done_ccb->ccb_h.func_code & XPT_FC_QUEUED) != 0) { @@ -4251,10 +4252,12 @@ done_ccb->ccb_h.pinfo.index = CAM_DONEQ_INDEX; if ((sim->flags & CAM_SIM_ON_DONEQ) == 0) { mtx_lock(&cam_simq_lock); + first = TAILQ_EMPTY(&cam_simq); TAILQ_INSERT_TAIL(&cam_simq, sim, links); mtx_unlock(&cam_simq_lock); sim->flags |= CAM_SIM_ON_DONEQ; - swi_sched(cambio_ih, 0); + if (first) + swi_sched(cambio_ih, 0); } } }