From owner-svn-src-projects@FreeBSD.ORG Sun Dec 16 17:44:43 2012 Return-Path: Delivered-To: svn-src-projects@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id 9449C536; Sun, 16 Dec 2012 17:44:43 +0000 (UTC) (envelope-from mavbsd@gmail.com) Received: from mail-wi0-f180.google.com (mail-wi0-f180.google.com [209.85.212.180]) by mx1.freebsd.org (Postfix) with ESMTP id BEB858FC0C; Sun, 16 Dec 2012 17:44:42 +0000 (UTC) Received: by mail-wi0-f180.google.com with SMTP id hj13so1477897wib.13 for ; Sun, 16 Dec 2012 09:44:36 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=sender:message-id:date:from:user-agent:mime-version:to:cc:subject :references:in-reply-to:content-type:content-transfer-encoding; bh=LeLksY2Grhm1N3TQDNNvXzutgpOFbWXOtX17dTexChU=; b=0JynrWrJ2ihusz4iV7+sUq9NRo+mb5tLgcGT48JDdY5wgOin6ksOMYangfSN+tMTbM BOqRS6KWdlTa1ojRqR4ayhg4UmiKWMEsB4BbniK+BuoxdOkr4Yzq062mssHBY1AQhHAb 4MUdXOReyJ5r73VYHRZ+wsgug8ASNJhQHOs0q4/YjohgCa418he5XR1usrlDHXyjautO vyF0pSGF1pqACwuwn+IFjywVOOPzKIVLvg/gHW48LV1zm7w1UdUkz+wsiSmVqmlZfuGW a2enGiWb2jEdEtdIU+TVAIxYFsin0QTOnYqEm705ZV+i513ygZ7YPf6rG7QhiHvvBy8g t9Rw== Received: by 10.194.79.34 with SMTP id g2mr13006356wjx.17.1355679876237; Sun, 16 Dec 2012 09:44:36 -0800 (PST) Received: from mavbook.mavhome.dp.ua (mavhome.mavhome.dp.ua. [213.227.240.37]) by mx.google.com with ESMTPS id fv2sm7524078wib.4.2012.12.16.09.44.34 (version=TLSv1/SSLv3 cipher=OTHER); Sun, 16 Dec 2012 09:44:35 -0800 (PST) Sender: Alexander Motin Message-ID: <50CE087A.2020200@FreeBSD.org> Date: Sun, 16 Dec 2012 19:44:26 +0200 From: Alexander Motin User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:13.0) Gecko/20120628 Thunderbird/13.0.1 MIME-Version: 1.0 To: Adrian Chadd Subject: Re: svn commit: r244287 - projects/calloutng/sys/x86/isa References: <201212161116.qBGBGEwn063983@svn.freebsd.org> <50CE009B.7010804@FreeBSD.org> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: svn-src-projects@freebsd.org, src-committers@freebsd.org X-BeenThere: svn-src-projects@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "SVN commit messages for the src " projects" tree" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 16 Dec 2012 17:44:43 -0000 On 16.12.2012 19:18, Adrian Chadd wrote: > On 16 December 2012 09:10, Alexander Motin wrote: > >> This change affects only one-shot operation mode of i8254 eventtimer, which >> we can't enable by default because it can't coexist with i8254 timecounter, >> which we can't disable by default because we can't be sure that there is any >> other timecounter. That is why I've written about fun. And no, I have no >> all possible weird i8254 clones to test, so any help and ideas are welcome. >> :) > > That's why I think you guys have to be very careful here. There's > still plenty of embedded x86 hardware out there which uses > not-quite-matching i8254 silicon implementations. > I realise it's not enabled by default, but by putting this code in > there, you risk having it eventually bubble up and become potentially > available/active on some quirky embedded platforms (or heck, > non-embedded platforms where for some reason the i8254 eventtimer is > active, but the timecounter isn't.) > > The point I'm trying to make here is that you guys shouldn't just > change things because you don't think that it'll not be used. You may > not think it's being used but the whole i386 space (embedded or > otherwise) is full of legacy (and non-legacy) hilarity. You should be > really careful that you don't break previous things which you just > don't have a chance in hell of being able to test thoroughly. > > We have 15 years of "mostly working" on a really really quirky > platform. You won't get that kind of testing again. > > I'm (kind of) sorry for being overly serious and ranty here - but this > is exactly the kind of thing that breaks hardware support in > unpredictable ways, making us all look bad. I understand your concerns, but we can't get somewhere without walking steps. As I've told, that feature is disabled by default, and even if it ever be possible, I will definitely think twice considering risks before enabling it. The only other thing I can tell into my defense is that I am running all this code on my personal laptop, several systems from different generations and classes on my lab table and some other nearby machines. That is not speaking about help from volunteers. If you know anything specific why or where that may cause problems, or which specific tests should I run, I will gladly accept your experience. -- Alexander Motin