[PATCH] iwlwifi: Deinline iwl_{read,write}{8,32}

2015-07-14 Thread Denys Vlasenko
callsites iwl_write8 493 bytes, 3 callsites Reduction in size is about 74,000 bytes: text data bss dec hex filename 90758147 17226024 36659200 144643371 89f152b vmlinux0 90687995 17221928 36659200 144569123 89df323 vmlinux.after Signed-off-by: Denys Vlasenko dvlas...@redhat.com

[PATCH] mac80211: Deinline drv_sta_state

2015-07-15 Thread Denys Vlasenko
With this .config: http://busybox.net/~vda/kernel_config, after deinlining the function size is 3132 bytes and there are 7 callsites. Total size reduction: about 20 kbytes. Signed-off-by: Denys Vlasenko dvlas...@redhat.com CC: John Linville linvi...@tuxdriver.com CC: Michal Kazior michal.kaz

[PATCH] mac80211: Deinline rate_control_rate_init, rate_control_rate_update

2015-07-15 Thread Denys Vlasenko
With this .config: http://busybox.net/~vda/kernel_config, after deinlining these functions have sizes and callsite counts as follows: rate_control_rate_init: 554 bytes, 8 calls rate_control_rate_update: 1596 bytes, 5 calls Total size reduction: about 11 kbytes. Signed-off-by: Denys Vlasenko

[PATCH 2/3] mac80211: Deinline drv_ampdu_action()

2015-09-23 Thread Denys Vlasenko
With this .config: http://busybox.net/~vda/kernel_config_ALLYES_Os, after deinlining the function size is 755 bytes and there are 6 callsites. Total size reduction is about 3.3 kbytes. Signed-off-by: Denys Vlasenko <dvlas...@redhat.com> CC: Johannes Berg <johannes.b...@intel.com&

[PATCH 3/3] mac80211: Deinline drv_get/set/reset_tsf()

2015-09-23 Thread Denys Vlasenko
-by: Denys Vlasenko <dvlas...@redhat.com> CC: Johannes Berg <johannes.b...@intel.com> CC: John Linville <linvi...@tuxdriver.com> CC: Michal Kazior <michal.kaz...@tieto.com> CC: linux-wireless@vger.kernel.org CC: linux-ker...@vger.kernel.org --- Rediffed against mac80211-ne

[PATCH 1/3] mac80211: Deinline drv_switch_vif_chanctx()

2015-09-23 Thread Denys Vlasenko
With this .config: http://busybox.net/~vda/kernel_config_ALLYES_Os, after deinlining the function size is 821 bytes and there are 2 callsites, reducing code size by about 800 bytes. Signed-off-by: Denys Vlasenko <dvlas...@redhat.com> CC: Johannes Berg <johannes.b...@intel.com> CC: J

[PATCH] iwlwifi: Deinline iwl_write8/write32/read32()

2015-09-24 Thread Denys Vlasenko
-by: Denys Vlasenko <dvlas...@redhat.com> CC: Johannes Berg <johannes.b...@intel.com> CC: Emmanuel Grumbach <emmanuel.grumb...@intel.com> CC: Intel Linux Wireless <i...@linux.intel.com> CC: Gregory Greenman <gregory.green...@intel.com> CC: John Linville <linvi...@

[PATCH 3/6] mac80211: Deinline drv_ampdu_action()

2015-09-18 Thread Denys Vlasenko
With this .config: http://busybox.net/~vda/kernel_config_ALLYES_Os, after deinlining the function size is 755 bytes and there are 6 callsites. Total size reduction is about 3.3 kbytes. Signed-off-by: Denys Vlasenko <dvlas...@redhat.com> CC: John Linville <linvi...@tuxdriver.com>

[PATCH 4/6] mac80211: Deinline drv_get/set/reset_tsf()

2015-09-18 Thread Denys Vlasenko
-by: Denys Vlasenko <dvlas...@redhat.com> CC: John Linville <linvi...@tuxdriver.com> CC: Michal Kazior <michal.kaz...@tieto.com> CC: Johannes Berg <johannes.b...@intel.com> CC: linux-wireless@vger.kernel.org CC: linux-ker...@vger.kernel.org --- net/ma

[PATCH 1/6] mac80211: Deinline drv_conf_tx()

2015-09-18 Thread Denys Vlasenko
With this .config: http://busybox.net/~vda/kernel_config_ALLYES_Os, after deinlining the function size is 785 bytes and there are 7 callsites. Total size reduction is about 3.5 kbytes. Signed-off-by: Denys Vlasenko <dvlas...@redhat.com> CC: John Linville <linvi...@tuxdriver.com>

