From owner-freebsd-new-bus@FreeBSD.ORG Sun Nov 8 16:53:23 2009 Return-Path: Delivered-To: freebsd-new-bus@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 30FC7106566C; Sun, 8 Nov 2009 16:53:23 +0000 (UTC) (envelope-from asmrookie@gmail.com) Received: from mail-bw0-f213.google.com (mail-bw0-f213.google.com [209.85.218.213]) by mx1.freebsd.org (Postfix) with ESMTP id 3C6C58FC19; Sun, 8 Nov 2009 16:53:21 +0000 (UTC) Received: by bwz5 with SMTP id 5so2814165bwz.3 for ; Sun, 08 Nov 2009 08:53:21 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:sender:received:date :x-google-sender-auth:message-id:subject:from:to:content-type; bh=me+o6nRiTx3NpeZ0D20Ev0VlUBLILufcBO+iyUBNY2M=; b=fdq0sUBgoCTXD7gkDwSmnCjVISy1sc83K/zGzglbxxhCIxJ415O8VU46GTb5Mhb3pm yHnebjnVTSrhA+QpTmcVQWLQl7STgvG3RDT+ueGV+9/R7uYUA/Tv2iRJXdWY7inLNi0a s+02j7Fefby+38yVeD0+5j1g6kQnub1xeWK0g= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:sender:date:x-google-sender-auth:message-id:subject :from:to:content-type; b=pmKIsL/1ZrKwPmTl+CzIIYJYkpVyulJ3T4mX9/s7xfRKGbQR60+GTGT3+3fIZg3S6S AfMIvEa+SxuKksSwD3r0fhfKL/WtwNUVK0jTXOra+EzYwpU0FBoGU5bXXD5KqduT57R6 AQAHePEgIKHKaVXrTCNRHTiWxTqFY3/CEv+LA= MIME-Version: 1.0 Sender: asmrookie@gmail.com Received: by 10.223.4.137 with SMTP id 9mr1037589far.95.1257699200829; Sun, 08 Nov 2009 08:53:20 -0800 (PST) Date: Sun, 8 Nov 2009 17:53:20 +0100 X-Google-Sender-Auth: eba04ea05d7f9b69 Message-ID: <3bbf2fe10911080853qad0057w490694e5627681fe@mail.gmail.com> From: Attilio Rao To: freebsd-new-bus@freebsd.org, John Baldwin , Scott Long , Warner Losh Content-Type: text/plain; charset=UTF-8 Cc: Subject: [PATCH] Collapsing device_state_t with devinfo_state_t X-BeenThere: freebsd-new-bus@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: FreeBSD's new-bus architecture List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 08 Nov 2009 16:53:23 -0000 This patch should collpase device_state_t with devinfo_state_t and remove a bogus replication of this struct: http://www.freebsd.org/~attilio/devinfo.diff This patch breaks ABI so it cannot be MFC'ed (and it is not intended to do). Thanks, Attilio -- Peace can only be achieved by understanding - A. Einstein From owner-freebsd-new-bus@FreeBSD.ORG Sun Nov 8 16:54:53 2009 Return-Path: Delivered-To: freebsd-new-bus@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 6B2A3106566C; Sun, 8 Nov 2009 16:54:53 +0000 (UTC) (envelope-from asmrookie@gmail.com) Received: from mail-bw0-f213.google.com (mail-bw0-f213.google.com [209.85.218.213]) by mx1.freebsd.org (Postfix) with ESMTP id 4B4798FC1E; Sun, 8 Nov 2009 16:54:52 +0000 (UTC) Received: by bwz5 with SMTP id 5so2815003bwz.3 for ; Sun, 08 Nov 2009 08:54:51 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:sender:received:in-reply-to :references:date:x-google-sender-auth:message-id:subject:from:to :content-type; bh=cI9mLxv/qcnMm31wPVs1U4HBxB4nuSpDBvjWF8QENpw=; b=whrqSsm0nicMFw/0HVen0GHl+bD+wZI096v0pkDhgH0UKayEphC2l53aA9GBwwXVOg RJWgaLg8Of0t+rlscyQN5ZByqVem2282j7njvZxW4vRsZCr94cAaK6EHvQm1E+LM9EKc tM8tZHEn0flwwU96k6MnsC1KxbGEdAC/MpKBc= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:sender:in-reply-to:references:date :x-google-sender-auth:message-id:subject:from:to:content-type; b=sEhGWytnJVAnHpr7lqlAEcIc20GS8KhmrjFg6wSGeptO0enAILI0KYVz6fJuZNafS+ sLoX+dmJQFG4xI117fBDtKBcT6Ey7WjrjWfDM7khdW3DOWH6U+ICMlUgkiC2vhjNDvWO mSkEISKab6oFTaxZABQHHPaYaKxAacbxOoA0k= MIME-Version: 1.0 Sender: asmrookie@gmail.com Received: by 10.223.95.72 with SMTP id c8mr974653fan.73.1257699290738; Sun, 08 Nov 2009 08:54:50 -0800 (PST) In-Reply-To: <3bbf2fe10911080853qad0057w490694e5627681fe@mail.gmail.com> References: <3bbf2fe10911080853qad0057w490694e5627681fe@mail.gmail.com> Date: Sun, 8 Nov 2009 17:54:50 +0100 X-Google-Sender-Auth: d2c01a2450da8262 Message-ID: <3bbf2fe10911080854x64d18e40hf639634e625dd11b@mail.gmail.com> From: Attilio Rao To: freebsd-new-bus@freebsd.org, John Baldwin , Scott Long , Warner Losh Content-Type: text/plain; charset=UTF-8 Cc: Subject: Re: [PATCH] Collapsing device_state_t with devinfo_state_t X-BeenThere: freebsd-new-bus@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: FreeBSD's new-bus architecture List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 08 Nov 2009 16:54:53 -0000 2009/11/8 Attilio Rao : > This patch should collpase device_state_t with devinfo_state_t and > remove a bogus replication of this struct: > http://www.freebsd.org/~attilio/devinfo.diff Sorry, forgot to mention: I thought about adding _bus.h interface because I thought that devinfo.h wanted to avoid a namespace pollution, but that's not really the case as 85% of the bus.h is already under _KERNEL labels. That means the pollution is minimal and restricted to 1-2 further structs. Attilio -- Peace can only be achieved by understanding - A. Einstein From owner-freebsd-new-bus@FreeBSD.ORG Mon Nov 9 01:34:11 2009 Return-Path: Delivered-To: freebsd-new-bus@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id CA3C8106566B; Mon, 9 Nov 2009 01:34:11 +0000 (UTC) (envelope-from imp@bsdimp.com) Received: from harmony.bsdimp.com (bsdimp.com [199.45.160.85]) by mx1.freebsd.org (Postfix) with ESMTP id 81BD88FC08; Mon, 9 Nov 2009 01:34:11 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by harmony.bsdimp.com (8.14.3/8.14.1) with ESMTP id nA91Pnxa066532; Sun, 8 Nov 2009 18:25:49 -0700 (MST) (envelope-from imp@bsdimp.com) Date: Sun, 08 Nov 2009 18:25:56 -0700 (MST) Message-Id: <20091108.182556.1680365520.imp@bsdimp.com> To: attilio@freebsd.org From: "M. Warner Losh" In-Reply-To: <3bbf2fe10911080854x64d18e40hf639634e625dd11b@mail.gmail.com> References: <3bbf2fe10911080853qad0057w490694e5627681fe@mail.gmail.com> <3bbf2fe10911080854x64d18e40hf639634e625dd11b@mail.gmail.com> X-Mailer: Mew version 5.2 on Emacs 21.3 / Mule 5.0 (SAKAKI) Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: freebsd-new-bus@freebsd.org, scottl@freebsd.org Subject: Re: [PATCH] Collapsing device_state_t with devinfo_state_t X-BeenThere: freebsd-new-bus@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: FreeBSD's new-bus architecture List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 09 Nov 2009 01:34:11 -0000 In message: <3bbf2fe10911080854x64d18e40hf639634e625dd11b@mail.gmail.com> Attilio Rao writes: : 2009/11/8 Attilio Rao : : > This patch should collpase device_state_t with devinfo_state_t and : > remove a bogus replication of this struct: : > http://www.freebsd.org/~attilio/devinfo.diff : : Sorry, forgot to mention: : I thought about adding _bus.h interface because I thought that : devinfo.h wanted to avoid a namespace pollution, but that's not really : the case as 85% of the bus.h is already under _KERNEL labels. That : means the pollution is minimal and restricted to 1-2 further structs. I'm cool with this patch, and that decision. Warner From owner-freebsd-new-bus@FreeBSD.ORG Mon Nov 9 02:02:23 2009 Return-Path: Delivered-To: freebsd-new-bus@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id B5EC0106566B; Mon, 9 Nov 2009 02:02:23 +0000 (UTC) (envelope-from asmrookie@gmail.com) Received: from mail-bw0-f213.google.com (mail-bw0-f213.google.com [209.85.218.213]) by mx1.freebsd.org (Postfix) with ESMTP id B1EC78FC08; Mon, 9 Nov 2009 02:02:22 +0000 (UTC) Received: by bwz5 with SMTP id 5so3113560bwz.3 for ; Sun, 08 Nov 2009 18:02:21 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:sender:received:in-reply-to :references:date:x-google-sender-auth:message-id:subject:from:to:cc :content-type; bh=MOH4TrI6imcSEhRPpd6aSrZshFn+3lKohUOn/isqCZo=; b=uK+PumOgWAupJKAs59ckYKhPKIQhB6DuPWDkxv2iTSAPr2EeaEZRxgOL88IlFBnzbk 24GwbfjVAp8IeB9hi/Cvz0i7QvEOQA27JRFgBgmVbLRZ+IzDLf1fnQDeNXWKIYyvQN0P gRCzXxD1D0CXhUcs8Ote3ekGeA9re8SpLSRDg= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:sender:in-reply-to:references:date :x-google-sender-auth:message-id:subject:from:to:cc:content-type; b=lxmHNT8Dhak8Oe6YPztNdgM6YPAesJ/RehmXq8stIaIQ8UejHfnjn/yXCxfHE6hR1B Fz/UyFqgSysBbGjQSMzvPsK14+AqkIGPxO5sqZUgNij7Zgd9jgwW8xi1epDJth/uHX6a fCxcNTJ/Wc57LTyTII2y0ei7y67kd0yhZraro= MIME-Version: 1.0 Sender: asmrookie@gmail.com Received: by 10.223.144.82 with SMTP id y18mr1066661fau.74.1257732141521; Sun, 08 Nov 2009 18:02:21 -0800 (PST) In-Reply-To: <3F7C0F44-7E90-4196-8CFA-724E1297B2A6@samsco.org> References: <3bbf2fe10911080853qad0057w490694e5627681fe@mail.gmail.com> <3F7C0F44-7E90-4196-8CFA-724E1297B2A6@samsco.org> Date: Mon, 9 Nov 2009 03:02:21 +0100 X-Google-Sender-Auth: 09302e11feda207f Message-ID: <3bbf2fe10911081802q2b691218hd2a2736fefcb232c@mail.gmail.com> From: Attilio Rao To: Scott Long Content-Type: text/plain; charset=UTF-8 Cc: Warner Losh , freebsd-new-bus@freebsd.org, Scott Long Subject: Re: [PATCH] Collapsing device_state_t with devinfo_state_t X-BeenThere: freebsd-new-bus@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: FreeBSD's new-bus architecture List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 09 Nov 2009 02:02:23 -0000 2009/11/9 Scott Long : > On Nov 8, 2009, at 9:53 AM, Attilio Rao wrote: >> >> This patch should collpase device_state_t with devinfo_state_t and >> remove a bogus replication of this struct: >> http://www.freebsd.org/~attilio/devinfo.diff >> >> This patch breaks ABI so it cannot be MFC'ed (and it is not intended to >> do). >> > > Does this hinder FreeBSD 8 from ever getting a Giant-free newbus? As foretold in several e-mails and threads, FreeBSD-8 has all the needed support for Giant-free newbus, that's just an improvement I couldn't let happen before because we had to maintain ABI stability over the release process, but it doesn't compromise at all possibility to MFC Giant-free newbus. Attilio -- Peace can only be achieved by understanding - A. Einstein From owner-freebsd-new-bus@FreeBSD.ORG Mon Nov 9 02:12:00 2009 Return-Path: Delivered-To: freebsd-new-bus@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 1CE56106566B for ; Mon, 9 Nov 2009 02:12:00 +0000 (UTC) (envelope-from scottl@samsco.org) Received: from pooker.samsco.org (pooker.samsco.org [168.103.85.57]) by mx1.freebsd.org (Postfix) with ESMTP id C9DFA8FC16 for ; Mon, 9 Nov 2009 02:11:59 +0000 (UTC) Received: from [IPv6:::1] (pooker.samsco.org [168.103.85.57]) (authenticated bits=0) by pooker.samsco.org (8.14.2/8.14.2) with ESMTP id nA91emNd074777; Sun, 8 Nov 2009 18:40:48 -0700 (MST) (envelope-from scottl@samsco.org) Mime-Version: 1.0 (Apple Message framework v1076) Content-Type: text/plain; charset=us-ascii; format=flowed; delsp=yes From: Scott Long In-Reply-To: <3bbf2fe10911080853qad0057w490694e5627681fe@mail.gmail.com> Date: Sun, 8 Nov 2009 18:40:48 -0700 Content-Transfer-Encoding: 7bit Message-Id: <3F7C0F44-7E90-4196-8CFA-724E1297B2A6@samsco.org> References: <3bbf2fe10911080853qad0057w490694e5627681fe@mail.gmail.com> To: Attilio Rao X-Mailer: Apple Mail (2.1076) X-Spam-Status: No, score=-4.3 required=3.8 tests=ALL_TRUSTED,AWL,BAYES_00 autolearn=ham version=3.1.8 X-Spam-Checker-Version: SpamAssassin 3.1.8 (2007-02-13) on pooker.samsco.org Cc: Warner Losh , freebsd-new-bus@freebsd.org, Scott Long Subject: Re: [PATCH] Collapsing device_state_t with devinfo_state_t X-BeenThere: freebsd-new-bus@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: FreeBSD's new-bus architecture List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 09 Nov 2009 02:12:00 -0000 On Nov 8, 2009, at 9:53 AM, Attilio Rao wrote: > This patch should collpase device_state_t with devinfo_state_t and > remove a bogus replication of this struct: > http://www.freebsd.org/~attilio/devinfo.diff > > This patch breaks ABI so it cannot be MFC'ed (and it is not intended > to do). > Does this hinder FreeBSD 8 from ever getting a Giant-free newbus? Scott