public inbox for linux-acpi@vger.kernel.org
 help / color / mirror / Atom feed
From: Lin Ming <ming.m.lin@intel.com>
To: "Kleen, Andi" <andi.kleen@intel.com>,
	"Moore, Robert" <robert.moore@intel.com>
Cc: linux-acpi <linux-acpi@vger.kernel.org>,
	"Brown, Len" <len.brown@intel.com>
Subject: RE: ACPICA patch set, release 20080701 & 20080729
Date: Mon, 11 Aug 2008 10:03:49 +0800	[thread overview]
Message-ID: <1218420229.22703.8.camel@minggr.sh.intel.com> (raw)
In-Reply-To: <9D39833986E69849A2A8E74C1078B6B3CE13FE@orsmsx415.amr.corp.intel.com>

[-- Attachment #1: Type: text/plain, Size: 4243 bytes --]

On Fri, 2008-08-08 at 05:50 +0800, Moore, Robert wrote:
> Here's the patchlist:
> 
> ACPICA: Cleanup macro definition file
> ACPICA: Copy dynamically loaded tables to local buffer
> ACPICA: Fix warning for 64-bit build
> ACPICA: Add check for invalid handle in acpi_get_object_info
> ACPICA: Fix memory leak when deleting thermal/processor objects
> ACPICA: Allow same ACPI table to be loaded/unloaded more than once
> ACPICA: Fix possible memory leak in Unload() operator
> ACPICA: Return method arg count from acpi_get_object_info
> ACPICA: Fix wrong resource descriptor length for 64-bit build
> ACPICA: Fix table compare code, length then data
> ACPICA: Update version to 20080701
> ACPICA: Return status from global init function
> ACPICA: Add function to decode reference obj types to strings
> ACPICA: Improve object conversion error messages
> ACPICA: x2APIC support: changes for MADT and SRAT ACPI tables
> ACPICA: Add function to dereference returned reference objects
> ACPICA: Additional error checking for pathname utilities
> ACPICA: Update version to 20080729
> 
> 
> Most of these are bug fixes, new stuff would be the following:
> 
> ACPICA: Cleanup macro definition file
> ACPICA: Return method arg count from acpi_get_object_info
> ACPICA: Add function to decode reference obj types to strings
> ACPICA: Improve object conversion error messages
> ACPICA: x2APIC support: changes for MADT and SRAT ACPI tables
> 
> 
> Leaving these as the bug fixes that should be integrated:
> 
> ACPICA: Copy dynamically loaded tables to local buffer
> ACPICA: Fix warning for 64-bit build
> ACPICA: Add check for invalid handle in acpi_get_object_info
> ACPICA: Fix memory leak when deleting thermal/processor objects
> ACPICA: Allow same ACPI table to be loaded/unloaded more than once
> ACPICA: Fix possible memory leak in Unload() operator
> ACPICA: Fix wrong resource descriptor length for 64-bit build
> ACPICA: Fix table compare code, length then data
> ACPICA: Return status from global init function
> ACPICA: Add function to dereference returned reference objects
> ACPICA: Additional error checking for pathname utilities

Andi, 

The attachment is the bug fixes as Bob listed above.
It can be applied to your linux-acpi-2.6 tree(test branch).

Lin Ming

> 
> 
> I'm not sure what you should do with the version numbers, I don't think
> you should advertise a certain version of ACPICA until it has been
> *fully* integrated.
> 
> ACPICA: Update version to 20080701
> ACPICA: Update version to 20080729
> 
> 
> >-----Original Message-----
> >From: Kleen, Andi
> >Sent: Thursday, August 07, 2008 2:43 PM
> >To: Moore, Robert; Lin, Ming M
> >Cc: 'linux-acpi'; Brown, Len
> >Subject: RE: ACPICA patch set, release 20080701 & 20080729
> >
> >
> >
> >
> >>-----Original Message-----
> >>From: Moore, Robert
> >>Sent: Thursday, August 07, 2008 11:31 PM
> >>To: Kleen, Andi; Lin, Ming M
> >>Cc: 'linux-acpi'; Brown, Len
> >>Subject: RE: ACPICA patch set, release 20080701 & 20080729
> >>
> >>>And it's far too late for feature patches at this point. Can you
> >>>extract only the bug fixes and non intrusive cleanups?
> >>
> >>What does this mean exactly?
> >
> >The Linux kernel has this concept of merging windows. After a major
> release
> >there is a 2 week merging window where all the features go in. The
> merge
> >window closes with -rc1. After -rc1 some features can still go in, but
> in
> >general larger merges are frowned upon (as in Linus often yells at the
> >submitter) The focus is definitely on bug fixes post -rc1. Also
> >there seem to be unfortunately quite a lot of ACPI regressions this
> cycle,
> >so I have to be especially conservative.
> >
> >A full 2 cycle ACPICA merge would be likely too much
> >of a merge.  Also we need some testing time for major ACPICA changes
> >in -mm/linux-next anyways.
> >
> >I'm open for relatively clear bug fixes and unlikely to break anything
> >cleanups though to push in for .27. E.g. the pathname fixes would be
> >obvious candidates. Since you know the various patches better than me
> >it makes sense for you to preselect.
> >
> >I can pull full ACPICA into the test tree of course (assuming
> >it applies). Will do that tomorrow. But test is beyond .27.
> >
> >-Andi
> 

[-- Attachment #2: acpica.fixes.tar.gz --]
[-- Type: application/x-compressed-tar, Size: 12082 bytes --]

      reply	other threads:[~2008-08-11  2:04 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-08-04  8:15 ACPICA patch set, release 20080701 & 20080729 Lin Ming
2008-08-07 21:10 ` Kleen, Andi
2008-08-07 21:31   ` Moore, Robert
2008-08-07 21:42     ` Kleen, Andi
2008-08-07 21:50       ` Moore, Robert
2008-08-11  2:03         ` Lin Ming [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1218420229.22703.8.camel@minggr.sh.intel.com \
    --to=ming.m.lin@intel.com \
    --cc=andi.kleen@intel.com \
    --cc=len.brown@intel.com \
    --cc=linux-acpi@vger.kernel.org \
    --cc=robert.moore@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox