Re: [PATCH] hv: mark hv_driver read only after init

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

 



On Mon, 15 Aug 2016 09:11:38 -0700
Kees Cook <keescook@xxxxxxxxxxxx> wrote:

> On Mon, Aug 15, 2016 at 8:41 AM, Stephen Hemminger
> <stephen@xxxxxxxxxxxxxxxxxx> wrote:
> > On Fri, 12 Aug 2016 20:05:27 -0700
> > Kees Cook <keescook@xxxxxxxxxxxx> wrote:
> >  
> >> On Fri, Aug 12, 2016 at 4:35 PM, Stephen Hemminger
> >> <stephen@xxxxxxxxxxxxxxxxxx> wrote:  
> >> >
> >> > For hardening, the driver structure containing function pointers can
> >> > be marked read only after initial registration is done.  
> >>
> >> Yay more __ro_after_init! (Minor nit below...)
> >>
> >> Also, I wonder if there's a way to do build-time section checking,
> >> e.g. adding "is this hv_driver marked __ro_after_init?" in
> >> __vmbus_driver_register...  
> >
> > Is there a macro or tool to check this?  
> 
> There isn't, no. That's what I was suggesting might be useful to create. :)
> 
> -Kees
> 

Maybe a sparse annotation like __rcu?
I am not a sparse expert maybe someone else is.
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel



[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux