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

From: Luca Coelho
Date: Mon Oct 05 2015 - 09:01:33 EST


Hi Denys,

On Thu, 2015-09-24 at 16:58 +0200, Denys Vlasenko wrote:
> With this .config: http://busybox.net/~vda/kernel_config_ALLYES_Os,
> after deinlining these functions have sizes and callsite counts
> as follows:
>
> iwl_write8: 315 bytes, 3 calls
> iwl_write32: 296 bytes, 90 calls
> iwl_read32: 278 bytes, 51 calls
>
> Total size reduction is about 40 kbytes.
>
> Signed-off-by: Denys Vlasenko <dvlasenk@xxxxxxxxxx>
> CC: Johannes Berg <johannes.berg@xxxxxxxxx>
> CC: Emmanuel Grumbach <emmanuel.grumbach@xxxxxxxxx>
> CC: Intel Linux Wireless <ilw@xxxxxxxxxxxxxxx>
> CC: Gregory Greenman <gregory.greenman@xxxxxxxxx>
> CC: John Linville <linville@xxxxxxxxxxxxx>
> CC: linux-wireless@xxxxxxxxxxxxxxx
> CC: linux-kernel@xxxxxxxxxxxxxxx

Emmanuel already wrote and applied a patch like this. Check here:
http://mid.gmane.org/1442869141-31637-4-git-send-email-luca@xxxxxxxxx

Thanks for your patch anyway and for reporting this. :)

--
Cheers,
Luca.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/