Hi folks,
Any comment on this one?
Thanks,
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 TIPC at all.
>
> So, remove the unused hardirq.h.
>
> Signed-off-by: Yang Shi <[email protected]>
> Cc: Jon Maloy <[email protected]>
> Cc: Ying Xue <[email protected]>
> Cc: "David S. Miller" <[email protected]>
> ---
> net/tipc/core.h | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/net/tipc/core.h b/net/tipc/core.h
> index 5cc5398..099e072 100644
> --- a/net/tipc/core.h
> +++ b/net/tipc/core.h
> @@ -49,7 +49,6 @@
> #include <linux/uaccess.h>
> #include <linux/interrupt.h>
> #include <linux/atomic.h>
> -#include <asm/hardirq.h>
> #include <linux/netdevice.h>
> #include <linux/in.h>
> #include <linux/list.h>
>
> -----Original Message-----
> From: [email protected] [mailto:netdev-
> [email protected]] On Behalf Of Yang Shi
> Sent: Thursday, December 07, 2017 14:16
> To: [email protected]
> Cc: [email protected]; [email protected]; linux-
> [email protected]; [email protected]; Jon Maloy
> <[email protected]>; Ying Xue <[email protected]>; David S.
> Miller <[email protected]>
> Subject: Re: [PATCH 8/8] net: tipc: remove unused hardirq.h
>
> Hi folks,
>
> Any comment on this one?
If it compiles it is ok with me. Don't know why it was put there in the first place.
///jon
>
> Thanks,
> 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 TIPC at all.
> >
> > So, remove the unused hardirq.h.
> >
> > Signed-off-by: Yang Shi <[email protected]>
> > Cc: Jon Maloy <[email protected]>
> > Cc: Ying Xue <[email protected]>
> > Cc: "David S. Miller" <[email protected]>
> > ---
> > net/tipc/core.h | 1 -
> > 1 file changed, 1 deletion(-)
> >
> > diff --git a/net/tipc/core.h b/net/tipc/core.h index 5cc5398..099e072
> > 100644
> > --- a/net/tipc/core.h
> > +++ b/net/tipc/core.h
> > @@ -49,7 +49,6 @@
> > #include <linux/uaccess.h>
> > #include <linux/interrupt.h>
> > #include <linux/atomic.h>
> > -#include <asm/hardirq.h>
> > #include <linux/netdevice.h>
> > #include <linux/in.h>
> > #include <linux/list.h>
> >
On 12/7/17 11:20 AM, Jon Maloy wrote:
>
>
>> -----Original Message-----
>> From: [email protected] [mailto:netdev-
>> [email protected]] On Behalf Of Yang Shi
>> Sent: Thursday, December 07, 2017 14:16
>> To: [email protected]
>> Cc: [email protected]; [email protected]; linux-
>> [email protected]; [email protected]; Jon Maloy
>> <[email protected]>; Ying Xue <[email protected]>; David S.
>> Miller <[email protected]>
>> Subject: Re: [PATCH 8/8] net: tipc: remove unused hardirq.h
>>
>> Hi folks,
>>
>> Any comment on this one?
>
> If it compiles it is ok with me. Don't know why it was put there in the first place.
Yes, it does compile.
Yang
>
> ///jon
>
>>
>> Thanks,
>> 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 TIPC at all.
>>>
>>> So, remove the unused hardirq.h.
>>>
>>> Signed-off-by: Yang Shi <[email protected]>
>>> Cc: Jon Maloy <[email protected]>
>>> Cc: Ying Xue <[email protected]>
>>> Cc: "David S. Miller" <[email protected]>
>>> ---
>>> net/tipc/core.h | 1 -
>>> 1 file changed, 1 deletion(-)
>>>
>>> diff --git a/net/tipc/core.h b/net/tipc/core.h index 5cc5398..099e072
>>> 100644
>>> --- a/net/tipc/core.h
>>> +++ b/net/tipc/core.h
>>> @@ -49,7 +49,6 @@
>>> #include <linux/uaccess.h>
>>> #include <linux/interrupt.h>
>>> #include <linux/atomic.h>
>>> -#include <asm/hardirq.h>
>>> #include <linux/netdevice.h>
>>> #include <linux/in.h>
>>> #include <linux/list.h>
>>>