[PATCH 1/8] mm: kmemleak: remove unused hardirq.h

Subsystems: kmemleak, memory management, the rest

STALE3105d

27 messages, 9 authors, 2018-01-08 · open the first message on its own page

[PATCH 1/8] mm: kmemleak: remove unused hardirq.h

From: Yang Shi <hidden>
Date: 2017-11-17 23:02:14

Preempt counter APIs have been split out, currently, hardirq.h just
includes irq_enter/exit APIs which are not used by kmemleak at all.

So, remove the unused hardirq.h.

Signed-off-by: Yang Shi <redacted>
Cc: Michal Hocko <mhocko@suse.com>
Cc: Andrew Morton <akpm@linux-foundation.org>
Cc: Matthew Wilcox <redacted>
---
 mm/kmemleak.c | 1 -
 1 file changed, 1 deletion(-)
diff --git a/mm/kmemleak.c b/mm/kmemleak.c
index 7780cd8..25b977f 100644
--- a/mm/kmemleak.c
+++ b/mm/kmemleak.c
@@ -91,7 +91,6 @@
 #include <linux/stacktrace.h>
 #include <linux/cache.h>
 #include <linux/percpu.h>
-#include <linux/hardirq.h>
 #include <linux/bootmem.h>
 #include <linux/pfn.h>
 #include <linux/mmzone.h>
-- 
1.8.3.1

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

[PATCH 2/8] fs: pstore: remove unused hardirq.h

From: Yang Shi <hidden>
Date: 2017-11-17 23:02:15

Preempt counter APIs have been split out, currently, hardirq.h just
includes irq_enter/exit APIs which are not used by pstore at all.

So, remove the unused hardirq.h.

Signed-off-by: Yang Shi <redacted>
Cc: Kees Cook <redacted>
Cc: Anton Vorontsov <redacted>
Cc: Colin Cross <redacted>
Cc: Tony Luck <tony.luck@intel.com>
---
 fs/pstore/platform.c | 1 -
 1 file changed, 1 deletion(-)
diff --git a/fs/pstore/platform.c b/fs/pstore/platform.c
index 2b21d18..25dcef4 100644
--- a/fs/pstore/platform.c
+++ b/fs/pstore/platform.c
@@ -41,7 +41,6 @@
 #include <linux/timer.h>
 #include <linux/slab.h>
 #include <linux/uaccess.h>
-#include <linux/hardirq.h>
 #include <linux/jiffies.h>
 #include <linux/workqueue.h>
 
-- 
1.8.3.1

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

[PATCH 3/8] fs: btrfs: remove unused hardirq.h

From: Yang Shi <hidden>
Date: 2017-11-17 23:02:16

Preempt counter APIs have been split out, currently, hardirq.h just
includes irq_enter/exit APIs which are not used by btrfs at all.

So, remove the unused hardirq.h.

Signed-off-by: Yang Shi <redacted>
Cc: Chris Mason <clm@fb.com>
Cc: Josef Bacik <redacted>
Cc: David Sterba <dsterba@suse.com>
Cc: linux-btrfs@vger.kernel.org
---
 fs/btrfs/extent_map.c | 1 -
 1 file changed, 1 deletion(-)
diff --git a/fs/btrfs/extent_map.c b/fs/btrfs/extent_map.c
index 2e348fb..cced7f1 100644
--- a/fs/btrfs/extent_map.c
+++ b/fs/btrfs/extent_map.c
@@ -2,7 +2,6 @@
 #include <linux/err.h>
 #include <linux/slab.h>
 #include <linux/spinlock.h>
-#include <linux/hardirq.h>
 #include "ctree.h"
 #include "extent_map.h"
 #include "compression.h"
-- 
1.8.3.1

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

[PATCH 4/8] vfs: remove unused hardirq.h

From: Yang Shi <hidden>
Date: 2017-11-17 23:02:17

