[gentoo-commits] proj/linux-patches:4.15 commit in: /

[Alice Ferrazzi]

commit:     0933e08b7a49a8d4cd1a5bfbc3fba0411031c225
Author:     Alice Ferrazzi <alicef <AT> gentoo <DOT> org>
AuthorDate: Wed Feb 28 19:53:03 2018 +0000
Commit:     Alice Ferrazzi <alicef <AT> gentoo <DOT> org>
CommitDate: Wed Feb 28 19:53:03 2018 +0000
URL:        https://gitweb.gentoo.org/proj/linux-patches.git/commit/?id=0933e08b

dell-laptop: Allocate buffer on heap rather than globally

 0000_README                                        |   4 +
 1700_ia64_fix_ptrace.patch                         | 543 +++++++++++++++++----
 ...ocate_buffer_on_heap_rather_than_globally.patch | 458 +++++++++++++++++
 3 files changed, 920 insertions(+), 85 deletions(-)

diff --git a/0000_README b/0000_README
index a7bb4af..994c735 100644
--- a/0000_README
+++ b/0000_README
@@ -99,6 +99,10 @@ Patch:  2900_dev-root-proc-mount-fix.patch
 From:   https://bugs.gentoo.org/show_bug.cgi?id=438380
 Desc:   Ensure that /dev/root doesn't appear in /proc/mounts when bootint without an initramfs.
 
