From: Jiri Pirko <jpirko@xxxxxxxxxx> Date: Wed, 16 Mar 2011 09:40:23 +0100 > Wed, Mar 16, 2011 at 08:38:03AM CET, jpirko@xxxxxxxxxx wrote: >>Wed, Mar 16, 2011 at 08:02:51AM CET, davem@xxxxxxxxxxxxx wrote: >>>From: Jiri Pirko <jpirko@xxxxxxxxxx> >>>Date: Wed, 16 Mar 2011 07:50:09 +0100 >>> >>>>>Jiri, I know there is your patch set there, but I think you and Changli >>>>>still need to go back and forth one more time wrt. orig_dev semantics. >>>>>Since you have been posting this patch set for some time I'm still >>>>>willing to apply it for this merge window, but please make haste and >>>>>work out the remaining discussion. Thank you. >>>> >>>> Dave, you can apply the rest of the series and leave only the first >>>> patch (af_packet) out. There's no dependency. We will figure out things >>>> around origdev later. >>> >>>I was about to do that but I've found other problems. >>> >>>You cannot make the modifications you make to linux/if.h, those >>>interface flags are visible to userspace. >> >>What do you suggest? To remove unused flags and leave gaps there or to >>not to remove the flags at all? > > Well, the following comment: > <quote> > /* Private (from user) interface flags (netdevice->priv_flags). */ > </quote> > leads me to think that these flags should not be used by userspace. > So maybe it may not be problem to change those values. Comments don't block user applications from using defines we expose to them. The person who was wise enough to write that comment should have also been wise enough to add appropriate __KERNEL__ protection to the definitions. They were not, therefore we are stuck with them forever. -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
