Re: [PATCH] fbdev: Constify struct sbus_mmap_map

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 10/13/24 13:48, Christophe JAILLET wrote:
'struct sbus_mmap_map' are not modified in these drivers.

Constifying this structure moves some data to a read-only section, so
increases overall security.

Update sbusfb_mmap_helper() accordingly.

On a x86_64, with allmodconfig, as an example:
Before:
======
    text	   data	    bss	    dec	    hex	filename
    2452	    536	     16	   3004	    bbc	drivers/video/fbdev/bw2.o

After:
=====
    text	   data	    bss	    dec	    hex	filename
    2500	    483	     16	   2999	    bb7	drivers/video/fbdev/bw2.o

Signed-off-by: Christophe JAILLET <christophe.jaillet@xxxxxxxxxx>
---
Compile tested only with a cross compiler for sparc.
---
  drivers/video/fbdev/bw2.c     | 2 +-
  drivers/video/fbdev/cg14.c    | 2 +-
  drivers/video/fbdev/cg3.c     | 2 +-
  drivers/video/fbdev/cg6.c     | 2 +-
  drivers/video/fbdev/ffb.c     | 2 +-
  drivers/video/fbdev/leo.c     | 2 +-
  drivers/video/fbdev/p9100.c   | 2 +-
  drivers/video/fbdev/sbuslib.c | 2 +-
  drivers/video/fbdev/sbuslib.h | 2 +-
  drivers/video/fbdev/tcx.c     | 2 +-
  10 files changed, 10 insertions(+), 10 deletions(-)

applied.

Thanks!
Helge





[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux