Commit a91c7775 authored by Himangi Saraogi's avatar Himangi Saraogi Committed by Konrad Rzeszutek Wilk
Browse files

xen/arm: use BUG_ON 



Use BUG_ON(x) rather than if(x) BUG();

The semantic patch that fixes this problem is as follows:

// <smpl>
@@ identifier x; @@
-if (x) BUG();
+BUG_ON(x);
// </smpl>

Signed-off-by: default avatarHimangi Saraogi <himangi774@gmail.com>
Signed-off-by: default avatarKonrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Acked-by: default avatarStefano Stabellini <stefano.stabellini@eu.citrix.com>
parent 438b33c7

arch/arm/xen/enlighten.c

+2 −4
Original line number Diff line number Diff line
@@ -181,8 +181,7 @@ static void xen_restart(enum reboot_mode reboot_mode, const char *cmd) 
	struct sched_shutdown r = { .reason = SHUTDOWN_reboot };

	int rc;

	rc = HYPERVISOR_sched_op(SCHEDOP_shutdown, &r);

	if (rc)

		BUG();

	BUG_ON(rc);

}



static void xen_power_off(void)
@@ -190,8 +189,7 @@ static void xen_power_off(void) 
	struct sched_shutdown r = { .reason = SHUTDOWN_poweroff };

	int rc;

	rc = HYPERVISOR_sched_op(SCHEDOP_shutdown, &r);

	if (rc)

		BUG();

	BUG_ON(rc);

}



static int xen_cpu_notification(struct notifier_block *self,

CRA Git | Maintained and supported by SUSTech CRA and CCSE