Preempt counter APIs have been split out, currently, hardirq.h just
includes irq_enter/exit APIs which are not used by vfs at all.

So, remove the unused hardirq.h.

Signed-off-by: Yang Shi <redacted>
Cc: Alexander Viro <viro@zeniv.linux.org.uk>
---
 fs/dcache.c     | 1 -
 fs/file_table.c | 1 -
 2 files changed, 2 deletions(-)
diff --git a/fs/dcache.c b/fs/dcache.c
index f901413..9340e8c 100644
--- a/fs/dcache.c
+++ b/fs/dcache.c
@@ -32,7 +32,6 @@
 #include <linux/swap.h>
 #include <linux/bootmem.h>
 #include <linux/fs_struct.h>
-#include <linux/hardirq.h>
 #include <linux/bit_spinlock.h>
 #include <linux/rculist_bl.h>
 #include <linux/prefetch.h>
diff --git a/fs/file_table.c b/fs/file_table.c
index 61517f5..dab099e 100644
--- a/fs/file_table.c
+++ b/fs/file_table.c
@@ -23,7 +23,6 @@
 #include <linux/sysctl.h>
 #include <linux/percpu_counter.h>
 #include <linux/percpu.h>
-#include <linux/hardirq.h>
 #include <linux/task_work.h>
 #include <linux/ima.h>
 #include <linux/swap.h>
-- 
1.8.3.1

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

[PATCH 5/8] crypto: remove unused hardirq.h

From: Yang Shi <hidden>
Date: 2017-11-17 23:02:18

Preempt counter APIs have been split out, currently, hardirq.h just
includes irq_enter/exit APIs which are not used by crypto at all.

So, remove the unused hardirq.h.

Signed-off-by: Yang Shi <redacted>
Cc: Herbert Xu <herbert@gondor.apana.org.au>
Cc: "David S. Miller" <davem@davemloft.net>
Cc: linux-crypto@vger.kernel.org
---
 crypto/ablk_helper.c | 1 -
 crypto/blkcipher.c   | 1 -
 crypto/mcryptd.c     | 1 -
 3 files changed, 3 deletions(-)
diff --git a/crypto/ablk_helper.c b/crypto/ablk_helper.c
index 1441f07..ee52660 100644
--- a/crypto/ablk_helper.c
+++ b/crypto/ablk_helper.c
@@ -28,7 +28,6 @@
 #include <linux/crypto.h>
 #include <linux/init.h>
 #include <linux/module.h>
-#include <linux/hardirq.h>
 #include <crypto/algapi.h>
 #include <crypto/cryptd.h>
 #include <crypto/ablk_helper.h>
diff --git a/crypto/blkcipher.c b/crypto/blkcipher.c
index 6c43a0a..01c0d4a 100644
--- a/crypto/blkcipher.c
+++ b/crypto/blkcipher.c
@@ -18,7 +18,6 @@
 #include <crypto/internal/skcipher.h>
 #include <crypto/scatterwalk.h>
 #include <linux/errno.h>
-#include <linux/hardirq.h>
 #include <linux/kernel.h>
 #include <linux/module.h>
 #include <linux/seq_file.h>
diff --git a/crypto/mcryptd.c b/crypto/mcryptd.c
index 4e64726..9fa362c 100644
--- a/crypto/mcryptd.c
+++ b/crypto/mcryptd.c
@@ -26,7 +26,6 @@
 #include <linux/sched.h>
 #include <linux/sched/stat.h>
 #include <linux/slab.h>
-#include <linux/hardirq.h>
 
 #define MCRYPTD_MAX_CPU_QLEN 100
 #define MCRYPTD_BATCH 9
-- 
1.8.3.1

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

[PATCH 6/8] net: caif: remove unused hardirq.h

From: Yang Shi <hidden>
Date: 2017-11-17 23:02:19

Preempt counter APIs have been split out, currently, hardirq.h just
includes irq_enter/exit APIs which are not used by caif at all.

So, remove the unused hardirq.h.

Signed-off-by: Yang Shi <redacted>
Cc: Dmitry Tarnyagin <redacted>
Cc: "David S. Miller" <davem@davemloft.net>
---
 net/caif/cfpkt_skbuff.c | 1 -
 net/caif/chnl_net.c     | 1 -
 2 files changed, 2 deletions(-)
diff --git a/net/caif/cfpkt_skbuff.c b/net/caif/cfpkt_skbuff.c
index 71b6ab2..38c2b7a 100644
--- a/net/caif/cfpkt_skbuff.c
+++ b/net/caif/cfpkt_skbuff.c
@@ -8,7 +8,6 @@
 
 #include <linux/string.h>
 #include <linux/skbuff.h>
-#include <linux/hardirq.h>
 #include <linux/export.h>
 #include <net/caif/cfpkt.h>
 
diff --git a/net/caif/chnl_net.c b/net/caif/chnl_net.c
index 922ac1d..53ecda1 100644
--- a/net/caif/chnl_net.c
+++ b/net/caif/chnl_net.c
@@ -8,7 +8,6 @@
 #define pr_fmt(fmt) KBUILD_MODNAME ":%s(): " fmt, __func__
 
 #include <linux/fs.h>
-#include <linux/hardirq.h>
 #include <linux/init.h>
 #include <linux/module.h>
 #include <linux/netdevice.h>
-- 
1.8.3.1

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

[PATCH 7/8] net: ovs: remove unused hardirq.h

From: Yang Shi <hidden>
Date: 2017-11-17 23:02:20

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 <redacted>
Cc: Pravin Shelar <redacted>
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>
-- 
1.8.3.1

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

[PATCH 8/8] net: tipc: remove unused hardirq.h

From: Yang Shi <hidden>
Date: 2017-11-17 23:02:21

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 <redacted>
Cc: Jon Maloy <redacted>
Cc: Ying Xue <redacted>
Cc: "David S. Miller" <davem@davemloft.net>
---
 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>
-- 
1.8.3.1

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH 7/8] net: ovs: remove unused hardirq.h

From: Yang Shi <hidden>
Date: 2017-11-18 01:48:45

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:
quoted hunk
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 <redacted>
Cc: Pravin Shelar <redacted>
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>

Re: [PATCH 3/8] fs: btrfs: remove unused hardirq.h

From: David Sterba <hidden>
Date: 2017-11-20 13:25:16

On Sat, Nov 18, 2017 at 07:02:16AM +0800, Yang Shi wrote:
Preempt counter APIs have been split out, currently, hardirq.h just
includes irq_enter/exit APIs which are not used by btrfs at all.

So, remove the unused hardirq.h.

Signed-off-by: Yang Shi <redacted>
Cc: Chris Mason <clm@fb.com>
Cc: Josef Bacik <redacted>
Cc: David Sterba <dsterba@suse.com>
Cc: linux-btrfs@vger.kernel.org
Acked-by: David Sterba <dsterba@suse.com>

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH 5/8] crypto: remove unused hardirq.h

From: Yang Shi <hidden>
Date: 2017-11-20 16:29:12

The email to Herbert is returned, resent it.

Yang


On 11/17/17 3:02 PM, Yang Shi wrote:
quoted hunk
Preempt counter APIs have been split out, currently, hardirq.h just
includes irq_enter/exit APIs which are not used by crypto at all.

So, remove the unused hardirq.h.

Signed-off-by: Yang Shi <redacted>
Cc: Herbert Xu <herbert@gondor.apana.org.au>
Cc: "David S. Miller" <davem@davemloft.net>
Cc: linux-crypto@vger.kernel.org
---
  crypto/ablk_helper.c | 1 -
  crypto/blkcipher.c   | 1 -
  crypto/mcryptd.c     | 1 -
  3 files changed, 3 deletions(-)
diff --git a/crypto/ablk_helper.c b/crypto/ablk_helper.c
index 1441f07..ee52660 100644
--- a/crypto/ablk_helper.c
+++ b/crypto/ablk_helper.c
@@ -28,7 +28,6 @@
  #include <linux/crypto.h>
  #include <linux/init.h>
  #include <linux/module.h>
-#include <linux/hardirq.h>
  #include <crypto/algapi.h>
  #include <crypto/cryptd.h>
  #include <crypto/ablk_helper.h>
diff --git a/crypto/blkcipher.c b/crypto/blkcipher.c
index 6c43a0a..01c0d4a 100644
--- a/crypto/blkcipher.c
+++ b/crypto/blkcipher.c
@@ -18,7 +18,6 @@
  #include <crypto/internal/skcipher.h>
  #include <crypto/scatterwalk.h>
  #include <linux/errno.h>
-#include <linux/hardirq.h>
  #include <linux/kernel.h>
  #include <linux/module.h>
  #include <linux/seq_file.h>
diff --git a/crypto/mcryptd.c b/crypto/mcryptd.c
index 4e64726..9fa362c 100644
--- a/crypto/mcryptd.c
+++ b/crypto/mcryptd.c
@@ -26,7 +26,6 @@
  #include <linux/sched.h>
  #include <linux/sched/stat.h>
  #include <linux/slab.h>
-#include <linux/hardirq.h>
  
  #define MCRYPTD_MAX_CPU_QLEN 100
  #define MCRYPTD_BATCH 9

Re: [PATCH 2/8] fs: pstore: remove unused hardirq.h

From: Kees Cook <hidden>
Date: 2017-11-29 00:40:15

On Fri, Nov 17, 2017 at 3:02 PM, Yang Shi [off-list ref] wrote:
Preempt counter APIs have been split out, currently, hardirq.h just
includes irq_enter/exit APIs which are not used by pstore at all.

So, remove the unused hardirq.h.

Signed-off-by: Yang Shi <redacted>
Cc: Kees Cook <redacted>
Cc: Anton Vorontsov <redacted>
Cc: Colin Cross <redacted>
Cc: Tony Luck <tony.luck@intel.com>
Thanks! I've applied this for -next.

-Kees
quoted hunk
---
 fs/pstore/platform.c | 1 -
 1 file changed, 1 deletion(-)
diff --git a/fs/pstore/platform.c b/fs/pstore/platform.c
index 2b21d18..25dcef4 100644
--- a/fs/pstore/platform.c
+++ b/fs/pstore/platform.c
@@ -41,7 +41,6 @@
 #include <linux/timer.h>
 #include <linux/slab.h>
 #include <linux/uaccess.h>
-#include <linux/hardirq.h>
 #include <linux/jiffies.h>
 #include <linux/workqueue.h>

--
1.8.3.1


-- 
Kees Cook
Pixel Security

Re: [5/8] crypto: remove unused hardirq.h

From: Herbert Xu <herbert@gondor.apana.org.au>
Date: 2017-11-29 06:38:16

On Sat, Nov 18, 2017 at 07:02:18AM +0800, Yang Shi wrote:
Preempt counter APIs have been split out, currently, hardirq.h just
includes irq_enter/exit APIs which are not used by crypto at all.

So, remove the unused hardirq.h.

Signed-off-by: Yang Shi <redacted>
Cc: Herbert Xu <herbert@gondor.apana.org.au>
Cc: "David S. Miller" <davem@davemloft.net>
Cc: linux-crypto@vger.kernel.org
Patch applied.  Thanks.
-- 
Email: Herbert Xu [off-list ref]
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt

Re: [PATCH 4/8] vfs: remove unused hardirq.h

From: Yang Shi <hidden>
Date: 2017-12-07 19:12:52

Hi folks,

Any comment on this one?

Thanks,
Yang


On 11/17/17 3:02 PM, Yang Shi wrote:
quoted hunk
Preempt counter APIs have been split out, currently, hardirq.h just
includes irq_enter/exit APIs which are not used by vfs at all.

So, remove the unused hardirq.h.

Signed-off-by: Yang Shi <redacted>
Cc: Alexander Viro <viro@zeniv.linux.org.uk>
---
  fs/dcache.c     | 1 -
  fs/file_table.c | 1 -
  2 files changed, 2 deletions(-)
diff --git a/fs/dcache.c b/fs/dcache.c
index f901413..9340e8c 100644
--- a/fs/dcache.c
+++ b/fs/dcache.c
@@ -32,7 +32,6 @@
  #include <linux/swap.h>
  #include <linux/bootmem.h>
  #include <linux/fs_struct.h>
-#include <linux/hardirq.h>
  #include <linux/bit_spinlock.h>
  #include <linux/rculist_bl.h>
  #include <linux/prefetch.h>
diff --git a/fs/file_table.c b/fs/file_table.c
index 61517f5..dab099e 100644
--- a/fs/file_table.c
+++ b/fs/file_table.c
@@ -23,7 +23,6 @@
  #include <linux/sysctl.h>
  #include <linux/percpu_counter.h>
  #include <linux/percpu.h>
-#include <linux/hardirq.h>
  #include <linux/task_work.h>
  #include <linux/ima.h>
  #include <linux/swap.h>
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH 6/8] net: caif: remove unused hardirq.h

From: Yang Shi <hidden>
Date: 2017-12-07 19:13:55

Hi folks,

Any comment on this one?

Thanks,
Yang


On 11/17/17 3:02 PM, Yang Shi wrote:
quoted hunk
Preempt counter APIs have been split out, currently, hardirq.h just
includes irq_enter/exit APIs which are not used by caif at all.

So, remove the unused hardirq.h.

Signed-off-by: Yang Shi <redacted>
Cc: Dmitry Tarnyagin <redacted>
Cc: "David S. Miller" <davem@davemloft.net>
---
  net/caif/cfpkt_skbuff.c | 1 -
  net/caif/chnl_net.c     | 1 -
  2 files changed, 2 deletions(-)
diff --git a/net/caif/cfpkt_skbuff.c b/net/caif/cfpkt_skbuff.c
index 71b6ab2..38c2b7a 100644
--- a/net/caif/cfpkt_skbuff.c
+++ b/net/caif/cfpkt_skbuff.c
@@ -8,7 +8,6 @@
  
  #include <linux/string.h>
  #include <linux/skbuff.h>
-#include <linux/hardirq.h>
  #include <linux/export.h>
  #include <net/caif/cfpkt.h>
  
diff --git a/net/caif/chnl_net.c b/net/caif/chnl_net.c
index 922ac1d..53ecda1 100644
--- a/net/caif/chnl_net.c
+++ b/net/caif/chnl_net.c
@@ -8,7 +8,6 @@
  #define pr_fmt(fmt) KBUILD_MODNAME ":%s(): " fmt, __func__
  
  #include <linux/fs.h>
-#include <linux/hardirq.h>
  #include <linux/init.h>
  #include <linux/module.h>
  #include <linux/netdevice.h>

Re: [PATCH 7/8] net: ovs: remove unused hardirq.h

From: Yang Shi <hidden>
Date: 2017-12-07 19:14:58

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:
quoted
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 <redacted>
Cc: Pravin Shelar <redacted>
Cc: "David S. Miller" <davem-fT/PcQaiUtIeIZ0/mPfg9Q@public.gmane.org>
Cc: dev-yBygre7rU0TnMu66kgdUjQ@public.gmane.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>

Re: [PATCH 8/8] net: tipc: remove unused hardirq.h

From: Yang Shi <hidden>
Date: 2017-12-07 19:15:50

Hi folks,