+Patch:  2901_allocate_buffer_on_heap_rather_than_globally.patch
+From:   https://bugs.gentoo.org/646438
+Desc:   Patchwork [v2] platform/x86: dell-laptop: Allocate buffer on heap rather than globally Bug #646438
+
 Patch:  4200_fbcondecor.patch
 From:   http://www.mepiscommunity.org/fbcondecor
 Desc:   Bootsplash ported by Conrad Kostecki. (Bug #637434)

diff --git a/1700_ia64_fix_ptrace.patch b/1700_ia64_fix_ptrace.patch
index 6173b05..ad48a1a 100644
--- a/1700_ia64_fix_ptrace.patch
+++ b/1700_ia64_fix_ptrace.patch
@@ -1,87 +1,460 @@
-From patchwork Fri Feb  2 22:12:24 2018
-Content-Type: text/plain; charset="utf-8"
-MIME-Version: 1.0
-Content-Transfer-Encoding: 8bit
-Subject: ia64: fix ptrace(PTRACE_GETREGS) (unbreaks strace, gdb)
-From: Sergei Trofimovich <slyfox@g.o>
-X-Patchwork-Id: 10198159
-Message-Id: <20180202221224.16597-1-slyfox@g.o>
-To: Tony Luck <tony.luck@×××××.com>, Fenghua Yu <fenghua.yu@×××××.com>,
- linux-ia64@×××××××××××.org, linux-kernel@×××××××××××.org
-Cc: Sergei Trofimovich <slyfox@g.o>
-Date: Fri,  2 Feb 2018 22:12:24 +0000
-
-The strace breakage looks like that:
-./strace: get_regs: get_regs_error: Input/output error
-
-It happens because ia64 needs to load unwind tables
-to read certain registers. Unwind tables fail to load
-due to GCC quirk on the following code:
-
-    extern char __end_unwind[];
-    const struct unw_table_entry *end = (struct unw_table_entry *)table_end;
-    table->end = segment_base + end[-1].end_offset;
-
-GCC does not generate correct code for this single memory
-reference after constant propagation (see https://gcc.gnu.org/PR84184).
-Two triggers are required for bad code generation:
-- '__end_unwind' has alignment lower (char), than
-  'struct unw_table_entry' (8).
-- symbol offset is negative.
-
-This commit workarounds it by fixing alignment of '__end_unwind'.
-While at it use hidden symbols to generate shorter gp-relative
-relocations.
-
-CC: Tony Luck <tony.luck@×××××.com>
-CC: Fenghua Yu <fenghua.yu@×××××.com>
-CC: linux-ia64@×××××××××××.org
-CC: linux-kernel@×××××××××××.org
-Bug: https://github.com/strace/strace/issues/33
-Bug: https://gcc.gnu.org/PR84184
-Reported-by: Émeric Maschino <emeric.maschino@×××××.com>
-Signed-off-by: Sergei Trofimovich <slyfox@g.o>
-Tested-by: stanton_arch@××××.com
----
- arch/ia64/include/asm/sections.h |  1 -
- arch/ia64/kernel/unwind.c        | 15 ++++++++++++++-
- 2 files changed, 14 insertions(+), 2 deletions(-)
-
-diff --git a/arch/ia64/include/asm/sections.h b/arch/ia64/include/asm/sections.h
-index f3481408594e..0fc4f1757a44 100644
---- a/arch/ia64/include/asm/sections.h
-+++ b/arch/ia64/include/asm/sections.h
-@@ -24,7 +24,6 @@ extern char __start_gate_mckinley_e9_patchlist[], __end_gate_mckinley_e9_patchli
- extern char __start_gate_vtop_patchlist[], __end_gate_vtop_patchlist[];
- extern char __start_gate_fsyscall_patchlist[], __end_gate_fsyscall_patchlist[];
- extern char __start_gate_brl_fsys_bubble_down_patchlist[], __end_gate_brl_fsys_bubble_down_patchlist[];
--extern char __start_unwind[], __end_unwind[];
- extern char __start_ivt_text[], __end_ivt_text[];
- 
- #undef dereference_function_descriptor
-diff --git a/arch/ia64/kernel/unwind.c b/arch/ia64/kernel/unwind.c
-index e04efa088902..025ba6700790 100644
---- a/arch/ia64/kernel/unwind.c
-+++ b/arch/ia64/kernel/unwind.c
-@@ -2243,7 +2243,20 @@ __initcall(create_gate_table);
- void __init
- unw_init (void)
+diff --git a/drivers/platform/x86/dell-laptop.c b/drivers/platform/x86/dell-laptop.c
+index fc2dfc8..a7b1419 100644
+--- a/drivers/platform/x86/dell-laptop.c
++++ b/drivers/platform/x86/dell-laptop.c
+@@ -78,7 +78,6 @@ static struct platform_driver platform_driver = {
+ 	}
+ };
+ 
+-static struct calling_interface_buffer *buffer;
+ static struct platform_device *platform_device;
+ static struct backlight_device *dell_backlight_device;
+ static struct rfkill *wifi_rfkill;
+@@ -322,7 +321,8 @@ static const struct dmi_system_id dell_quirks[] __initconst = {
+ 	{ }
+ };
+ 
+-static void dell_set_arguments(u32 arg0, u32 arg1, u32 arg2, u32 arg3)
++static void dell_fill_request(struct calling_interface_buffer *buffer,
++			       u32 arg0, u32 arg1, u32 arg2, u32 arg3)
+ {
+ 	memset(buffer, 0, sizeof(struct calling_interface_buffer));
+ 	buffer->input[0] = arg0;
+@@ -331,7 +331,8 @@ static void dell_set_arguments(u32 arg0, u32 arg1, u32 arg2, u32 arg3)
+ 	buffer->input[3] = arg3;
+ }
+ 
+-static int dell_send_request(u16 class, u16 select)
++static int dell_send_request(struct calling_interface_buffer *buffer,
++			     u16 class, u16 select)
+ {
+ 	int ret;
+ 
+@@ -468,21 +469,22 @@ static int dell_rfkill_set(void *data, bool blocked)
+ 	int disable = blocked ? 1 : 0;
+ 	unsigned long radio = (unsigned long)data;
+ 	int hwswitch_bit = (unsigned long)data - 1;
++	struct calling_interface_buffer buffer;
+ 	int hwswitch;
+ 	int status;
+ 	int ret;
+ 
+-	dell_set_arguments(0, 0, 0, 0);
+-	ret = dell_send_request(CLASS_INFO, SELECT_RFKILL);
++	dell_fill_request(&buffer, 0, 0, 0, 0);
++	ret = dell_send_request(&buffer, CLASS_INFO, SELECT_RFKILL);
+ 	if (ret)
+ 		return ret;
+-	status = buffer->output[1];
++	status = buffer.output[1];
+ 
+-	dell_set_arguments(0x2, 0, 0, 0);
+-	ret = dell_send_request(CLASS_INFO, SELECT_RFKILL);
++	dell_fill_request(&buffer, 0x2, 0, 0, 0);
++	ret = dell_send_request(&buffer, CLASS_INFO, SELECT_RFKILL);
+ 	if (ret)
+ 		return ret;
+-	hwswitch = buffer->output[1];
++	hwswitch = buffer.output[1];
+ 
+ 	/* If the hardware switch controls this radio, and the hardware
+ 	   switch is disabled, always disable the radio */
+@@ -490,8 +492,8 @@ static int dell_rfkill_set(void *data, bool blocked)
+ 	    (status & BIT(0)) && !(status & BIT(16)))
+ 		disable = 1;
+ 
+-	dell_set_arguments(1 | (radio<<8) | (disable << 16), 0, 0, 0);
+-	ret = dell_send_request(CLASS_INFO, SELECT_RFKILL);
++	dell_fill_request(&buffer, 1 | (radio<<8) | (disable << 16), 0, 0, 0);
++	ret = dell_send_request(&buffer, CLASS_INFO, SELECT_RFKILL);
+ 	return ret;
+ }
+ 
+@@ -500,9 +502,11 @@ static void dell_rfkill_update_sw_state(struct rfkill *rfkill, int radio,
+ {
+ 	if (status & BIT(0)) {
+ 		/* Has hw-switch, sync sw_state to BIOS */
++		struct calling_interface_buffer buffer;
+ 		int block = rfkill_blocked(rfkill);
+-		dell_set_arguments(1 | (radio << 8) | (block << 16), 0, 0, 0);
+-		dell_send_request(CLASS_INFO, SELECT_RFKILL);
++		dell_fill_request(&buffer,
++				   1 | (radio << 8) | (block << 16), 0, 0, 0);
++		dell_send_request(&buffer, CLASS_INFO, SELECT_RFKILL);
+ 	} else {
+ 		/* No hw-switch, sync BIOS state to sw_state */
+ 		rfkill_set_sw_state(rfkill, !!(status & BIT(radio + 16)));
+@@ -519,21 +523,22 @@ static void dell_rfkill_update_hw_state(struct rfkill *rfkill, int radio,
+ static void dell_rfkill_query(struct rfkill *rfkill, void *data)
+ {
+ 	int radio = ((unsigned long)data & 0xF);
++	struct calling_interface_buffer buffer;
+ 	int hwswitch;
+ 	int status;
+ 	int ret;
+ 
+-	dell_set_arguments(0, 0, 0, 0);
+-	ret = dell_send_request(CLASS_INFO, SELECT_RFKILL);
+-	status = buffer->output[1];
++	dell_fill_request(&buffer, 0, 0, 0, 0);
++	ret = dell_send_request(&buffer, CLASS_INFO, SELECT_RFKILL);
++	status = buffer.output[1];
+ 
+ 	if (ret != 0 || !(status & BIT(0))) {
+ 		return;
+ 	}
+ 
+-	dell_set_arguments(0, 0x2, 0, 0);
+-	ret = dell_send_request(CLASS_INFO, SELECT_RFKILL);
+-	hwswitch = buffer->output[1];
++	dell_fill_request(&buffer, 0, 0x2, 0, 0);
++	ret = dell_send_request(&buffer, CLASS_INFO, SELECT_RFKILL);
++	hwswitch = buffer.output[1];
+ 
+ 	if (ret != 0)
+ 		return;
+@@ -550,22 +555,23 @@ static struct dentry *dell_laptop_dir;
+ 
+ static int dell_debugfs_show(struct seq_file *s, void *data)
  {
--	extern char __gp[];
-+	#define __ia64_hidden __attribute__((visibility("hidden")))
-+	/*
-+	 * We use hidden symbols to generate more efficient code using
-+	 * gp-relative addressing.
-+	 */
-+	extern char __gp[] __ia64_hidden;
-+	/*
-+	 * Unwind tables need to have proper alignment as init_unwind_table()
-+	 * uses negative offsets against '__end_unwind'.
-+	 * See https://gcc.gnu.org/PR84184
-+	 */
-+	extern const struct unw_table_entry __start_unwind[] __ia64_hidden;
-+	extern const struct unw_table_entry __end_unwind[] __ia64_hidden;
-+	#undef __ia64_hidden
- 	extern void unw_hash_index_t_is_too_narrow (void);
- 	long i, off;
++	struct calling_interface_buffer buffer;
+ 	int hwswitch_state;
+ 	int hwswitch_ret;
+ 	int status;
+ 	int ret;
+ 
+-	dell_set_arguments(0, 0, 0, 0);
+-	ret = dell_send_request(CLASS_INFO, SELECT_RFKILL);
++	dell_fill_request(&buffer, 0, 0, 0, 0);
++	ret = dell_send_request(&buffer, CLASS_INFO, SELECT_RFKILL);
+ 	if (ret)
+ 		return ret;
+-	status = buffer->output[1];
++	status = buffer.output[1];
+ 
+-	dell_set_arguments(0, 0x2, 0, 0);
+-	hwswitch_ret = dell_send_request(CLASS_INFO, SELECT_RFKILL);
++	dell_fill_request(&buffer, 0, 0x2, 0, 0);
++	hwswitch_ret = dell_send_request(&buffer, CLASS_INFO, SELECT_RFKILL);
+ 	if (hwswitch_ret)
+ 		return hwswitch_ret;
+-	hwswitch_state = buffer->output[1];
++	hwswitch_state = buffer.output[1];
+ 
+ 	seq_printf(s, "return:\t%d\n", ret);
+ 	seq_printf(s, "status:\t0x%X\n", status);
+@@ -646,22 +652,23 @@ static const struct file_operations dell_debugfs_fops = {
+ 
+ static void dell_update_rfkill(struct work_struct *ignored)
+ {
++	struct calling_interface_buffer buffer;
+ 	int hwswitch = 0;
+ 	int status;
+ 	int ret;
+ 
+-	dell_set_arguments(0, 0, 0, 0);
+-	ret = dell_send_request(CLASS_INFO, SELECT_RFKILL);
+-	status = buffer->output[1];
++	dell_fill_request(&buffer, 0, 0, 0, 0);
++	ret = dell_send_request(&buffer, CLASS_INFO, SELECT_RFKILL);
++	status = buffer.output[1];
+ 
+ 	if (ret != 0)
+ 		return;
+ 
+-	dell_set_arguments(0, 0x2, 0, 0);
+-	ret = dell_send_request(CLASS_INFO, SELECT_RFKILL);
++	dell_fill_request(&buffer, 0, 0x2, 0, 0);
++	ret = dell_send_request(&buffer, CLASS_INFO, SELECT_RFKILL);
+ 
+ 	if (ret == 0 && (status & BIT(0)))
+-		hwswitch = buffer->output[1];
++		hwswitch = buffer.output[1];
+ 
+ 	if (wifi_rfkill) {
+ 		dell_rfkill_update_hw_state(wifi_rfkill, 1, status, hwswitch);
+@@ -719,6 +726,7 @@ static struct notifier_block dell_laptop_rbtn_notifier = {
+ 
+ static int __init dell_setup_rfkill(void)
+ {
++	struct calling_interface_buffer buffer;
+ 	int status, ret, whitelisted;
+ 	const char *product;
+ 
+@@ -734,9 +742,9 @@ static int __init dell_setup_rfkill(void)
+ 	if (!force_rfkill && !whitelisted)
+ 		return 0;
+ 
+-	dell_set_arguments(0, 0, 0, 0);
+-	ret = dell_send_request(CLASS_INFO, SELECT_RFKILL);
+-	status = buffer->output[1];
++	dell_fill_request(&buffer, 0, 0, 0, 0);
++	ret = dell_send_request(&buffer, CLASS_INFO, SELECT_RFKILL);
++	status = buffer.output[1];
+ 
+ 	/* dell wireless info smbios call is not supported */
+ 	if (ret != 0)
+@@ -889,6 +897,7 @@ static void dell_cleanup_rfkill(void)
+ 
+ static int dell_send_intensity(struct backlight_device *bd)
+ {
++	struct calling_interface_buffer buffer;
+ 	struct calling_interface_token *token;
+ 	int ret;
+ 
+@@ -896,17 +905,21 @@ static int dell_send_intensity(struct backlight_device *bd)
+ 	if (!token)
+ 		return -ENODEV;
+ 
+-	dell_set_arguments(token->location, bd->props.brightness, 0, 0);
++	dell_fill_request(&buffer,
++			   token->location, bd->props.brightness, 0, 0);
+ 	if (power_supply_is_system_supplied() > 0)
+-		ret = dell_send_request(CLASS_TOKEN_WRITE, SELECT_TOKEN_AC);
++		ret = dell_send_request(&buffer,
++					CLASS_TOKEN_WRITE, SELECT_TOKEN_AC);
+ 	else
+-		ret = dell_send_request(CLASS_TOKEN_WRITE, SELECT_TOKEN_BAT);
++		ret = dell_send_request(&buffer,
++					CLASS_TOKEN_WRITE, SELECT_TOKEN_BAT);
+ 
+ 	return ret;
+ }
+ 
+ static int dell_get_intensity(struct backlight_device *bd)
+ {
++	struct calling_interface_buffer buffer;
+ 	struct calling_interface_token *token;
+ 	int ret;
+ 
+@@ -914,14 +927,17 @@ static int dell_get_intensity(struct backlight_device *bd)
+ 	if (!token)
+ 		return -ENODEV;
+ 
+-	dell_set_arguments(token->location, 0, 0, 0);
++	dell_fill_request(&buffer, token->location, 0, 0, 0);
+ 	if (power_supply_is_system_supplied() > 0)
+-		ret = dell_send_request(CLASS_TOKEN_READ, SELECT_TOKEN_AC);
++		ret = dell_send_request(&buffer,
++					CLASS_TOKEN_READ, SELECT_TOKEN_AC);
+ 	else
+-		ret = dell_send_request(CLASS_TOKEN_READ, SELECT_TOKEN_BAT);
++		ret = dell_send_request(&buffer,
++					CLASS_TOKEN_READ, SELECT_TOKEN_BAT);
+ 
+ 	if (ret == 0)
+-		ret = buffer->output[1];
++		ret = buffer.output[1];
++
+ 	return ret;
+ }
+ 
+@@ -1186,31 +1202,33 @@ static enum led_brightness kbd_led_level;
+ 
+ static int kbd_get_info(struct kbd_info *info)
+ {
++	struct calling_interface_buffer buffer;
+ 	u8 units;
+ 	int ret;
+ 
+-	dell_set_arguments(0, 0, 0, 0);
+-	ret = dell_send_request(CLASS_KBD_BACKLIGHT, SELECT_KBD_BACKLIGHT);
++	dell_fill_request(&buffer, 0, 0, 0, 0);
++	ret = dell_send_request(&buffer,
++				CLASS_KBD_BACKLIGHT, SELECT_KBD_BACKLIGHT);
+ 	if (ret)
+ 		return ret;
+ 
+-	info->modes = buffer->output[1] & 0xFFFF;
+-	info->type = (buffer->output[1] >> 24) & 0xFF;
+-	info->triggers = buffer->output[2] & 0xFF;
+-	units = (buffer->output[2] >> 8) & 0xFF;
+-	info->levels = (buffer->output[2] >> 16) & 0xFF;
++	info->modes = buffer.output[1] & 0xFFFF;
++	info->type = (buffer.output[1] >> 24) & 0xFF;
++	info->triggers = buffer.output[2] & 0xFF;
++	units = (buffer.output[2] >> 8) & 0xFF;
++	info->levels = (buffer.output[2] >> 16) & 0xFF;
+ 
+ 	if (quirks && quirks->kbd_led_levels_off_1 && info->levels)
+ 		info->levels--;
+ 
+ 	if (units & BIT(0))
+-		info->seconds = (buffer->output[3] >> 0) & 0xFF;
++		info->seconds = (buffer.output[3] >> 0) & 0xFF;
+ 	if (units & BIT(1))
+-		info->minutes = (buffer->output[3] >> 8) & 0xFF;
++		info->minutes = (buffer.output[3] >> 8) & 0xFF;
+ 	if (units & BIT(2))
+-		info->hours = (buffer->output[3] >> 16) & 0xFF;
++		info->hours = (buffer.output[3] >> 16) & 0xFF;
+ 	if (units & BIT(3))
+-		info->days = (buffer->output[3] >> 24) & 0xFF;
++		info->days = (buffer.output[3] >> 24) & 0xFF;
+ 
+ 	return ret;
+ }
+@@ -1270,31 +1288,34 @@ static int kbd_set_level(struct kbd_state *state, u8 level)
+ 
+ static int kbd_get_state(struct kbd_state *state)
+ {
++	struct calling_interface_buffer buffer;
+ 	int ret;
+ 
+-	dell_set_arguments(0x1, 0, 0, 0);
+-	ret = dell_send_request(CLASS_KBD_BACKLIGHT, SELECT_KBD_BACKLIGHT);
++	dell_fill_request(&buffer, 0, 0, 0, 0);
++	ret = dell_send_request(&buffer,
++				CLASS_KBD_BACKLIGHT, SELECT_KBD_BACKLIGHT);
+ 	if (ret)
+ 		return ret;
+ 
+-	state->mode_bit = ffs(buffer->output[1] & 0xFFFF);
++	state->mode_bit = ffs(buffer.output[1] & 0xFFFF);
+ 	if (state->mode_bit != 0)
+ 		state->mode_bit--;
+ 
+-	state->triggers = (buffer->output[1] >> 16) & 0xFF;
+-	state->timeout_value = (buffer->output[1] >> 24) & 0x3F;
+-	state->timeout_unit = (buffer->output[1] >> 30) & 0x3;
+-	state->als_setting = buffer->output[2] & 0xFF;
+-	state->als_value = (buffer->output[2] >> 8) & 0xFF;
+-	state->level = (buffer->output[2] >> 16) & 0xFF;
+-	state->timeout_value_ac = (buffer->output[2] >> 24) & 0x3F;
+-	state->timeout_unit_ac = (buffer->output[2] >> 30) & 0x3;
++	state->triggers = (buffer.output[1] >> 16) & 0xFF;
++	state->timeout_value = (buffer.output[1] >> 24) & 0x3F;
++	state->timeout_unit = (buffer.output[1] >> 30) & 0x3;
++	state->als_setting = buffer.output[2] & 0xFF;
++	state->als_value = (buffer.output[2] >> 8) & 0xFF;
++	state->level = (buffer.output[2] >> 16) & 0xFF;
++	state->timeout_value_ac = (buffer.output[2] >> 24) & 0x3F;
++	state->timeout_unit_ac = (buffer.output[2] >> 30) & 0x3;
+ 
+ 	return ret;
+ }
+ 
+ static int kbd_set_state(struct kbd_state *state)
+ {
++	struct calling_interface_buffer buffer;
+ 	int ret;
+ 	u32 input1;
+ 	u32 input2;
+@@ -1307,8 +1328,9 @@ static int kbd_set_state(struct kbd_state *state)
+ 	input2 |= (state->level & 0xFF) << 16;
+ 	input2 |= (state->timeout_value_ac & 0x3F) << 24;
+ 	input2 |= (state->timeout_unit_ac & 0x3) << 30;
+-	dell_set_arguments(0x2, input1, input2, 0);
+-	ret = dell_send_request(CLASS_KBD_BACKLIGHT, SELECT_KBD_BACKLIGHT);
++	dell_fill_request(&buffer, 0x2, input1, input2, 0);
++	ret = dell_send_request(&buffer,
++				CLASS_KBD_BACKLIGHT, SELECT_KBD_BACKLIGHT);
+ 
+ 	return ret;
+ }
+@@ -1335,6 +1357,7 @@ static int kbd_set_state_safe(struct kbd_state *state, struct kbd_state *old)
+ 
+ static int kbd_set_token_bit(u8 bit)
+ {
++	struct calling_interface_buffer buffer;
+ 	struct calling_interface_token *token;
+ 	int ret;
+ 
+@@ -1345,14 +1368,15 @@ static int kbd_set_token_bit(u8 bit)
+ 	if (!token)
+ 		return -EINVAL;
+ 
+-	dell_set_arguments(token->location, token->value, 0, 0);
+-	ret = dell_send_request(CLASS_TOKEN_WRITE, SELECT_TOKEN_STD);
++	dell_fill_request(&buffer, token->location, token->value, 0, 0);
++	ret = dell_send_request(&buffer, CLASS_TOKEN_WRITE, SELECT_TOKEN_STD);
+ 
+ 	return ret;
+ }
+ 
+ static int kbd_get_token_bit(u8 bit)
+ {
++	struct calling_interface_buffer buffer;
+ 	struct calling_interface_token *token;
+ 	int ret;
+ 	int val;
+@@ -1364,9 +1388,9 @@ static int kbd_get_token_bit(u8 bit)
+ 	if (!token)
+ 		return -EINVAL;
+ 
+-	dell_set_arguments(token->location, 0, 0, 0);
+-	ret = dell_send_request(CLASS_TOKEN_READ, SELECT_TOKEN_STD);
+-	val = buffer->output[1];
++	dell_fill_request(&buffer, token->location, 0, 0, 0);
++	ret = dell_send_request(&buffer, CLASS_TOKEN_READ, SELECT_TOKEN_STD);
++	val = buffer.output[1];
+ 
+ 	if (ret)
+ 		return ret;
+@@ -2102,6 +2126,7 @@ static struct notifier_block dell_laptop_notifier = {
+ 
+ int dell_micmute_led_set(int state)
+ {
++	struct calling_interface_buffer buffer;
+ 	struct calling_interface_token *token;
+ 
+ 	if (state == 0)
+@@ -2114,8 +2139,8 @@ int dell_micmute_led_set(int state)
+ 	if (!token)
+ 		return -ENODEV;
+ 
+-	dell_set_arguments(token->location, token->value, 0, 0);
+-	dell_send_request(CLASS_TOKEN_WRITE, SELECT_TOKEN_STD);
++	dell_fill_request(&buffer, token->location, token->value, 0, 0);
++	dell_send_request(&buffer, CLASS_TOKEN_WRITE, SELECT_TOKEN_STD);
+ 
+ 	return state;
+ }
+@@ -2146,13 +2171,6 @@ static int __init dell_init(void)
+ 	if (ret)
+ 		goto fail_platform_device2;
+ 
+-	buffer = kzalloc(sizeof(struct calling_interface_buffer), GFP_KERNEL);
+-	if (!buffer) {
+-		ret = -ENOMEM;
+-		goto fail_buffer;
+-	}
+-
+-
+ 	ret = dell_setup_rfkill();
+ 
+ 	if (ret) {
+@@ -2177,10 +2195,13 @@ static int __init dell_init(void)
+ 
+ 	token = dell_smbios_find_token(BRIGHTNESS_TOKEN);
+ 	if (token) {
+-		dell_set_arguments(token->location, 0, 0, 0);
+-		ret = dell_send_request(CLASS_TOKEN_READ, SELECT_TOKEN_AC);
++		struct calling_interface_buffer buffer;
++
++		dell_fill_request(&buffer, token->location, 0, 0, 0);
++		ret = dell_send_request(&buffer,
++					CLASS_TOKEN_READ, SELECT_TOKEN_AC);
+ 		if (ret)
+-			max_intensity = buffer->output[3];
++			max_intensity = buffer.output[3];
+ 	}
  
+ 	if (max_intensity) {
+@@ -2214,8 +2235,6 @@ static int __init dell_init(void)
+ fail_get_brightness:
+ 	backlight_device_unregister(dell_backlight_device);
+ fail_backlight:
+-	kfree(buffer);
+-fail_buffer:
+ 	dell_cleanup_rfkill();
+ fail_rfkill:
+ 	platform_device_del(platform_device);
+@@ -2235,7 +2254,6 @@ static void __exit dell_exit(void)
+ 		touchpad_led_exit();
+ 	kbd_led_exit();
+ 	backlight_device_unregister(dell_backlight_device);
+-	kfree(buffer);
+ 	dell_cleanup_rfkill();
+ 	if (platform_device) {
+ 		platform_device_unregister(platform_device);

diff --git a/2901_allocate_buffer_on_heap_rather_than_globally.patch b/2901_allocate_buffer_on_heap_rather_than_globally.patch
new file mode 100644
index 0000000..1912d9f
--- /dev/null
+++ b/2901_allocate_buffer_on_heap_rather_than_globally.patch
@@ -0,0 +1,458 @@
+--- dell-laptop.c.orig	2018-02-28 19:24:04.598049000 +0000
++++ linux-4.15.0/drivers/platform/x86/dell-laptop.c	2018-02-28 19:40:00.358049000 +0000
+@@ -78,7 +78,6 @@ static struct platform_driver platform_d
+ 	}
+ };
+ 
+-static struct calling_interface_buffer *buffer;
+ static struct platform_device *platform_device;
+ static struct backlight_device *dell_backlight_device;
+ static struct rfkill *wifi_rfkill;
+@@ -286,7 +285,8 @@ static const struct dmi_system_id dell_q
+ 	{ }
+ };
+ 
+-void dell_set_arguments(u32 arg0, u32 arg1, u32 arg2, u32 arg3)
++void dell_set_arguments(struct calling_interface_buffer *buffer,
++        u32 arg0, u32 arg1, u32 arg2, u32 arg3)
+ {
+ 	memset(buffer, 0, sizeof(struct calling_interface_buffer));
+ 	buffer->input[0] = arg0;
+@@ -295,7 +295,8 @@ void dell_set_arguments(u32 arg0, u32 ar
+ 	buffer->input[3] = arg3;
+ }
+ 
+-int dell_send_request(u16 class, u16 select)
++int dell_send_request(struct calling_interface_buffer *buffer,
++        u16 class, u16 select)
+ {
+ 	int ret;
+ 
+@@ -432,21 +433,22 @@ static int dell_rfkill_set(void *data, b
+ 	int disable = blocked ? 1 : 0;
+ 	unsigned long radio = (unsigned long)data;
+ 	int hwswitch_bit = (unsigned long)data - 1;
++	struct calling_interface_buffer buffer;
+ 	int hwswitch;
+ 	int status;
+ 	int ret;
+ 
+-	dell_set_arguments(0, 0, 0, 0);
+-	ret = dell_send_request(CLASS_INFO, SELECT_RFKILL);
++	dell_fill_request(&buffer, 0, 0, 0, 0);
++	ret = dell_send_request(&buffer, CLASS_INFO, SELECT_RFKILL);
+ 	if (ret)
+ 		return ret;
+-	status = buffer->output[1];
++	status = buffer.output[1];
+ 
+-	dell_set_arguments(0x2, 0, 0, 0);
+-	ret = dell_send_request(CLASS_INFO, SELECT_RFKILL);
++	dell_fill_request(&buffer, 0x2, 0, 0, 0);
++	ret = dell_send_request(&buffer, CLASS_INFO, SELECT_RFKILL);
+ 	if (ret)
+ 		return ret;
+-	hwswitch = buffer->output[1];
++	hwswitch = buffer.output[1];
+ 
+ 	/* If the hardware switch controls this radio, and the hardware
+ 	   switch is disabled, always disable the radio */
+@@ -454,8 +456,8 @@ static int dell_rfkill_set(void *data, b
+ 	    (status & BIT(0)) && !(status & BIT(16)))
+ 		disable = 1;
+ 
+-	dell_set_arguments(1 | (radio<<8) | (disable << 16), 0, 0, 0);
+-	ret = dell_send_request(CLASS_INFO, SELECT_RFKILL);
++	dell_fill_request(&buffer, 1 | (radio<<8) | (disable << 16), 0, 0, 0);
++	ret = dell_send_request(&buffer, CLASS_INFO, SELECT_RFKILL);
+ 	return ret;
+ }
+ 
+@@ -464,9 +466,11 @@ static void dell_rfkill_update_sw_state(
+ {
+ 	if (status & BIT(0)) {
+ 		/* Has hw-switch, sync sw_state to BIOS */
++		struct calling_interface_buffer buffer;
+ 		int block = rfkill_blocked(rfkill);
+-		dell_set_arguments(1 | (radio << 8) | (block << 16), 0, 0, 0);
+-		dell_send_request(CLASS_INFO, SELECT_RFKILL);
++		dell_fill_request(&buffer,
++				   1 | (radio << 8) | (block << 16), 0, 0, 0);
++		dell_send_request(&buffer, CLASS_INFO, SELECT_RFKILL);
+ 	} else {
+ 		/* No hw-switch, sync BIOS state to sw_state */
+ 		rfkill_set_sw_state(rfkill, !!(status & BIT(radio + 16)));
+@@ -483,21 +487,22 @@ static void dell_rfkill_update_hw_state(
+ static void dell_rfkill_query(struct rfkill *rfkill, void *data)
+ {
+ 	int radio = ((unsigned long)data & 0xF);
++	struct calling_interface_buffer buffer;
+ 	int hwswitch;
+ 	int status;
+ 	int ret;
+ 
+-	dell_set_arguments(0, 0, 0, 0);
+-	ret = dell_send_request(CLASS_INFO, SELECT_RFKILL);
+-	status = buffer->output[1];
++	dell_fill_request(&buffer, 0, 0, 0, 0);
++	ret = dell_send_request(&buffer, CLASS_INFO, SELECT_RFKILL);
++	status = buffer.output[1];
+ 
+ 	if (ret != 0 || !(status & BIT(0))) {
+ 		return;
+ 	}
+ 
+-	dell_set_arguments(0, 0x2, 0, 0);
+-	ret = dell_send_request(CLASS_INFO, SELECT_RFKILL);
+-	hwswitch = buffer->output[1];
++	dell_fill_request(&buffer, 0, 0x2, 0, 0);
++	ret = dell_send_request(&buffer, CLASS_INFO, SELECT_RFKILL);
++	hwswitch = buffer.output[1];
+ 
+ 	if (ret != 0)
+ 		return;
+@@ -514,22 +519,23 @@ static struct dentry *dell_laptop_dir;
+ 
+ static int dell_debugfs_show(struct seq_file *s, void *data)
+ {
++	struct calling_interface_buffer buffer;
+ 	int hwswitch_state;
+ 	int hwswitch_ret;
+ 	int status;
+ 	int ret;
+ 
+-	dell_set_arguments(0, 0, 0, 0);
+-	ret = dell_send_request(CLASS_INFO, SELECT_RFKILL);
++	dell_fill_request(&buffer, 0, 0, 0, 0);
++	ret = dell_send_request(&buffer, CLASS_INFO, SELECT_RFKILL);
+ 	if (ret)
+ 		return ret;
+-	status = buffer->output[1];
++	status = buffer.output[1];
+ 
+-	dell_set_arguments(0, 0x2, 0, 0);
+-	hwswitch_ret = dell_send_request(CLASS_INFO, SELECT_RFKILL);
++	dell_fill_request(&buffer, 0, 0x2, 0, 0);
++	hwswitch_ret = dell_send_request(&buffer, CLASS_INFO, SELECT_RFKILL);
+ 	if (hwswitch_ret)
+ 		return hwswitch_ret;
+-	hwswitch_state = buffer->output[1];
++	hwswitch_state = buffer.output[1];
+ 
+ 	seq_printf(s, "return:\t%d\n", ret);
+ 	seq_printf(s, "status:\t0x%X\n", status);
+@@ -610,22 +616,23 @@ static const struct file_operations dell
+ 
+ static void dell_update_rfkill(struct work_struct *ignored)
+ {
++	struct calling_interface_buffer buffer;
+ 	int hwswitch = 0;
+ 	int status;
+ 	int ret;
+ 
+-	dell_set_arguments(0, 0, 0, 0);
+-	ret = dell_send_request(CLASS_INFO, SELECT_RFKILL);
+-	status = buffer->output[1];
++	dell_fill_request(&buffer, 0, 0, 0, 0);
++	ret = dell_send_request(&buffer, CLASS_INFO, SELECT_RFKILL);
++	status = buffer.output[1];
+ 
+ 	if (ret != 0)
+ 		return;
+ 
+-	dell_set_arguments(0, 0x2, 0, 0);
+-	ret = dell_send_request(CLASS_INFO, SELECT_RFKILL);
++	dell_fill_request(&buffer, 0, 0x2, 0, 0);
++	ret = dell_send_request(&buffer, CLASS_INFO, SELECT_RFKILL);
+ 
+ 	if (ret == 0 && (status & BIT(0)))
+-		hwswitch = buffer->output[1];
++		hwswitch = buffer.output[1];
+ 
+ 	if (wifi_rfkill) {
+ 		dell_rfkill_update_hw_state(wifi_rfkill, 1, status, hwswitch);
+@@ -683,6 +690,7 @@ static struct notifier_block dell_laptop
+ 
+ static int __init dell_setup_rfkill(void)
+ {
++	struct calling_interface_buffer buffer;
+ 	int status, ret, whitelisted;
+ 	const char *product;
+ 
+@@ -698,9 +706,9 @@ static int __init dell_setup_rfkill(void
+ 	if (!force_rfkill && !whitelisted)
+ 		return 0;
+ 
+-	dell_set_arguments(0, 0, 0, 0);
+-	ret = dell_send_request(CLASS_INFO, SELECT_RFKILL);
+-	status = buffer->output[1];
++	dell_fill_request(&buffer, 0, 0, 0, 0);
++	ret = dell_send_request(&buffer, CLASS_INFO, SELECT_RFKILL);
++	status = buffer.output[1];
+ 
+ 	/* dell wireless info smbios call is not supported */
+ 	if (ret != 0)
+@@ -853,6 +861,7 @@ static void dell_cleanup_rfkill(void)
+ 
+ static int dell_send_intensity(struct backlight_device *bd)
+ {
++	struct calling_interface_buffer buffer;
+ 	struct calling_interface_token *token;
+ 	int ret;
+ 
+@@ -860,17 +869,21 @@ static int dell_send_intensity(struct ba
+ 	if (!token)
+ 		return -ENODEV;
+ 
+-	dell_set_arguments(token->location, bd->props.brightness, 0, 0);
++	dell_fill_request(&buffer,
++			   token->location, bd->props.brightness, 0, 0);
+ 	if (power_supply_is_system_supplied() > 0)
+-		ret = dell_send_request(CLASS_TOKEN_WRITE, SELECT_TOKEN_AC);
++		ret = dell_send_request(&buffer,
++					CLASS_TOKEN_WRITE, SELECT_TOKEN_AC);
+ 	else
+-		ret = dell_send_request(CLASS_TOKEN_WRITE, SELECT_TOKEN_BAT);
++		ret = dell_send_request(&buffer,
++					CLASS_TOKEN_WRITE, SELECT_TOKEN_BAT);
+ 
+ 	return ret;
+ }
+ 
+ static int dell_get_intensity(struct backlight_device *bd)
+ {
++	struct calling_interface_buffer buffer;
+ 	struct calling_interface_token *token;
+ 	int ret;
+ 
+@@ -878,14 +891,17 @@ static int dell_get_intensity(struct bac
+ 	if (!token)
+ 		return -ENODEV;
+ 
+-	dell_set_arguments(token->location, 0, 0, 0);
++	dell_fill_request(&buffer, token->location, 0, 0, 0);
+ 	if (power_supply_is_system_supplied() > 0)
+-		ret = dell_send_request(CLASS_TOKEN_READ, SELECT_TOKEN_AC);
++		ret = dell_send_request(&buffer,
++					CLASS_TOKEN_READ, SELECT_TOKEN_AC);
+ 	else
+-		ret = dell_send_request(CLASS_TOKEN_READ, SELECT_TOKEN_BAT);
++		ret = dell_send_request(&buffer,
++					CLASS_TOKEN_READ, SELECT_TOKEN_BAT);
+ 
+ 	if (ret == 0)
+-		ret = buffer->output[1];
++		ret = buffer.output[1];
++
+ 	return ret;
+ }
+ 
+@@ -1149,31 +1165,33 @@ static DEFINE_MUTEX(kbd_led_mutex);
+ 
+ static int kbd_get_info(struct kbd_info *info)
+ {
++	struct calling_interface_buffer buffer;
+ 	u8 units;
+ 	int ret;
+ 
+-	dell_set_arguments(0, 0, 0, 0);
+-	ret = dell_send_request(CLASS_KBD_BACKLIGHT, SELECT_KBD_BACKLIGHT);
++	dell_fill_request(&buffer, 0, 0, 0, 0);
++	ret = dell_send_request(&buffer,
++				CLASS_KBD_BACKLIGHT, SELECT_KBD_BACKLIGHT);
+ 	if (ret)
+ 		return ret;
+ 
+-	info->modes = buffer->output[1] & 0xFFFF;
+-	info->type = (buffer->output[1] >> 24) & 0xFF;
+-	info->triggers = buffer->output[2] & 0xFF;
+-	units = (buffer->output[2] >> 8) & 0xFF;
+-	info->levels = (buffer->output[2] >> 16) & 0xFF;
++	info->modes = buffer.output[1] & 0xFFFF;
++	info->type = (buffer.output[1] >> 24) & 0xFF;
++	info->triggers = buffer.output[2] & 0xFF;
++	units = (buffer.output[2] >> 8) & 0xFF;
++	info->levels = (buffer.output[2] >> 16) & 0xFF;
+ 
+ 	if (quirks && quirks->kbd_led_levels_off_1 && info->levels)
+ 		info->levels--;
+ 
+ 	if (units & BIT(0))
+-		info->seconds = (buffer->output[3] >> 0) & 0xFF;
++		info->seconds = (buffer.output[3] >> 0) & 0xFF;
+ 	if (units & BIT(1))
+-		info->minutes = (buffer->output[3] >> 8) & 0xFF;
++		info->minutes = (buffer.output[3] >> 8) & 0xFF;
+ 	if (units & BIT(2))
+-		info->hours = (buffer->output[3] >> 16) & 0xFF;
++		info->hours = (buffer.output[3] >> 16) & 0xFF;
+ 	if (units & BIT(3))
+-		info->days = (buffer->output[3] >> 24) & 0xFF;
++		info->days = (buffer.output[3] >> 24) & 0xFF;
+ 
+ 	return ret;
+ }
+@@ -1233,31 +1251,34 @@ static int kbd_set_level(struct kbd_stat
+ 
+ static int kbd_get_state(struct kbd_state *state)
+ {
++	struct calling_interface_buffer buffer;
+ 	int ret;
+ 
+-	dell_set_arguments(0x1, 0, 0, 0);
+-	ret = dell_send_request(CLASS_KBD_BACKLIGHT, SELECT_KBD_BACKLIGHT);
++	dell_fill_request(&buffer, 0, 0, 0, 0);
++	ret = dell_send_request(&buffer,
++				CLASS_KBD_BACKLIGHT, SELECT_KBD_BACKLIGHT);
+ 	if (ret)
+ 		return ret;
+ 
+-	state->mode_bit = ffs(buffer->output[1] & 0xFFFF);
++	state->mode_bit = ffs(buffer.output[1] & 0xFFFF);
+ 	if (state->mode_bit != 0)
+ 		state->mode_bit--;
+ 
+-	state->triggers = (buffer->output[1] >> 16) & 0xFF;
+-	state->timeout_value = (buffer->output[1] >> 24) & 0x3F;
+-	state->timeout_unit = (buffer->output[1] >> 30) & 0x3;
+-	state->als_setting = buffer->output[2] & 0xFF;
+-	state->als_value = (buffer->output[2] >> 8) & 0xFF;
+-	state->level = (buffer->output[2] >> 16) & 0xFF;
+-	state->timeout_value_ac = (buffer->output[2] >> 24) & 0x3F;
+-	state->timeout_unit_ac = (buffer->output[2] >> 30) & 0x3;
++	state->triggers = (buffer.output[1] >> 16) & 0xFF;
++	state->timeout_value = (buffer.output[1] >> 24) & 0x3F;
++	state->timeout_unit = (buffer.output[1] >> 30) & 0x3;
++	state->als_setting = buffer.output[2] & 0xFF;
++	state->als_value = (buffer.output[2] >> 8) & 0xFF;
++	state->level = (buffer.output[2] >> 16) & 0xFF;
++	state->timeout_value_ac = (buffer.output[2] >> 24) & 0x3F;
++	state->timeout_unit_ac = (buffer.output[2] >> 30) & 0x3;
+ 
+ 	return ret;
+ }
+ 
+ static int kbd_set_state(struct kbd_state *state)
+ {
++	struct calling_interface_buffer buffer;
+ 	int ret;
+ 	u32 input1;
+ 	u32 input2;
+@@ -1270,8 +1291,9 @@ static int kbd_set_state(struct kbd_stat
+ 	input2 |= (state->level & 0xFF) << 16;
+ 	input2 |= (state->timeout_value_ac & 0x3F) << 24;
+ 	input2 |= (state->timeout_unit_ac & 0x3) << 30;
+-	dell_set_arguments(0x2, input1, input2, 0);
+-	ret = dell_send_request(CLASS_KBD_BACKLIGHT, SELECT_KBD_BACKLIGHT);
++	dell_fill_request(&buffer, 0x2, input1, input2, 0);
++	ret = dell_send_request(&buffer,
++				CLASS_KBD_BACKLIGHT, SELECT_KBD_BACKLIGHT);
+ 
+ 	return ret;
+ }
+@@ -1298,6 +1320,7 @@ static int kbd_set_state_safe(struct kbd
+ 
+ static int kbd_set_token_bit(u8 bit)
+ {
++	struct calling_interface_buffer buffer;
+ 	struct calling_interface_token *token;
+ 	int ret;
+ 
+@@ -1308,14 +1331,15 @@ static int kbd_set_token_bit(u8 bit)
+ 	if (!token)
+ 		return -EINVAL;
+ 
+-	dell_set_arguments(token->location, token->value, 0, 0);
+-	ret = dell_send_request(CLASS_TOKEN_WRITE, SELECT_TOKEN_STD);
++	dell_fill_request(&buffer, token->location, token->value, 0, 0);
++	ret = dell_send_request(&buffer, CLASS_TOKEN_WRITE, SELECT_TOKEN_STD);
+ 
+ 	return ret;
+ }
+ 
+ static int kbd_get_token_bit(u8 bit)
+ {
++	struct calling_interface_buffer buffer;
+ 	struct calling_interface_token *token;
+ 	int ret;
+ 	int val;
+@@ -1327,9 +1351,9 @@ static int kbd_get_token_bit(u8 bit)
+ 	if (!token)
+ 		return -EINVAL;
+ 
+-	dell_set_arguments(token->location, 0, 0, 0);
+-	ret = dell_send_request(CLASS_TOKEN_READ, SELECT_TOKEN_STD);
+-	val = buffer->output[1];
++	dell_fill_request(&buffer, token->location, 0, 0, 0);
++	ret = dell_send_request(&buffer, CLASS_TOKEN_READ, SELECT_TOKEN_STD);
++	val = buffer.output[1];
+ 
+ 	if (ret)
+ 		return ret;
+@@ -2046,6 +2070,7 @@ static struct notifier_block dell_laptop
+ 
+ int dell_micmute_led_set(int state)
+ {
++	struct calling_interface_buffer buffer;
+ 	struct calling_interface_token *token;
+ 
+ 	if (state == 0)
+@@ -2058,8 +2083,8 @@ int dell_micmute_led_set(int state)
+ 	if (!token)
+ 		return -ENODEV;
+ 
+-	dell_set_arguments(token->location, token->value, 0, 0);
+-	dell_send_request(CLASS_TOKEN_WRITE, SELECT_TOKEN_STD);
++	dell_fill_request(&buffer, token->location, token->value, 0, 0);
++	dell_send_request(&buffer, CLASS_TOKEN_WRITE, SELECT_TOKEN_STD);
+ 
+ 	return state;
+ }
+@@ -2090,13 +2115,6 @@ static int __init dell_init(void)
+ 	if (ret)
+ 		goto fail_platform_device2;
+ 
+-	buffer = kzalloc(sizeof(struct calling_interface_buffer), GFP_KERNEL);
+-	if (!buffer) {
+-		ret = -ENOMEM;
+-		goto fail_buffer;
+-	}
+-
+-
+ 	ret = dell_setup_rfkill();
+ 
+ 	if (ret) {
+@@ -2121,10 +2139,13 @@ static int __init dell_init(void)
+ 
+ 	token = dell_smbios_find_token(BRIGHTNESS_TOKEN);
+ 	if (token) {
+-		dell_set_arguments(token->location, 0, 0, 0);
+-		ret = dell_send_request(CLASS_TOKEN_READ, SELECT_TOKEN_AC);
++		struct calling_interface_buffer buffer;
++
++		dell_fill_request(&buffer, token->location, 0, 0, 0);
++		ret = dell_send_request(&buffer,
++					CLASS_TOKEN_READ, SELECT_TOKEN_AC);
+ 		if (ret)
+-			max_intensity = buffer->output[3];
++			max_intensity = buffer.output[3];
+ 	}
+ 
+ 	if (max_intensity) {
+@@ -2158,8 +2179,6 @@ static int __init dell_init(void)
+ fail_get_brightness:
+ 	backlight_device_unregister(dell_backlight_device);
+ fail_backlight:
+-	kfree(buffer);
+-fail_buffer:
+ 	dell_cleanup_rfkill();
+ fail_rfkill:
+ 	platform_device_del(platform_device);
+@@ -2179,7 +2198,6 @@ static void __exit dell_exit(void)
+ 		touchpad_led_exit();
+ 	kbd_led_exit();
+ 	backlight_device_unregister(dell_backlight_device);
+-	kfree(buffer);
+ 	dell_cleanup_rfkill();
+ 	if (platform_device) {
+ 		platform_device_unregister(platform_device);