Commit ea936b78 authored by Bob Moore's avatar Bob Moore Committed by Len Brown
Browse files

ACPI: ACPICA 20060217 



Implemented a change to the IndexField support to match
the behavior of the Microsoft AML interpreter. The value
written to the Index register is now a byte offset,
no longer an index based upon the width of the Data
register. This should fix IndexField problems seen on
some machines where the Data register is not exactly one
byte wide. The ACPI specification will be clarified on
this point.

Fixed a problem where several resource descriptor
types could overrun the internal descriptor buffer due
to size miscalculation: VendorShort, VendorLong, and
Interrupt. This was noticed on IA64 machines, but could
affect all platforms.

Fixed a problem where individual resource descriptors were
misaligned within the internal buffer, causing alignment
faults on IA64 platforms.

Signed-off-by: default avatarBob Moore <robert.moore@intel.com>
Signed-off-by: default avatarLen Brown <len.brown@intel.com>
parent 52fc0b02

drivers/acpi/executer/exprep.c

+12 −5
Original line number Diff line number Diff line
@@ -519,13 +519,20 @@ acpi_status acpi_ex_prep_field_value(struct acpi_create_field_info *info) 
		acpi_ut_add_reference(obj_desc->index_field.index_obj);



		/*

		 * February 2006: Changed to match MS behavior

		 *

		 * The value written to the Index register is the byte offset of the

		 * target field

		 * Note: may change code to: ACPI_DIV_8 (Info->field_bit_position)

		 * target field.

		 *

		 * Previously, the value was calculated as an index in terms of the

		 * width of the Data register, as below:

		 *

		 *   obj_desc->index_field.Value = (u32)

		 *       (Info->field_bit_position / ACPI_MUL_8 (

		 *           obj_desc->Field.access_byte_width));

		 */

		obj_desc->index_field.value = (u32)

		    (info->field_bit_position /

		     ACPI_MUL_8(obj_desc->field.access_byte_width));

		obj_desc->index_field.value =

		    (u32) ACPI_DIV_8(info->field_bit_position);



		ACPI_DEBUG_PRINT((ACPI_DB_BFIELD,

				  "index_field: bit_off %X, Off %X, Value %X, Gran %X, Index %p, Data %p\n",

drivers/acpi/namespace/nsutils.c

+3 −2
Original line number Diff line number Diff line
@@ -78,6 +78,7 @@ acpi_ns_report_error(char *module_name, 
		     char *internal_name, acpi_status lookup_status)

{

	acpi_status status;

	u32 bad_name;

	char *name = NULL;



	acpi_ut_report_error(module_name, line_number);
@@ -86,8 +87,8 @@ acpi_ns_report_error(char *module_name, 


		/* There is a non-ascii character in the name */



		acpi_os_printf("[0x%4.4X] (NON-ASCII)",

			       *(ACPI_CAST_PTR(u32, internal_name)));

		ACPI_MOVE_32_TO_32(&bad_name, internal_name);

		acpi_os_printf("[0x%4.4X] (NON-ASCII)", bad_name);

	} else {

		/* Convert path to external format */

drivers/acpi/parser/psloop.c

+1 −0
Original line number Diff line number Diff line
@@ -767,6 +767,7 @@ acpi_status acpi_ps_parse_loop(struct acpi_walk_state *walk_state) 
						return_ACPI_STATUS(status2);

					}

				}



				acpi_ps_pop_scope(parser_state, &op,

						  &walk_state->arg_types,

						  &walk_state->arg_count);

drivers/acpi/resources/rscalc.c

+45 −55
Original line number Diff line number Diff line
@@ -162,9 +162,11 @@ acpi_rs_stream_option_length(u32 resource_length, 
		    resource_length - minimum_aml_resource_length - 1;

	}



	/* Round up length to 32 bits for internal structure alignment */



	return ((u32) ACPI_ROUND_UP_to_32_bITS(string_length));

	/*

	 * Round the length up to a multiple of the native word in order to

	 * guarantee that the entire resource descriptor is native word aligned

	 */

	return ((u32) ACPI_ROUND_UP_TO_NATIVE_WORD(string_length));

}



/*******************************************************************************
@@ -336,7 +338,7 @@ acpi_rs_get_list_length(u8 * aml_buffer, 
	acpi_status status;

	u8 *end_aml;

	u8 *buffer;

	u32 buffer_size = 0;

	u32 buffer_size;

	u16 temp16;

	u16 resource_length;

	u32 extra_struct_bytes;
@@ -345,6 +347,7 @@ acpi_rs_get_list_length(u8 * aml_buffer, 


	ACPI_FUNCTION_TRACE("rs_get_list_length");



	*size_needed = 0;

	end_aml = aml_buffer + aml_buffer_length;



	/* Walk the list of AML resource descriptors */
@@ -391,37 +394,28 @@ acpi_rs_get_list_length(u8 * aml_buffer, 
			break;



		case ACPI_RESOURCE_NAME_VENDOR_SMALL:

		case ACPI_RESOURCE_NAME_VENDOR_LARGE:

			/*

			 * Vendor Resource:

			 * Ensure a 32-bit boundary for the structure

			 * Get the number of vendor data bytes

			 */

			extra_struct_bytes = (u32)

			    ACPI_ROUND_UP_to_32_bITS(resource_length) -

			    resource_length;

			extra_struct_bytes = resource_length;

			break;



		case ACPI_RESOURCE_NAME_END_TAG:

			/*

			 * End Tag: This is the normal exit, add size of end_tag

			 * End Tag:

			 * This is the normal exit, add size of end_tag

			 */

			*size_needed = buffer_size + ACPI_RS_SIZE_MIN;

			*size_needed += ACPI_RS_SIZE_MIN;

			return_ACPI_STATUS(AE_OK);



		case ACPI_RESOURCE_NAME_VENDOR_LARGE:

			/*

			 * Vendor Resource:

			 * Add vendor data and ensure a 32-bit boundary for the structure

			 */

			extra_struct_bytes = (u32)

			    ACPI_ROUND_UP_to_32_bITS(resource_length) -

			    resource_length;

			break;



		case ACPI_RESOURCE_NAME_ADDRESS32:

		case ACPI_RESOURCE_NAME_ADDRESS16:

		case ACPI_RESOURCE_NAME_ADDRESS64:

			/*

			 * 32-Bit or 16-bit Address Resource:

			 * Add the size of any optional data (resource_source)

			 * Address Resource:

			 * Add the size of the optional resource_source

			 */

			extra_struct_bytes =

			    acpi_rs_stream_option_length(resource_length,
@@ -430,50 +424,46 @@ acpi_rs_get_list_length(u8 * aml_buffer, 


		case ACPI_RESOURCE_NAME_EXTENDED_IRQ:

			/*

			 * Extended IRQ:

			 * Point past the interrupt_vector_flags to get the

			 * interrupt_table_length.

			 * Extended IRQ Resource:

			 * Using the interrupt_table_length, add 4 bytes for each additional

			 * interrupt. Note: at least one interrupt is required and is

			 * included in the minimum descriptor size (reason for the -1)

			 */

			buffer++;

			extra_struct_bytes = (buffer[1] - 1) * sizeof(u32);



			extra_struct_bytes = (u32)

			    /*

			     * Add 4 bytes for each additional interrupt. Note: at

			     * least one interrupt is required and is included in

			     * the minimum descriptor size

			     */

			    ((*buffer - 1) * sizeof(u32)) +

			    /* Add the size of any optional data (resource_source) */

			/* Add the size of the optional resource_source */



			extra_struct_bytes +=

			    acpi_rs_stream_option_length(resource_length -

							 extra_struct_bytes,

							 minimum_aml_resource_length);

			break;



		case ACPI_RESOURCE_NAME_ADDRESS64:

			/*

			 * 64-Bit Address Resource:

			 * Add the size of any optional data (resource_source)

			 * Ensure a 64-bit boundary for the structure

			 */

			extra_struct_bytes = (u32)

			    ACPI_ROUND_UP_to_64_bITS

			    (acpi_rs_stream_option_length

			     (resource_length, minimum_aml_resource_length));

			break;



		default:

			break;

		}



		/* Update the required buffer size for the internal descriptor structs */

		/*

		 * Update the required buffer size for the internal descriptor structs

		 *

		 * Important: Round the size up for the appropriate alignment. This

		 * is a requirement on IA64.

		 */

		buffer_size = acpi_gbl_resource_struct_sizes[resource_index] +

		    extra_struct_bytes;

		buffer_size = ACPI_ROUND_UP_TO_NATIVE_WORD(buffer_size);



		*size_needed += buffer_size;



		temp16 = (u16) (acpi_gbl_resource_struct_sizes[resource_index] +

				extra_struct_bytes);

		buffer_size += (u32) ACPI_ROUND_UP_TO_NATIVE_WORD(temp16);

		ACPI_DEBUG_PRINT((ACPI_DB_RESOURCES,

				  "Type %.2X, Aml %.2X internal %.2X\n",

				  acpi_ut_get_resource_type(aml_buffer),

				  acpi_ut_get_descriptor_length(aml_buffer),

				  buffer_size));



		/*

		 * Point to the next resource within the stream

		 * using the size of the header plus the length contained in the header

		 * Point to the next resource within the AML stream using the length

		 * contained in the resource descriptor header

		 */

		aml_buffer += acpi_ut_get_descriptor_length(aml_buffer);

	}
@@ -589,7 +579,7 @@ acpi_rs_get_pci_routing_table_length(union acpi_operand_object *package_object, 


		/* Round up the size since each element must be aligned */



		temp_size_needed = ACPI_ROUND_UP_to_64_bITS(temp_size_needed);

		temp_size_needed = ACPI_ROUND_UP_to_64_bIT(temp_size_needed);



		/* Point to the next union acpi_operand_object */
@@ -597,7 +587,7 @@ acpi_rs_get_pci_routing_table_length(union acpi_operand_object *package_object, 
	}



	/*

	 * Adding an extra element to the end of the list, essentially a

	 * Add an extra element to the end of the list, essentially a

	 * NULL terminator

	 */

	*buffer_size_needed =

drivers/acpi/resources/rscreate.c

+1 −1
Original line number Diff line number Diff line
@@ -332,7 +332,7 @@ acpi_rs_create_pci_routing_table(union acpi_operand_object *package_object, 
		/* Now align the current length */



		user_prt->length =

		    (u32) ACPI_ROUND_UP_to_64_bITS(user_prt->length);

		    (u32) ACPI_ROUND_UP_to_64_bIT(user_prt->length);



		/* 4) Fourth subobject: Dereference the PRT.source_index */

CRA Git | Maintained and supported by SUSTech CRA and CCSE