>_acpi_{module,function}_name have no need to be (static) pointers to
static
>string literals - they can just be arrays, reducing memory footprint.
This can be applied to ACPI_MODULE_NAME as well.
#define ACPI_MODULE_NAME(Name) static char ACPI_UNUSED_VAR
_AcpiModuleName[] = Name;
Applied with the change below, this results in a total savings of 13.7K
in the full acpica subsystem.
Bob
>-----Original Message-----
>From: Moore, Robert
>Sent: Tuesday, April 29, 2008 1:25 PM
>To: 'akpm@xxxxxxxxxxxxxxxxxxxx'; lenb@xxxxxxxxxx
>Cc: linux-acpi@xxxxxxxxxxxxxxx; jbeulich@xxxxxxxxxx
>Subject: RE: [patch 4/9] acpi: adjust _acpi_{module,function}_name
>definitions
>
>
>
>>_acpi_{module,function}_name have no need to be (static) pointers to
>static
>>string literals - they can just be arrays, reducing memory footprint.
>
>>_ _acpi_function_name can be
>>static (removing the code needed to initialize the variable.
>
>Yes, this is good. Thanks. I see a reduction of 8.1K (code+data) across
the
>entire subsystem.
>
>
>>_acpi_module_name also can be 'const' qualified (requires adding
'const'
>>qualifiers to various functions' parameters)
>
>We can probably do this, as long as it doesn't start a chain of "const
>pollution".
>
>Bob
>
>
>
>
>
>>-----Original Message-----
>>From: linux-acpi-owner@xxxxxxxxxxxxxxx [mailto:linux-acpi-
>>owner@xxxxxxxxxxxxxxx] On Behalf Of akpm@xxxxxxxxxxxxxxxxxxxx
>>Sent: Friday, April 18, 2008 1:27 PM
>>To: lenb@xxxxxxxxxx
>>Cc: linux-acpi@xxxxxxxxxxxxxxx; akpm@xxxxxxxxxxxxxxxxxxxx;
>>jbeulich@xxxxxxxxxx
>>Subject: [patch 4/9] acpi: adjust _acpi_{module,function}_name
definitions
>>
>>From: "Jan Beulich" <jbeulich@xxxxxxxxxx>
>>
>>_acpi_{module,function}_name have no need to be (static) pointers to
>static
>>string literals - they can just be arrays, reducing memory footprint.
>>_acpi_module_name also can be 'const' qualified (requires adding
'const'
>>qualifiers to various functions' parameters), and _acpi_function_name
can
>>be
>>static (removing the code needed to initialize the variable.
>>
>>Signed-off-by: Jan Beulich <jbeulich@xxxxxxxxxx>
>>Cc: Len Brown <lenb@xxxxxxxxxx>
>>Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
>>---
>>
>> drivers/acpi/executer/exdump.c | 4 +-
>> drivers/acpi/namespace/nsutils.c | 23 ++++++------
>> drivers/acpi/utilities/utdebug.c | 30 ++++++++--------
>> drivers/acpi/utilities/utmisc.c | 11 +++---
>> drivers/acpi/utilities/utobject.c | 4 +-
>> include/acpi/acinterp.h | 5 +-
>> include/acpi/aclocal.h | 4 +-
>> include/acpi/acmacros.h | 7 +--
>> include/acpi/acnamesp.h | 19 +++++-----
>> include/acpi/acutils.h | 51 ++++++++++++----------------
>> 10 files changed, 78 insertions(+), 80 deletions(-)
>>
>>diff -puN drivers/acpi/executer/exdump.c~acpi-adjust-
>>_acpi_modulefunction_name-definitions drivers/acpi/executer/exdump.c
>>--- a/drivers/acpi/executer/exdump.c~acpi-adjust-
>_acpi_modulefunction_name-
>>definitions
>>+++ a/drivers/acpi/executer/exdump.c
>>@@ -771,9 +771,9 @@ void acpi_ex_dump_operand(union acpi_ope
>> void
>> acpi_ex_dump_operands(union acpi_operand_object **operands,
>> acpi_interpreter_mode interpreter_mode,
>>- char *ident,
>>+ const char *ident,
>> u32 num_levels,
>>- char *note, char *module_name, u32 line_number)
>>+ const char *note, const char *module_name, u32
>>line_number)
>> {
>> acpi_native_uint i;
>>
>>diff -puN drivers/acpi/namespace/nsutils.c~acpi-adjust-
>>_acpi_modulefunction_name-definitions drivers/acpi/namespace/nsutils.c
>>--- a/drivers/acpi/namespace/nsutils.c~acpi-adjust-
>>_acpi_modulefunction_name-definitions
>>+++ a/drivers/acpi/namespace/nsutils.c
>>@@ -73,9 +73,9 @@ acpi_name acpi_ns_find_parent_name(struc
>>
>>**********************************************************************
****
>*
>>***/
>>
>> void
>>-acpi_ns_report_error(char *module_name,
>>+acpi_ns_report_error(const char *module_name,
>> u32 line_number,
>>- char *internal_name, acpi_status lookup_status)
>>+ const char *internal_name, acpi_status
lookup_status)
>> {
>> acpi_status status;
>> u32 bad_name;
>>@@ -130,11 +130,11 @@ acpi_ns_report_error(char *module_name,
>>
>>**********************************************************************
****
>*
>>***/
>>
>> void
>>-acpi_ns_report_method_error(char *module_name,
>>+acpi_ns_report_method_error(const char *module_name,
>> u32 line_number,
>>- char *message,
>>+ const char *message,
>> struct acpi_namespace_node *prefix_node,
>>- char *path, acpi_status method_status)
>>+ const char *path, acpi_status method_status)
>> {
>> acpi_status status;
>> struct acpi_namespace_node *node = prefix_node;
>>@@ -167,7 +167,7 @@ acpi_ns_report_method_error(char *module
>>
>>**********************************************************************
****
>*
>>***/
>>
>> void
>>-acpi_ns_print_node_pathname(struct acpi_namespace_node *node, char
>>*message)
>>+acpi_ns_print_node_pathname(struct acpi_namespace_node *node, const
char
>>*message)
>> {
>> struct acpi_buffer buffer;
>> acpi_status status;
>>@@ -296,7 +296,7 @@ u32 acpi_ns_local(acpi_object_type type)
>>
>> void acpi_ns_get_internal_name_length(struct acpi_namestring_info
*info)
>> {
>>- char *next_external_char;
>>+ const char *next_external_char;
>> u32 i;
>>
>> ACPI_FUNCTION_ENTRY();
>>@@ -363,7 +363,7 @@ acpi_status acpi_ns_build_internal_name(
>> {
>> u32 num_segments = info->num_segments;
>> char *internal_name = info->internal_name;
>>- char *external_name = info->next_external_char;
>>+ const char *external_name = info->next_external_char;
>> char *result = NULL;
>> acpi_native_uint i;
>>
>>@@ -472,7 +472,8 @@ acpi_status acpi_ns_build_internal_name(
>> *
>>
>>**********************************************************************
****
>*
>>****/
>>
>>-acpi_status acpi_ns_internalize_name(char *external_name, char
>>**converted_name)
>>+acpi_status acpi_ns_internalize_name(const char *external_name,
>>+ char **converted_name)
>> {
>> char *internal_name;
>> struct acpi_namestring_info info;
>>@@ -528,7 +529,7 @@ acpi_status acpi_ns_internalize_name(cha
>>
>> acpi_status
>> acpi_ns_externalize_name(u32 internal_name_length,
>>- char *internal_name,
>>+ const char *internal_name,
>> u32 * converted_name_length, char
**converted_name)
>> {
>> acpi_native_uint names_index = 0;
>>@@ -823,7 +824,7 @@ u32 acpi_ns_opens_scope(acpi_object_type
>>
>> acpi_status
>> acpi_ns_get_node(struct acpi_namespace_node *prefix_node,
>>- char *pathname,
>>+ const char *pathname,
>> u32 flags, struct acpi_namespace_node **return_node)
>> {
>> union acpi_generic_state scope_info;
>>diff -puN drivers/acpi/utilities/utdebug.c~acpi-adjust-
>>_acpi_modulefunction_name-definitions drivers/acpi/utilities/utdebug.c
>>--- a/drivers/acpi/utilities/utdebug.c~acpi-adjust-
>>_acpi_modulefunction_name-definitions
>>+++ a/drivers/acpi/utilities/utdebug.c
>>@@ -156,8 +156,8 @@ static const char *acpi_ut_trim_function
>> void ACPI_INTERNAL_VAR_XFACE
>> acpi_ut_debug_print(u32 requested_debug_level,
>> u32 line_number,
>>- const char *function_name,
>>- char *module_name, u32 component_id, char *format,
...)
>>+ const char *function_name, const char *module_name,
>>+ u32 component_id, const char *format, ...)
>> {
>> acpi_thread_id thread_id;
>> va_list args;
>>@@ -227,8 +227,8 @@ ACPI_EXPORT_SYMBOL(acpi_ut_debug_print)
>> void ACPI_INTERNAL_VAR_XFACE
>> acpi_ut_debug_print_raw(u32 requested_debug_level,
>> u32 line_number,
>>- const char *function_name,
>>- char *module_name, u32 component_id, char
*format, ...)
>>+ const char *function_name, const char
*module_name,
>>+ u32 component_id, const char *format, ...)
>> {
>> va_list args;
>>
>>@@ -261,7 +261,7 @@ ACPI_EXPORT_SYMBOL(acpi_ut_debug_print_r
>>
>>**********************************************************************
****
>*
>>***/
>> void
>> acpi_ut_trace(u32 line_number,
>>- const char *function_name, char *module_name, u32
component_id)
>>+ const char *function_name, const char *module_name, u32
>>component_id)
>> {
>>
>> acpi_gbl_nesting_level++;
>>@@ -293,7 +293,7 @@ ACPI_EXPORT_SYMBOL(acpi_ut_trace)
>> void
>> acpi_ut_trace_ptr(u32 line_number,
>> const char *function_name,
>>- char *module_name, u32 component_id, void *pointer)
>>+ const char *module_name, u32 component_id, void
*pointer)
>> {
>> acpi_gbl_nesting_level++;
>> acpi_ut_track_stack_ptr();
>>@@ -324,7 +324,7 @@ acpi_ut_trace_ptr(u32 line_number,
>> void
>> acpi_ut_trace_str(u32 line_number,
>> const char *function_name,
>>- char *module_name, u32 component_id, char *string)
>>+ const char *module_name, u32 component_id, const char
>>*string)
>> {
>>
>> acpi_gbl_nesting_level++;
>>@@ -356,7 +356,7 @@ acpi_ut_trace_str(u32 line_number,
>> void
>> acpi_ut_trace_u32(u32 line_number,
>> const char *function_name,
>>- char *module_name, u32 component_id, u32 integer)
>>+ const char *module_name, u32 component_id, u32
integer)
>> {
>>
>> acpi_gbl_nesting_level++;
>>@@ -385,8 +385,8 @@ acpi_ut_trace_u32(u32 line_number,
>>
>>**********************************************************************
****
>*
>>***/
>>
>> void
>>-acpi_ut_exit(u32 line_number,
>>- const char *function_name, char *module_name, u32
component_id)
>>+acpi_ut_exit(u32 line_number, const char *function_name,
>>+ const char *module_name, u32 component_id)
>> {
>>
>> acpi_ut_debug_print(ACPI_LV_FUNCTIONS,
>>@@ -416,8 +416,8 @@ ACPI_EXPORT_SYMBOL(acpi_ut_exit)
>>
>>**********************************************************************
****
>*
>>***/
>> void
>> acpi_ut_status_exit(u32 line_number,
>>- const char *function_name,
>>- char *module_name, u32 component_id, acpi_status
status)
>>+ const char *function_name, const char *module_name,
>>+ u32 component_id, acpi_status status)
>> {
>>
>> if (ACPI_SUCCESS(status)) {
>>@@ -457,8 +457,8 @@ ACPI_EXPORT_SYMBOL(acpi_ut_status_exit)
>>
>>**********************************************************************
****
>*
>>***/
>> void
>> acpi_ut_value_exit(u32 line_number,
>>- const char *function_name,
>>- char *module_name, u32 component_id, acpi_integer
value)
>>+ const char *function_name, const char *module_name,
>>+ u32 component_id, acpi_integer value)
>> {
>>
>> acpi_ut_debug_print(ACPI_LV_FUNCTIONS,
>>@@ -490,7 +490,7 @@ ACPI_EXPORT_SYMBOL(acpi_ut_value_exit)
>> void
>> acpi_ut_ptr_exit(u32 line_number,
>> const char *function_name,
>>- char *module_name, u32 component_id, u8 * ptr)
>>+ const char *module_name, u32 component_id, u8 * ptr)
>> {
>>
>> acpi_ut_debug_print(ACPI_LV_FUNCTIONS,
>>diff -puN drivers/acpi/utilities/utmisc.c~acpi-adjust-
>>_acpi_modulefunction_name-definitions drivers/acpi/utilities/utmisc.c
>>--- a/drivers/acpi/utilities/utmisc.c~acpi-adjust-
>>_acpi_modulefunction_name-definitions
>>+++ a/drivers/acpi/utilities/utmisc.c
>>@@ -1024,7 +1024,7 @@ acpi_ut_walk_package_tree(union acpi_ope
>>
>>**********************************************************************
****
>*
>>***/
>>
>> void ACPI_INTERNAL_VAR_XFACE
>>-acpi_ut_error(char *module_name, u32 line_number, char *format, ...)
>>+acpi_ut_error(const char *module_name, u32 line_number, const char
>>*format, ...)
>> {
>> va_list args;
>>
>>@@ -1037,8 +1037,8 @@ acpi_ut_error(char *module_name, u32 lin
>> }
>>
>> void ACPI_INTERNAL_VAR_XFACE
>>-acpi_ut_exception(char *module_name,
>>- u32 line_number, acpi_status status, char *format,
...)
>>+acpi_ut_exception(const char *module_name,
>>+ u32 line_number, acpi_status status, const char
*format, ...)
>> {
>> va_list args;
>>
>>@@ -1053,7 +1053,8 @@ acpi_ut_exception(char *module_name,
>> EXPORT_SYMBOL(acpi_ut_exception);
>>
>> void ACPI_INTERNAL_VAR_XFACE
>>-acpi_ut_warning(char *module_name, u32 line_number, char *format,
...)
>>+acpi_ut_warning(const char *module_name,
>>+ u32 line_number, const char *format, ...)
>> {
>> va_list args;
>>
>>@@ -1067,7 +1068,7 @@ acpi_ut_warning(char *module_name, u32 l
>> }
>>
>> void ACPI_INTERNAL_VAR_XFACE
>>-acpi_ut_info(char *module_name, u32 line_number, char *format, ...)
>>+acpi_ut_info(const char *module_name, u32 line_number, const char
>*format,
>>...)
>> {
>> va_list args;
>>
>>diff -puN drivers/acpi/utilities/utobject.c~acpi-adjust-
>>_acpi_modulefunction_name-definitions
drivers/acpi/utilities/utobject.c
>>--- a/drivers/acpi/utilities/utobject.c~acpi-adjust-
>>_acpi_modulefunction_name-definitions
>>+++ a/drivers/acpi/utilities/utobject.c
>>@@ -83,7 +83,7 @@ acpi_ut_get_element_length(u8 object_typ
>> *
>>
>>**********************************************************************
****
>*
>>***/
>>
>>-union acpi_operand_object *acpi_ut_create_internal_object_dbg(char
>>*module_name,
>>+union acpi_operand_object *acpi_ut_create_internal_object_dbg(const
char
>>*module_name,
>> u32
line_number,
>> u32
component_id,
>>
acpi_object_type
>>@@ -347,7 +347,7 @@ u8 acpi_ut_valid_internal_object(void *o
>> *
>>
>>**********************************************************************
****
>*
>>***/
>>
>>-void *acpi_ut_allocate_object_desc_dbg(char *module_name,
>>+void *acpi_ut_allocate_object_desc_dbg(const char *module_name,
>> u32 line_number, u32
component_id)
>> {
>> union acpi_operand_object *object;
>>diff -puN
include/acpi/acinterp.h~acpi-adjust-_acpi_modulefunction_name-
>>definitions include/acpi/acinterp.h
>>--- a/include/acpi/acinterp.h~acpi-adjust-_acpi_modulefunction_name-
>>definitions
>>+++ a/include/acpi/acinterp.h
>>@@ -367,9 +367,10 @@ void acpi_ex_dump_operand(union acpi_ope
>> void
>> acpi_ex_dump_operands(union acpi_operand_object **operands,
>> acpi_interpreter_mode interpreter_mode,
>>- char *ident,
>>+ const char *ident,
>> u32 num_levels,
>>- char *note, char *module_name, u32 line_number);
>>+ const char *note, const char *module_name,
>>+ u32 line_number);
>>
>> #ifdef ACPI_FUTURE_USAGE
>> void
>>diff -puN
include/acpi/aclocal.h~acpi-adjust-_acpi_modulefunction_name-
>>definitions include/acpi/aclocal.h
>>--- a/include/acpi/aclocal.h~acpi-adjust-_acpi_modulefunction_name-
>>definitions
>>+++ a/include/acpi/aclocal.h
>>@@ -282,8 +282,8 @@ struct acpi_predefined_names {
>> /* Info structure used to convert external<->internal namestrings */
>>
>> struct acpi_namestring_info {
>>- char *external_name;
>>- char *next_external_char;
>>+ const char *external_name;
>>+ const char *next_external_char;
>> char *internal_name;
>> u32 length;
>> u32 num_segments;
>>diff -puN
include/acpi/acmacros.h~acpi-adjust-_acpi_modulefunction_name-
>>definitions include/acpi/acmacros.h
>>--- a/include/acpi/acmacros.h~acpi-adjust-_acpi_modulefunction_name-
>>definitions
>>+++ a/include/acpi/acmacros.h
>>@@ -414,7 +414,7 @@ struct acpi_integer_overlay {
>> * error messages. The __FILE__ macro is not very useful for this,
>because
>>it
>> * often includes the entire pathname to the module
>> */
>>-#define ACPI_MODULE_NAME(name) static char ACPI_UNUSED_VAR
>>*_acpi_module_name = name;
>>+#define ACPI_MODULE_NAME(name) static const char
ACPI_UNUSED_VAR
>>_acpi_module_name[] = name;
>> #else
>> #define ACPI_MODULE_NAME(name)
>> #endif
>>@@ -467,8 +467,7 @@ struct acpi_integer_overlay {
>> /*
>> * If ACPI_GET_FUNCTION_NAME was not defined in the
compiler-dependent
>>header,
>> * define it now. This is the case where there the compiler does not
>>support
>>- * a __FUNCTION__ macro or equivalent. We save the function name on
the
>>- * local stack.
>>+ * a __FUNCTION__ macro or equivalent.
>> */
>> #ifndef ACPI_GET_FUNCTION_NAME
>> #define ACPI_GET_FUNCTION_NAME _acpi_function_name
>>@@ -479,7 +478,7 @@ struct acpi_integer_overlay {
>> * Note: (const char) is used to be compatible with the debug
interfaces
>> * and macros such as __FUNCTION__.
>> */
>>-#define ACPI_FUNCTION_NAME(name) const char
*_acpi_function_name =
>>#name;
>>+#define ACPI_FUNCTION_NAME(name) static const char
>>_acpi_function_name[] = #name;
>>
>> #else
>> /* Compiler supports __FUNCTION__ (or equivalent) -- Ignore this
macro */
>>diff -puN
include/acpi/acnamesp.h~acpi-adjust-_acpi_modulefunction_name-
>>definitions include/acpi/acnamesp.h
>>--- a/include/acpi/acnamesp.h~acpi-adjust-_acpi_modulefunction_name-
>>definitions
>>+++ a/include/acpi/acnamesp.h
>>@@ -201,7 +201,7 @@ acpi_ns_pattern_match(struct acpi_namesp
>>
>> acpi_status
>> acpi_ns_get_node(struct acpi_namespace_node *prefix_node,
>>- char *external_pathname,
>>+ const char *external_pathname,
>> u32 flags, struct acpi_namespace_node **out_node);
>>
>> acpi_size acpi_ns_get_pathname_length(struct acpi_namespace_node
*node);
>>@@ -265,28 +265,29 @@ acpi_object_type acpi_ns_get_type(struct
>> u32 acpi_ns_local(acpi_object_type type);
>>
>> void
>>-acpi_ns_report_error(char *module_name,
>>+acpi_ns_report_error(const char *module_name,
>> u32 line_number,
>>- char *internal_name, acpi_status lookup_status);
>>+ const char *internal_name, acpi_status
lookup_status);
>>
>> void
>>-acpi_ns_report_method_error(char *module_name,
>>+acpi_ns_report_method_error(const char *module_name,
>> u32 line_number,
>>- char *message,
>>+ const char *message,
>> struct acpi_namespace_node *node,
>>- char *path, acpi_status lookup_status);
>>+ const char *path, acpi_status
lookup_status);
>>
>>-void acpi_ns_print_node_pathname(struct acpi_namespace_node *node,
char
>>*msg);
>>+void acpi_ns_print_node_pathname(struct acpi_namespace_node *node,
>>+ const char *msg);
>>
>> acpi_status acpi_ns_build_internal_name(struct acpi_namestring_info
>>*info);
>>
>> void acpi_ns_get_internal_name_length(struct acpi_namestring_info
*info);
>>
>>-acpi_status acpi_ns_internalize_name(char *dotted_name, char
>>**converted_name);
>>+acpi_status acpi_ns_internalize_name(const char *dotted_name, char
>>**converted_name);
>>
>> acpi_status
>> acpi_ns_externalize_name(u32 internal_name_length,
>>- char *internal_name,
>>+ const char *internal_name,
>> u32 * converted_name_length, char
**converted_name);
>>
>> struct acpi_namespace_node *acpi_ns_map_handle_to_node(acpi_handle
>>handle);
>>diff -puN
include/acpi/acutils.h~acpi-adjust-_acpi_modulefunction_name-
>>definitions include/acpi/acutils.h
>>--- a/include/acpi/acutils.h~acpi-adjust-_acpi_modulefunction_name-
>>definitions
>>+++ a/include/acpi/acutils.h
>>@@ -245,85 +245,80 @@ void acpi_ut_track_stack_ptr(void);
>>
>> void
>> acpi_ut_trace(u32 line_number,
>>- const char *function_name, char *module_name, u32
>>component_id);
>>+ const char *function_name, const char *module_name, u32
>>component_id);
>>
>> void
>> acpi_ut_trace_ptr(u32 line_number,
>> const char *function_name,
>>- char *module_name, u32 component_id, void *pointer);
>>+ const char *module_name, u32 component_id, void
*pointer);
>>
>> void
>> acpi_ut_trace_u32(u32 line_number,
>> const char *function_name,
>>- char *module_name, u32 component_id, u32 integer);
>>+ const char *module_name, u32 component_id, u32
integer);
>>
>> void
>> acpi_ut_trace_str(u32 line_number,
>> const char *function_name,
>>- char *module_name, u32 component_id, char *string);
>>+ const char *module_name, u32 component_id, const char
>>*string);
>>
>> void
>> acpi_ut_exit(u32 line_number,
>>- const char *function_name, char *module_name, u32
component_id);
>>+ const char *function_name, const char *module_name, u32
>>component_id);
>>
>> void
>> acpi_ut_status_exit(u32 line_number,
>> const char *function_name,
>>- char *module_name, u32 component_id, acpi_status
status);
>>+ const char *module_name, u32 component_id,
acpi_status
>>status);
>>
>> void
>> acpi_ut_value_exit(u32 line_number,
>> const char *function_name,
>>- char *module_name, u32 component_id, acpi_integer
value);
>>+ const char *module_name, u32 component_id,
acpi_integer
>>value);
>>
>> void
>> acpi_ut_ptr_exit(u32 line_number,
>> const char *function_name,
>>- char *module_name, u32 component_id, u8 * ptr);
>>+ const char *module_name, u32 component_id, u8 * ptr);
>>
>> void acpi_ut_dump_buffer(u8 * buffer, u32 count, u32 display, u32
>>component_id);
>>
>> void acpi_ut_dump_buffer2(u8 * buffer, u32 count, u32 display);
>>
>>-void acpi_ut_report_error(char *module_name, u32 line_number);
>>-
>>-void acpi_ut_report_info(char *module_name, u32 line_number);
>>-
>>-void acpi_ut_report_warning(char *module_name, u32 line_number);
>>-
>> /* Error and message reporting interfaces */
>>
>> void ACPI_INTERNAL_VAR_XFACE
>> acpi_ut_debug_print(u32 requested_debug_level,
>> u32 line_number,
>> const char *function_name,
>>- char *module_name,
>>- u32 component_id, char *format, ...)
ACPI_PRINTF_LIKE(6);
>>+ const char *module_name,
>>+ u32 component_id,
>>+ const char *format, ...) ACPI_PRINTF_LIKE(6);
>>
>> void ACPI_INTERNAL_VAR_XFACE
>> acpi_ut_debug_print_raw(u32 requested_debug_level,
>> u32 line_number,
>> const char *function_name,
>>- char *module_name,
>>+ const char *module_name,
>> u32 component_id,
>>- char *format, ...) ACPI_PRINTF_LIKE(6);
>>+ const char *format, ...) ACPI_PRINTF_LIKE(6);
>>
>> void ACPI_INTERNAL_VAR_XFACE
>>-acpi_ut_error(char *module_name,
>>- u32 line_number, char *format, ...) ACPI_PRINTF_LIKE(3);
>>+acpi_ut_error(const char *module_name,
>>+ u32 line_number, const char *format, ...)
ACPI_PRINTF_LIKE(3);
>>
>> void ACPI_INTERNAL_VAR_XFACE
>>-acpi_ut_exception(char *module_name,
>>+acpi_ut_exception(const char *module_name,
>> u32 line_number,
>>- acpi_status status, char *format, ...)
ACPI_PRINTF_LIKE(4);
>>+ acpi_status status, const char *format, ...)
>>ACPI_PRINTF_LIKE(4);
>>
>> void ACPI_INTERNAL_VAR_XFACE
>>-acpi_ut_warning(char *module_name,
>>- u32 line_number, char *format, ...) ACPI_PRINTF_LIKE(3);
>>+acpi_ut_warning(const char *module_name,
>>+ u32 line_number, const char *format, ...)
ACPI_PRINTF_LIKE(3);
>>
>> void ACPI_INTERNAL_VAR_XFACE
>>-acpi_ut_info(char *module_name,
>>- u32 line_number, char *format, ...) ACPI_PRINTF_LIKE(3);
>>+acpi_ut_info(const char *module_name,
>>+ u32 line_number, const char *format, ...)
ACPI_PRINTF_LIKE(3);
>>
>> /*
>> * utdelete - Object deletion and reference counts
>>@@ -376,13 +371,13 @@ acpi_ut_execute_sxds(struct acpi_namespa
>> /*
>> * utobject - internal object create/delete/cache routines
>> */
>>-union acpi_operand_object *acpi_ut_create_internal_object_dbg(char
>>*module_name,
>>+union acpi_operand_object *acpi_ut_create_internal_object_dbg(const
char
>>*module_name,
>> u32
line_number,
>> u32
component_id,
>>
acpi_object_type
>> type);
>>
>>-void *acpi_ut_allocate_object_desc_dbg(char *module_name,
>>+void *acpi_ut_allocate_object_desc_dbg(const char *module_name,
>> u32 line_number, u32
component_id);
>>
>> #define acpi_ut_create_internal_object(t)
>>acpi_ut_create_internal_object_dbg
>>(_acpi_module_name,__LINE__,_COMPONENT,t)
>>_
>>--
>>To unsubscribe from this list: send the line "unsubscribe linux-acpi"
in
>>the body of a message to majordomo@xxxxxxxxxxxxxxx
>>More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