Any comment on this one?

Thanks,
Yang


On 11/17/17 3:02 PM, Yang Shi wrote:
quoted hunk
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 <redacted>
Cc: Jon Maloy <redacted>
Cc: Ying Xue <redacted>
Cc: "David S. Miller" <davem@davemloft.net>
---
  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>

RE: [PATCH 8/8] net: tipc: remove unused hardirq.h

From: Jon Maloy <hidden>
Date: 2017-12-07 19:20:44

-----Original Message-----
From: netdev-owner@vger.kernel.org [mailto:netdev-
owner@vger.kernel.org] On Behalf Of Yang Shi
Sent: Thursday, December 07, 2017 14:16
To: linux-kernel@vger.kernel.org
Cc: linux-mm@kvack.org; linux-fsdevel@vger.kernel.org; linux-
crypto@vger.kernel.org; netdev@vger.kernel.org; Jon Maloy
[off-list ref]; Ying Xue [off-list ref]; David S.
Miller [off-list ref]
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:
quoted
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 <redacted>
Cc: Jon Maloy <redacted>
Cc: Ying Xue <redacted>
Cc: "David S. Miller" <davem@davemloft.net>
---
  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>

Re: [PATCH 4/8] vfs: remove unused hardirq.h

From: Al Viro <viro@ZenIV.linux.org.uk>
Date: 2017-12-07 19:21:56

On Fri, Dec 08, 2017 at 03:12:52AM +0800, Yang Shi wrote:
Hi folks,

Any comment on this one?
Applied

Re: [ovs-dev] [PATCH 7/8] net: ovs: remove unused hardirq.h

From: Pravin Shelar <hidden>
Date: 2017-12-07 19:27:29

On Fri, Nov 17, 2017 at 3:02 PM, Yang Shi [off-list ref] 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 <redacted>
Cc: Pravin Shelar <redacted>
Cc: "David S. Miller" <davem@davemloft.net>
Cc: dev@openvswitch.org
Acked-by: Pravin B Shelar <redacted>

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH 8/8] net: tipc: remove unused hardirq.h

From: Yang Shi <hidden>
Date: 2017-12-07 20:02:33


On 12/7/17 11:20 AM, Jon Maloy wrote:
quoted
-----Original Message-----
From: netdev-owner@vger.kernel.org [mailto:netdev-
owner@vger.kernel.org] On Behalf Of Yang Shi
Sent: Thursday, December 07, 2017 14:16
To: linux-kernel@vger.kernel.org
Cc: linux-mm@kvack.org; linux-fsdevel@vger.kernel.org; linux-
crypto@vger.kernel.org; netdev@vger.kernel.org; Jon Maloy
[off-list ref]; Ying Xue [off-list ref]; David S.
Miller [off-list ref]
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
quoted
Thanks,
Yang


On 11/17/17 3:02 PM, Yang Shi wrote:
quoted
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 <redacted>
Cc: Jon Maloy <redacted>
Cc: Ying Xue <redacted>
Cc: "David S. Miller" <davem@davemloft.net>
---
   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>
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH 8/8] net: tipc: remove unused hardirq.h

From: Ying Xue <hidden>
Date: 2017-12-08 01:40:30

On 11/18/2017 07:02 AM, 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 <redacted>
Cc: Jon Maloy <redacted>
Cc: Ying Xue <redacted>
Cc: "David S. Miller" <davem@davemloft.net>
Tested-by: Ying Xue <redacted>
Acked-by: Ying Xue <redacted>
quoted hunk
---
 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>
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH 8/8] net: tipc: remove unused hardirq.h

From: Yang Shi <hidden>
Date: 2018-01-04 22:46:48

Hi David,

Any more comment on this change?

Thanks,
Yang


