rcupdate: remove dead code

DEBUG_OBJECTS_RCU_HEAD depends on PREEMPT, so #ifndef CONFIG_PREEMPT is totally useless in kernel/rcupdate.c. Signed-off-by: WANG Cong <amwang@redhat.com> Cc: Paul E. McKenney <paulmck@linux.vnet.ibm.com> Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>

rcupdate: remove dead code
DEBUG_OBJECTS_RCU_HEAD depends on PREEMPT, so #ifndef CONFIG_PREEMPT is totally useless in kernel/rcupdate.c. Signed-off-by: WANG Cong <amwang@redhat.com> Cc: Paul E. McKenney <paulmck@linux.vnet.ibm.com> Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
Amerigo Wang · Paul E. McKenney
1 parent fea651267e
Showing 1 changed file with 0 additions and 5 deletions Side-by-side Diff
kernel/rcupdate.c
@@ -215,10 +215,6 @@
 		 * If we detect that we are nested in a RCU read-side critical
 		 * section, we should simply fail, otherwise we would deadlock.
 		 */
-#ifndef CONFIG_PREEMPT
-		WARN_ON(1);
-		return 0;
-#else
 		if (rcu_preempt_depth() != 0 || preempt_count() != 0 ||
 		    irqs_disabled()) {
 			WARN_ON(1);
@@ -229,7 +225,6 @@
 		rcu_barrier_bh();
 		debug_object_free(head, &rcuhead_debug_descr);
 		return 1;
-#endif
 	default:
 		return 0;
 	}
...	...	@@ -215,10 +215,6 @@
215	215	* If we detect that we are nested in a RCU read-side critical
216	216	* section, we should simply fail, otherwise we would deadlock.
217	217	*/
218		-#ifndef CONFIG_PREEMPT
219		- WARN_ON(1);
220		- return 0;
221		-#else
222	218	if (rcu_preempt_depth() != 0 \|\| preempt_count() != 0 \|\|
223	219	irqs_disabled()) {
224	220	WARN_ON(1);
...	...	@@ -229,7 +225,6 @@
229	225	rcu_barrier_bh();
230	226	debug_object_free(head, &rcuhead_debug_descr);
231	227	return 1;
232		-#endif
233	228	default:
234	229	return 0;
235	230	}