Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 25 Aug 2004 11:53:19 -0600
From:      Scott Long <scottl@freebsd.org>
To:        Nate Lawson <nate@root.org>
Cc:        obrien@freebsd.org
Subject:   Re: cvs commit: src/sys/dev/acpica acpi.c acpi_resource.c acpivar.h
Message-ID:  <412CD20F.2090708@freebsd.org>
In-Reply-To: <412BC4C5.3000708@root.org>
References:  <200408231628.i7NGSg8G084516@repoman.freebsd.org> <20040824180620.GA56576@dragon.nuxi.com> <412B9629.3080108@root.org> <20040824213627.GA61527@dragon.nuxi.com> <412BC4C5.3000708@root.org>

next in thread | previous in thread | raw e-mail | index | archive | help
Nate Lawson wrote:
> David O'Brien wrote:
> 
>> On Tue, Aug 24, 2004 at 12:25:29PM -0700, Nate Lawson wrote:
>>
>>> David O'Brien wrote:
>>>
>>>> On Mon, Aug 23, 2004 at 04:28:42PM +0000, Nate Lawson wrote:
>>>>
>>>>
>>>>> njl         2004-08-23 16:28:42 UTC
>>>>> FreeBSD src repository
>>>>> Modified files:
>>>>>  sys/dev/acpica       acpi.c acpi_resource.c acpivar.h Log:
>>>>> Rework sysresource management.  Instead of having each sysresource 
>>>>> object
>>>>
>>>>
>>>> ...
>>>>
>>>>
>>>>> Tested by:      Pawel Worach <sajd_at_telia.com>
>>>>> Tested by:      Radek Kozlowski <radek_at_raadradd.com>
>>>>
>>>>
>>>>
>>>> Tested only on i386, right??
>>>
>>>
>>> Still waiting for the amd64 laptop donation.  ;-)  In any case, I've 
>>> just committed a fix.  The amd64,ia64 nexus hadn't been caught up 
>>> with i386 and the resource code depends on the new method.
>>
>>
>> In the future, may I make a request that you post patches for large
>> changes before committing them?  Now that ACPI is a multi-platform thing
>> that is very basic to booting, we have to be careful about changes to it.
> 
> 
> For large commits (i.e. mpsafe patch), I post patches and get testing. 
> For critical bugfixes like this, I get the reporter to test after I test 
> and then commit.  With the minimum MFC period, I want to get critical 
> bugfixes into HEAD as soon as they are tested (to start the clock) and 
> plan to be responsive to any subsequent bug reports.
> 

I'm not going to harp on this other than to say that commits to RELENG_5
need to have adequate review and testing before they are submitted for
approval.  I'll personally test anything that you need on amd64 if you
ask.

Scott



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?412CD20F.2090708>