From owner-freebsd-bugs@FreeBSD.ORG Mon Oct 13 19:35:16 2014 Return-Path: Delivered-To: freebsd-bugs@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 30FFD8A9 for ; Mon, 13 Oct 2014 19:35:16 +0000 (UTC) Received: from kenobi.freebsd.org (kenobi.freebsd.org [IPv6:2001:1900:2254:206a::16:76]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id F25FC676 for ; Mon, 13 Oct 2014 19:35:15 +0000 (UTC) Received: from bugs.freebsd.org ([127.0.1.118]) by kenobi.freebsd.org (8.14.9/8.14.9) with ESMTP id s9DJZFTC021576 for ; Mon, 13 Oct 2014 19:35:15 GMT (envelope-from bugzilla-noreply@freebsd.org) From: bugzilla-noreply@freebsd.org To: freebsd-bugs@FreeBSD.org Subject: [Bug 194209] ahciems should be optional (at build or at run time) Date: Mon, 13 Oct 2014 19:35:16 +0000 X-Bugzilla-Reason: AssignedTo X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: Base System X-Bugzilla-Component: kern X-Bugzilla-Version: 11.0-CURRENT X-Bugzilla-Keywords: X-Bugzilla-Severity: Affects Some People X-Bugzilla-Who: ngie@FreeBSD.org X-Bugzilla-Status: Needs Triage X-Bugzilla-Priority: --- X-Bugzilla-Assigned-To: freebsd-bugs@FreeBSD.org X-Bugzilla-Target-Milestone: --- X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: short_desc Message-ID: In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit X-Bugzilla-URL: https://bugs.freebsd.org/bugzilla/ Auto-Submitted: auto-generated MIME-Version: 1.0 X-BeenThere: freebsd-bugs@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: Bug reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 13 Oct 2014 19:35:16 -0000 https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=194209 Garrett Cooper changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|ahciems should be optional |ahciems should be optional | |(at build or at run time) --- Comment #2 from Garrett Cooper --- (In reply to Alexander Motin from comment #1) > Not sure I understand your proposition. Do you want ahciem to be separate > module, or do you want some tunable to disable it, or something else? I jumped to a conclusion without providing a proper problem statement. Basically we have a platform at work that has ahciems support, but our software isn't coded up to handle ahciems currently, so this causes our software to poke at the boot drives for ahciems status a little too much. Having the support on by default makes sense from a usability perspective. In general, being able to enable/disable ahciems on a global or a per-controller basis (like atc.wc.enable) is desirable, because the end-user might not want ahciems support to be enabled, or the device support provided/integrated in with some vendors might be buggy/broken. PS I was working on a patch to separate out the drivers, but then I realized that it would introduce unnecessary complexity because of how the driver currently hangs ahciems devices off of ahci controllers, and how the ahciems instances are created/destroyed at ahci controller attach/detach. -- You are receiving this mail because: You are the assignee for the bug.