ASLTS: Serialize all methods that create named objects.

This change greatly reduces the remark that iASL emits when a
non-serialized control method creates named objects. David Box.

3 files changed, 42 insertions, 42 deletions

diff --git a/tests/aslts/src/runtime/collections/functional/table/load.asl b/tests/aslts/src/runtime/collections/functional/table/load.asl
index 729bb7c87..357ed8c79 100644
--- a/tests/aslts/src/runtime/collections/functional/table/load.asl
+++ b/tests/aslts/src/runtime/collections/functional/table/load.asl
@@ -163,7 +163,7 @@ Device(DTM0) {
 	}
 
 	// components/utilities/utmisc.c AcpiUtGenerateChecksum() analog
-	Method(CHSM, 2)	// buf, len
+	Method(CHSM, 2, Serialized)	// buf, len
 	{
 		Name(lpN0, 0)
 		Name(lpC0, 0)
@@ -219,7 +219,7 @@ Device(DTM0) {
 	}
 
 	// Prepares and Loads the next Table of multiple Tables Load test
-	Method(LD)
+	Method(LD,, Serialized)
 	{
 		if (LNot(LLess(HI0N, HI0M))) {
 			Store("LD: too many tables loaded", Debug)
@@ -523,7 +523,7 @@ Device(DTM0) {
 	}
 
 	// DDBHandle storing into an Object by Reference in Argx
-	Method(tst3, 1)
+	Method(tst3, 1, Serialized)
 	{
 		Name(HI0, 0)
 
@@ -704,7 +704,7 @@ Device(DTM0) {
 		// Auxiliary method for ArgX, part1
 		// Arg1 - reference to store the DDBHandle
 		// Arg2 - OpRegion Object of a specified type
-		Method(m001, 3)
+		Method(m001, 3, Serialized)
 		{
 			OperationRegion(OPRm, 0xff, 0, 0x1000)
 
@@ -761,7 +761,7 @@ Device(DTM0) {
 		}
 
 		// Arg1 - OpRegion Object of a specified type
-		Method(m003, 2)
+		Method(m003, 2, Serialized)
 		{
 			Concatenate(arg0, "-m003", arg0)
 
@@ -886,7 +886,7 @@ Device(DTM0) {
 		RFU1, 0x2f8,
 	}
 
-	Method(tst8, 1)
+	Method(tst8, 1, Serialized)
 	{
 		Name(DDBH, 0)
 
@@ -1002,7 +1002,7 @@ Device(DTM0) {
 
 	External(\AUXD.M000, MethodObj)
 
-	Method(tst9, 1)
+	Method(tst9, 1, Serialized)
 	{
 		Name(DDBH, 0)
 
@@ -1243,7 +1243,7 @@ Device(DTM0) {
 		RFU3, 0x8f8,
 	}
 
-	Method(tstb, 1)
+	Method(tstb, 1, Serialized)
 	{
 		Name(DDB0, 0)
 		Name(DDBH, 0)
@@ -1383,7 +1383,7 @@ Device(DTM0) {
 
 	// Exceptions when an OpRegion passed as the Object
 	// parameter of Load is not of SystemMemory type
-	Method(tstc, 1)
+	Method(tstc, 1, Serialized)
 	{
 		Name(DDBH, 0)
 
@@ -1482,7 +1482,7 @@ Device(DTM0) {
 
 	// Exceptions when the table contained in an OpRegion
 	// (Field) is not an SSDT
-	Method(tstd, 1)
+	Method(tstd, 1, Serialized)
 	{
 		Name(HI0, 0)
 
@@ -1534,7 +1534,7 @@ Device(DTM0) {
 	// than the length of the respective OpRegion or Region Field,
 	// or less than the length of the Table Header
 	// Arg1: 0 - the 'greater' case, 1 - the 'less' case
-	Method(tste, 2)
+	Method(tste, 2, Serialized)
 	{
 		Name(HI0, 0)
 
@@ -1614,7 +1614,7 @@ Device(DTM0) {
 	}
 
 	// Exceptions when the checksum of the supplied SSDT is invalid
-	Method(tstf, 1)
+	Method(tstf, 1, Serialized)
 	{
 		Name(HI0, 0)
 
@@ -1676,7 +1676,7 @@ Device(DTM0) {
 	}
 
 	// Object of any type can be used as the DDBHandle argument
-	Method(tstg, 1)
+	Method(tstg, 1, Serialized)
 	{
 		Name(DDB0, 0)
 		Name(DDB1, 0)
@@ -1801,7 +1801,7 @@ Device(DTM0) {
 
 	// AE_OWNER_ID_LIMIT exception when too many Tables loaded,
 	// Arg1: 0 - Load case, 1 - LoadTable case
-	Method(tsth, 2)
+	Method(tsth, 2, Serialized)
 	{
 		Name(MAXT, 0xf6)
 		Name(DDB1, 0)
@@ -1870,7 +1870,7 @@ Device(DTM0) {
 
 	// Exception when SSDT specified as the Object parameter
 	// of the Load operator is already loaded
-	Method(tsti, 1)
+	Method(tsti, 1, Serialized)
 	{
 		Name(HI0, 0)
 		Name(HI1, 0)
@@ -1941,7 +1941,7 @@ Device(DTM0) {
 
 	// Exception when there already is an previously created Object
 	// referred by the namepath of the new Object in the Table loaded
-	Method(tstj, 1)
+	Method(tstj, 1, Serialized)
 	{
 		Name(HI0, 0)
 		Name(HI1, 0)
@@ -2044,7 +2044,7 @@ Device(DTM0) {
 	}
 }
 
-Method(TLD0)
+Method(TLD0,, Serialized)
 {
 	Name(ts, "TLD0")
 
@@ -2137,7 +2137,7 @@ Method(TLD0)
 }
 
 // Exceptional conditions
-Method(TLD1)
+Method(TLD1,, Serialized)
 {
 	Name(ts, "TLD1")
 
diff --git a/tests/aslts/src/runtime/collections/functional/table/loadtable.asl b/tests/aslts/src/runtime/collections/functional/table/loadtable.asl
index 32ab10980..2d2a112cb 100644
--- a/tests/aslts/src/runtime/collections/functional/table/loadtable.asl
+++ b/tests/aslts/src/runtime/collections/functional/table/loadtable.asl
@@ -147,7 +147,7 @@ Device(DTM2) {
 	Name(DDBH, 0)
 
 	// Check DataTable Region
-	Method(chdr, 1)
+	Method(chdr, 1, Serialized)
 	{
 		DataTableRegion (DR00, "OEM1", "", "")
 		Field(DR00, AnyAcc, NoLock, Preserve) {
@@ -164,7 +164,7 @@ Device(DTM2) {
 	}
 
 	// Simple Loadtable test
-	Method(tst0, 1)
+	Method(tst0, 1, Serialized)
 	{
 		Name(DDBH, 0)
 
@@ -220,7 +220,7 @@ Device(DTM2) {
 
 	// All comparisons of Loadtable parameters are case sensitive,
 	// if no table matches the specified parameters, then 0 is returned
-	Method(tst1, 1)
+	Method(tst1, 1, Serialized)
 	{
 		Name(DDBH, 0)
 
@@ -374,7 +374,7 @@ Device(DTM2) {
 
 	// Any of the RootPathString, ParameterPathString, and ParameterData
 	// parameters in LoadTable expression can be omitted
-	Method(tst2, 1)
+	Method(tst2, 1, Serialized)
 	{
 		Name(DDB0, 0)
 		Name(DDB1, 0)
@@ -545,7 +545,7 @@ Device(DTM2) {
 
 	// Different sources of the String parameters: Named Objects, LocalX,
 	// ArgX, elements of Packages, results of functions, any TermArg
-	Method(tst3, 1)
+	Method(tst3, 1, Serialized)
 	{
 		Name(DDB0, 0)
 		Name(DDB1, 0)
@@ -732,7 +732,7 @@ Device(DTM2) {
 	// Different sources of the optional parameters (RootPathString,
 	// ParameterPathString, and ParameterData): Named Objects, LocalX,
 	// ArgX, elements of Packages, results of functions, any TermArg
-	Method(tst4, 1)
+	Method(tst4, 1, Serialized)
 	{
 		Name(DDB0, 0)
 		Name(DDB1, 0)
@@ -930,7 +930,7 @@ Device(DTM2) {
 	// Namespace location to load the Definition Block is determined
 	// by the RootPathString parameter of Loadtable
 	// Arg1: RootPathString
-	Method(tst5, 2)
+	Method(tst5, 2, Serialized)
 	{
 		Name(DDBH, 0)
 
@@ -995,7 +995,7 @@ Device(DTM2) {
 
 	// "\" is assumed to be Namespace location to load the Definition
 	// Block if RootPathString parameter is not specified
-	Method(tst6, 1)
+	Method(tst6, 1, Serialized)
 	{
 		Name(DDBH, 0)
 
@@ -1049,7 +1049,7 @@ Device(DTM2) {
 	// or caret character, then the path of the object set up on success
 	// is ParameterPathString. It is RootPathString.ParameterPathString
 	// in any case.
-	Method(tst7, 1)
+	Method(tst7, 1, Serialized)
 	{
 		Name(DDBH, 0)
 		Name(PLDT, 0)
@@ -1117,7 +1117,7 @@ Device(DTM2) {
 	// Exceptions when the SignatureString is greater than four characters,
 	// the OEMIDString is greater than six characters, or the OEMTableID is
 	// greater than eight characters
-	Method(tst8, 1)
+	Method(tst8, 1, Serialized)
 	{
 		Name(DDBH, 0)
 
@@ -1199,7 +1199,7 @@ Device(DTM2) {
 	}
 
 	// Exceptions when the matched table is already loaded
-	Method(tsta, 1)
+	Method(tsta, 1, Serialized)
 	{
 		Name(DDBH, 0)
 
@@ -1289,7 +1289,7 @@ Device(DTM2) {
 
 	// Exceptions when there already is an previously loaded Object
 	// referred by the path in the Namespace
-	Method(tstb, 1)
+	Method(tstb, 1, Serialized)
 	{
 		Name(DDBH, 0)
 
@@ -1386,7 +1386,7 @@ Device(DTM2) {
 	}
 
 	// Implicit operand conversion of the parameters specified to be strings
-	Method(tste, 1)
+	Method(tste, 1, Serialized)
 	{
 		Name(DDBH, 2)
 		Name(SOID, "")
@@ -1659,7 +1659,7 @@ Device(DTM2) {
 
 	// Exceptions when the parameter of the Loadtable operator
 	// is of incorrect types
-	Method(tsth, 1)
+	Method(tsth, 1, Serialized)
 	{
 		Name(DDB0, 0)
 		Name(DDB1, 0)
@@ -1934,7 +1934,7 @@ Device(DTM2) {
 
 	// Exceptions when the ParameterData parameter of the Loadtable operator
 	// can not be saved into the Object referred by ParameterPathString
-	Method(tsti, 1)
+	Method(tsti, 1, Serialized)
 	{
 		Name(DDB0, 0)
 		Name(DDB1, 0)
@@ -2198,7 +2198,7 @@ Device(DTM2) {
 	}
 }
 
-Method(TLT0)
+Method(TLT0,, Serialized)
 {
 	Name(ts, "TLT0")
 
@@ -2285,7 +2285,7 @@ Method(TLT0)
 }
 
 // Exceptional conditions
-Method(TLT1)
+Method(TLT1,, Serialized)
 {
 	Name(ts, "TLT1")
 
diff --git a/tests/aslts/src/runtime/collections/functional/table/unload.asl b/tests/aslts/src/runtime/collections/functional/table/unload.asl
index a68495ba7..e367c0c1b 100644
--- a/tests/aslts/src/runtime/collections/functional/table/unload.asl
+++ b/tests/aslts/src/runtime/collections/functional/table/unload.asl
@@ -84,7 +84,7 @@ Device(DTM1) {
 	// are checked in load.asl
 
 	// DDBHandle returned by Method call
-	Method(tst0, 1)
+	Method(tst0, 1, Serialized)
 	{
 		Name(HI0, 0)
 
@@ -162,7 +162,7 @@ Device(DTM1) {
 
 	// All namespace objects created as a result of the corresponding
 	// Load operation are absent in the namespace after UnLoad
-	Method(tst1, 1)
+	Method(tst1, 1, Serialized)
 	{
 		Name(DDB0, 0)
 		Name(DDB1, 0)
@@ -418,7 +418,7 @@ Device(DTM1) {
 
 	// Exceptions when the parameter of the UnLoad operator
 	// is not of DDBHandle type
-	Method(tst3, 1)
+	Method(tst3, 1, Serialized)
 	{
 		Name(DDB0, 0)
 		Name(DDB1, 0)
@@ -539,7 +539,7 @@ Device(DTM1) {
 	}
 
 	// Exceptions when UnLoad is executed with the same DDBHandle repeatedly
-	Method(tst4, 1)
+	Method(tst4, 1, Serialized)
 	{
 		Name(DDB0, 0)
 		Name(DDB1, 0)
@@ -601,7 +601,7 @@ Device(DTM1) {
 	}
 
 	// Exceptions when the operand of UnLoad operator is absent
-	Method(tst5, 1)
+	Method(tst5, 1, Serialized)
 	{
 		Name(DDB0, 0)
 
@@ -646,7 +646,7 @@ Device(DTM1) {
 	}
 }
 
-Method(TUL0)
+Method(TUL0,, Serialized)
 {
 	Name(ts, "TUL0")
 
@@ -676,7 +676,7 @@ Method(TUL0)
 }
 
 // Exceptional conditions
-Method(TUL1)
+Method(TUL1,, Serialized)
 {
 	Name(ts, "TUL1")