Re: [PATCH] interconnect: qcom: Constify struct qcom_icc_qosbox

Dmitry Baryshkov <[email protected]>
Newsgroups org.kernel.vger.linux-pm,org.kernel.vger.kernel-janitors,org.kernel.vger.linux-arm-msm,org.kernel.vger.linux-kernel
Message-ID <5nhxxcayafpu2gknp6hazzyvj5q363eqooucfijbmz6oxftkjd@b4rs6snqkhxt>
On Wed, Aug 12, 2026 at 07:08:42PM +0200, Christophe JAILLET wrote:
> 'struct qcom_icc_qosbox' are not modified in these drivers.
> 
> Constifying these structures moves some data to a read-only section, so
> increases overall security.
> 
> On a x86_64, with allmodconfig:
> Before:
> ======
>    text	   data	    bss	    dec	    hex	filename
>    7807	  18632	      0	  26439	   6747	drivers/interconnect/qcom/eliza.o
>   10027	  20176	      0	  30203	   75fb	drivers/interconnect/qcom/milos.o
>    8768	  22640	      0	  31408	   7ab0	drivers/interconnect/qcom/sm6350.o
>    7626	  22000	      0	  29626	   73ba	drivers/interconnect/qcom/sm8650.o
> 
> After:
> =====
>    text	   data	    bss	    dec	    hex	filename
>    8095	  18344	      0	  26439	   6747	drivers/interconnect/qcom/eliza.o
>   11079	  19120	      0	  30199	   75f7	drivers/interconnect/qcom/milos.o
>    9668	  21744	      0	  31412	   7ab4	drivers/interconnect/qcom/sm6350.o
>    8746	  20880	      0	  29626	   73ba	drivers/interconnect/qcom/sm8650.o
> 
> Signed-off-by: Christophe JAILLET <[email protected]>
> ---
> Compile tested only.
> ---
>  drivers/interconnect/qcom/eliza.c  | 18 ++++----
>  drivers/interconnect/qcom/milos.c  | 66 ++++++++++++++--------------
>  drivers/interconnect/qcom/sm6350.c | 56 ++++++++++++------------
>  drivers/interconnect/qcom/sm8650.c | 70 +++++++++++++++---------------
>  4 files changed, 105 insertions(+), 105 deletions(-)
> 

Reviewed-by: Dmitry Baryshkov <[email protected]>


-- 
With best wishes
Dmitry
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.