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>

index | next in thread | previous in thread | raw e-mail

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


home | help

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