On 12/7/17 5:40 PM, Ying Xue wrote:
On 11/18/2017 07:02 AM, Yang Shi wrote:
quoted
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 <redacted>
Cc: Jon Maloy <redacted>
Cc: Ying Xue <redacted>
Cc: "David S. Miller" <davem@davemloft.net>
Tested-by: Ying Xue <redacted>
Acked-by: Ying Xue <redacted>
quoted
---
  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>

Re: [ovs-dev] [PATCH 7/8] net: ovs: remove unused hardirq.h

From: Yang Shi <hidden>
Date: 2018-01-04 22:47:57

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 [off-list ref] wrote:
quoted
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 <redacted>
Cc: Pravin Shelar <redacted>
Cc: "David S. Miller" <davem@davemloft.net>
Cc: dev@openvswitch.org
Acked-by: Pravin B Shelar <redacted>
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH 6/8] net: caif: remove unused hardirq.h

From: Yang Shi <hidden>
Date: 2018-01-04 22:51:36

Hi David,

I'm not sure if CAIF is still maintained by Dmitry Tarnyagin. Do you 
have any comment on this one?

Thanks,
Yang


On 12/7/17 11:13 AM, Yang Shi wrote:
Hi folks,

Any comment on this one?

Thanks,
Yang


On 11/17/17 3:02 PM, Yang Shi wrote:
quoted
Preempt counter APIs have been split out, currently, hardirq.h just
includes irq_enter/exit APIs which are not used by caif at all.

So, remove the unused hardirq.h.

Signed-off-by: Yang Shi <redacted>
Cc: Dmitry Tarnyagin <redacted>
Cc: "David S. Miller" <davem@davemloft.net>
---
  net/caif/cfpkt_skbuff.c | 1 -
  net/caif/chnl_net.c     | 1 -
  2 files changed, 2 deletions(-)
diff --git a/net/caif/cfpkt_skbuff.c b/net/caif/cfpkt_skbuff.c
index 71b6ab2..38c2b7a 100644
--- a/net/caif/cfpkt_skbuff.c
+++ b/net/caif/cfpkt_skbuff.c
@@ -8,7 +8,6 @@
  #include <linux/string.h>
  #include <linux/skbuff.h>
-#include <linux/hardirq.h>
  #include <linux/export.h>
  #include <net/caif/cfpkt.h>
diff --git a/net/caif/chnl_net.c b/net/caif/chnl_net.c
index 922ac1d..53ecda1 100644
--- a/net/caif/chnl_net.c
+++ b/net/caif/chnl_net.c
@@ -8,7 +8,6 @@
  #define pr_fmt(fmt) KBUILD_MODNAME ":%s(): " fmt, __func__
  #include <linux/fs.h>
-#include <linux/hardirq.h>
  #include <linux/init.h>
  #include <linux/module.h>
  #include <linux/netdevice.h>
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH 8/8] net: tipc: remove unused hardirq.h

From: David Miller <davem@davemloft.net>
Date: 2018-01-05 15:17:06

From: "Yang Shi" <redacted>
Date: Fri, 05 Jan 2018 06:46:48 +0800
Any more comment on this change?
These patches were not really submitted properly.

If you post a series, the series goes to one destination and
one tree.

If they are supposed to go to multiple trees, submit them
individually rather than as a series.  With clear indications
in the Subject lines which tree should be taking the patch.

Thank you.

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH 8/8] net: tipc: remove unused hardirq.h

From: Yang Shi <hidden>
Date: 2018-01-08 19:43:14


On 1/5/18 7:17 AM, David Miller wrote:
From: "Yang Shi" <redacted>
Date: Fri, 05 Jan 2018 06:46:48 +0800
quoted
Any more comment on this change?
These patches were not really submitted properly.

If you post a series, the series goes to one destination and
one tree.

If they are supposed to go to multiple trees, submit them
individually rather than as a series.  With clear indications
in the Subject lines which tree should be taking the patch.
Thanks for the comment. I will resend the net patches in a separate 
series to you.

Yang
Thank you.
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help