Message ID | 1510959741-31109-7-git-send-email-yang.s@alibaba-inc.com |
---|---|
State | Not Applicable, archived |
Delegated to: | David Miller |
Headers | show |
Series | [1/8] mm: kmemleak: remove unused hardirq.h | expand |
It looks the email address of Pravin in MAINTAINERS file is obsolete, sent to the right address. Yang On 11/17/17 3:02 PM, Yang Shi wrote: > Preempt counter APIs have been split out, currently, hardirq.h just > includes irq_enter/exit APIs which are not used by openvswitch at all. > > So, remove the unused hardirq.h. > > Signed-off-by: Yang Shi <yang.s@alibaba-inc.com> > Cc: Pravin Shelar <pshelar@nicira.com> > Cc: "David S. Miller" <davem@davemloft.net> > Cc: dev@openvswitch.org > --- > net/openvswitch/vport-internal_dev.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/net/openvswitch/vport-internal_dev.c b/net/openvswitch/vport-internal_dev.c > index 04a3128..2f47c65 100644 > --- a/net/openvswitch/vport-internal_dev.c > +++ b/net/openvswitch/vport-internal_dev.c > @@ -16,7 +16,6 @@ > * 02110-1301, USA > */ > > -#include <linux/hardirq.h> > #include <linux/if_vlan.h> > #include <linux/kernel.h> > #include <linux/netdevice.h> >
Hi folks, Any comment on this one? Thanks, Yang On 11/17/17 5:48 PM, Yang Shi wrote: > It looks the email address of Pravin in MAINTAINERS file is obsolete, > sent to the right address. > > Yang > > > On 11/17/17 3:02 PM, Yang Shi wrote: >> Preempt counter APIs have been split out, currently, hardirq.h just >> includes irq_enter/exit APIs which are not used by openvswitch at all. >> >> So, remove the unused hardirq.h. >> >> Signed-off-by: Yang Shi <yang.s@alibaba-inc.com> >> Cc: Pravin Shelar <pshelar@nicira.com> >> Cc: "David S. Miller" <davem@davemloft.net> >> Cc: dev@openvswitch.org >> --- >> net/openvswitch/vport-internal_dev.c | 1 - >> 1 file changed, 1 deletion(-) >> >> diff --git a/net/openvswitch/vport-internal_dev.c >> b/net/openvswitch/vport-internal_dev.c >> index 04a3128..2f47c65 100644 >> --- a/net/openvswitch/vport-internal_dev.c >> +++ b/net/openvswitch/vport-internal_dev.c >> @@ -16,7 +16,6 @@ >> * 02110-1301, USA >> */ >> -#include <linux/hardirq.h> >> #include <linux/if_vlan.h> >> #include <linux/kernel.h> >> #include <linux/netdevice.h> >>
On Fri, Nov 17, 2017 at 3:02 PM, Yang Shi <yang.s@alibaba-inc.com> wrote: > Preempt counter APIs have been split out, currently, hardirq.h just > includes irq_enter/exit APIs which are not used by openvswitch at all. > > So, remove the unused hardirq.h. > > Signed-off-by: Yang Shi <yang.s@alibaba-inc.com> > Cc: Pravin Shelar <pshelar@nicira.com> > Cc: "David S. Miller" <davem@davemloft.net> > Cc: dev@openvswitch.org Acked-by: Pravin B Shelar <pshelar@ovn.org>
Hi David, Any comment is appreciated. Thanks, Yang On 12/7/17 11:27 AM, Pravin Shelar wrote: > On Fri, Nov 17, 2017 at 3:02 PM, Yang Shi <yang.s@alibaba-inc.com> wrote: >> Preempt counter APIs have been split out, currently, hardirq.h just >> includes irq_enter/exit APIs which are not used by openvswitch at all. >> >> So, remove the unused hardirq.h. >> >> Signed-off-by: Yang Shi <yang.s@alibaba-inc.com> >> Cc: Pravin Shelar <pshelar@nicira.com> >> Cc: "David S. Miller" <davem@davemloft.net> >> Cc: dev@openvswitch.org > > Acked-by: Pravin B Shelar <pshelar@ovn.org> >
diff --git a/net/openvswitch/vport-internal_dev.c b/net/openvswitch/vport-internal_dev.c index 04a3128..2f47c65 100644 --- a/net/openvswitch/vport-internal_dev.c +++ b/net/openvswitch/vport-internal_dev.c @@ -16,7 +16,6 @@ * 02110-1301, USA */ -#include <linux/hardirq.h> #include <linux/if_vlan.h> #include <linux/kernel.h> #include <linux/netdevice.h>
Preempt counter APIs have been split out, currently, hardirq.h just includes irq_enter/exit APIs which are not used by openvswitch at all. So, remove the unused hardirq.h. Signed-off-by: Yang Shi <yang.s@alibaba-inc.com> Cc: Pravin Shelar <pshelar@nicira.com> Cc: "David S. Miller" <davem@davemloft.net> Cc: dev@openvswitch.org --- net/openvswitch/vport-internal_dev.c | 1 - 1 file changed, 1 deletion(-)