[IA64] Eliminate NULL test after alloc_bootmem in iosapic_alloc_rte()

Julia Lawall · aegl · commit e2569b7e572c · 2008-06-24T10:28:55.000-07:00
As noted by Akinobu Mita alloc_bootmem and related functions never return
NULL and always return a zeroed region of memory.  Thus a NULL test or
memset after calls to these functions is unnecessary.

Signed-off-by: Julia Lawall &lt;julia@diku.dk&gt;
Signed-off-by: Tony Luck &lt;tony.luck@intel.com&gt;
diff --git a/arch/ia64/kernel/iosapic.c b/arch/ia64/kernel/iosapic.c
@@ -558,8 +558,6 @@ static struct iosapic_rte_info * __init_refok iosapic_alloc_rte (void)
 	if (!iosapic_kmalloc_ok && list_empty(&free_rte_list)) {
 		rte = alloc_bootmem(sizeof(struct iosapic_rte_info) *
 				    NR_PREALLOCATE_RTE_ENTRIES);
-		if (!rte)
-			return NULL;
 		for (i = 0; i < NR_PREALLOCATE_RTE_ENTRIES; i++, rte++)
 			list_add(&rte->rte_list, &free_rte_list);
 	}

Original file line number	Diff line number	Diff line change
`@@ -558,8 +558,6 @@ static struct iosapic_rte_info * __init_refok iosapic_alloc_rte (void)`
`558`	`558`	`if (!iosapic_kmalloc_ok && list_empty(&free_rte_list)) {`
`559`	`559`	`rte = alloc_bootmem(sizeof(struct iosapic_rte_info) *`
`560`	`560`	`NR_PREALLOCATE_RTE_ENTRIES);`
`561`		`- if (!rte)`
`562`		`- return NULL;`
`563`	`561`	`for (i = 0; i < NR_PREALLOCATE_RTE_ENTRIES; i++, rte++)`
`564`	`562`	`list_add(&rte->rte_list, &free_rte_list);`
`565`	`563`	`}`