[PATCH 2/6] mac80211: Deinline drv_sta_rc_update()

2015-09-18 Thread Denys Vlasenko
With this .config: http://busybox.net/~vda/kernel_config_ALLYES_Os, after deinlining the function size is 706 bytes and there are 2 callsites, reducing code size by about 700 bytes. Signed-off-by: Denys Vlasenko <dvlas...@redhat.com> CC: John Linville <linvi...@tuxdriver.com> CC: M

[PATCH 5/6] mac80211: Deinline drv_add/remove/change_interface()

2015-09-18 Thread Denys Vlasenko
kbytes. Signed-off-by: Denys Vlasenko <dvlas...@redhat.com> CC: John Linville <linvi...@tuxdriver.com> CC: Michal Kazior <michal.kaz...@tieto.com> CC: Johannes Berg <johannes.b...@intel.com> CC: linux-wireless@vger.kernel.org CC: linux-ker...@vger.kernel.org --- net/ma

[PATCH 0/6] mac80211: Deinline large functions in driver-ops.h

2015-09-18 Thread Denys Vlasenko
vger.kernel.org CC: linux-ker...@vger.kernel.org Denys Vlasenko (6): mac80211: Deinline drv_conf_tx() mac80211: Deinline drv_sta_rc_update() mac80211: Deinline drv_ampdu_action() mac80211: Deinline drv_get/set/reset_tsf() mac80211: Deinline drv_add/remove/change_interface()

[PATCH 6/6] mac80211: Deinline drv_switch_vif_chanctx()

2015-09-18 Thread Denys Vlasenko
With this .config: http://busybox.net/~vda/kernel_config_ALLYES_Os, after deinlining the function size is 821 bytes and there are 2 callsites, reducing code size by about 800 bytes. Signed-off-by: Denys Vlasenko <dvlas...@redhat.com> CC: John Linville <linvi...@tuxdriver.com> CC: M

[PATCH RESEND] NFC: hci: delete unused nfc_llc_get_rx_head_tail_room()

2016-06-05 Thread Denys Vlasenko
tion units anyway, and these functions are not used in their .c file, thus they are never inlined. IOW: "inline" here does not help in any way. Signed-off-by: Denys Vlasenko <dvlas...@redhat.com> CC: Samuel Ortiz <sa...@linux.intel.com> CC: Christophe Ricard <christophe.r

[PATCH] rtlwifi: rtl818x: Deinline indexed IO functions, save 21568 bytes

2016-04-25 Thread Denys Vlasenko
nux_before 95872674 20860320 35991552 152724546 91a6442 vmlinux Signed-off-by: Denys Vlasenko <dvlas...@redhat.com> CC: Larry Finger <larry.fin...@lwfinger.net> CC: Chaoming Li <chaoming...@realsil.com.cn> CC: linux-wireless@vger.kernel.org CC: linux-ker...@vger.kernel.org --- .../

[PATCH] NFC: hci: delete unused nfc_llc_get_rx_head_tail_room()

2016-04-15 Thread Denys Vlasenko
tion units anyway, and these functions are not used in their .c file, thus they are never inlined. IOW: "inline" here does not help in any way. Signed-off-by: Denys Vlasenko <dvlas...@redhat.com> CC: Samuel Ortiz <sa...@linux.intel.com> CC: Christophe Ricard <christophe.r

[PATCH] net: make getname() functions return length rather than use int* parameter

2018-02-12 Thread Denys Vlasenko
not changed. textdata bss dec hex filename 30108430 2633624 873672 33615726 200ef6e vmlinux.before.o 30108109 2633612 873672 33615393 200ee21 vmlinux.o Signed-off-by: Denys Vlasenko <dvlas...@redhat.com> CC: David S. Miller <da...@davemloft.net> CC: linu

[PATCH v2] net: make getname() functions return length rather than use int* parameter

2018-02-12 Thread Denys Vlasenko
not changed. textdata bss dec hex filename 30108430 2633624 873672 33615726 200ef6e vmlinux.before.o 30108109 2633612 873672 33615393 200ee21 vmlinux.o Signed-off-by: Denys Vlasenko <dvlas...@redhat.com> CC: David S. Miller <da...@davemloft.net> CC: linu

Re: [PATCH] net: make getname() functions return length rather than use int* parameter

2018-02-12 Thread Denys Vlasenko
On 02/12/2018 06:47 PM, David Miller wrote: From: Denys Vlasenko <dvlas...@redhat.com> Date: Mon, 12 Feb 2018 15:15:18 +0100 Before: All these functions either return a negative error indicator, or store length of sockaddr into "int *socklen" parameter and return zero o