[Cocci] [PATCH] compat/compat-drivers/linux-next: fb skip_vt_switch
julia.lawall at lip6.fr
Thu Mar 28 17:19:59 CET 2013
On Thu, 28 Mar 2013, Jesse Barnes wrote:
> On Thu, 28 Mar 2013 05:04:26 -0700
> "Luis R. Rodriguez" <mcgrof at do-not-panic.com> wrote:
> > The new commit by Jesse that extended the fb_info with a skip_vt_switch
> > element is the simplest example of a data structure expansion. We'd backport
> > this by adding a static inline to compat so that new kernels muck with the
> > new element and for older kernels this would be a no-op. This reduces the
> > size of the backport and unclutters the required patch with #idefs, and
> > insteads leaves only a replacement of the usage of the new elements with
> > a static inline, this however would still be required on our end:
> > - info->skip_vt_switch = true;
> > + fb_enable_skip_vt_switch(info);
> > So we'd then have to just add this static inline change for each new driver...
> > There may be a way to get SmPL to do this for us...
type of info *info;
- info->skip_vt_switch = true;
for whatever the type of info is.
Then I guess there would be a similar rule for the false case?
> Yeah I'm not attached to the direct structure reference; a couple of
> inlines are just as easy to read. So no argument from me.
> Jesse Barnes, Intel Open Source Technology Center
More information about the Cocci