Messages in this thread |  | | Date | Thu, 17 May 2001 15:05:14 -0400 | From | Jeff Garzik <> | Subject | Re: Linux 2.4.4-ac10 |
| |
"J . A . Magallon" wrote: > --- linux-2.4.4-ac10/include/linux/raid/md_k.h.orig Thu May 17 19:35:41 > 2001 > +++ linux-2.4.4-ac10/include/linux/raid/md_k.h Thu May 17 19:36:15 2001 > @@ -38,6 +38,8 @@ > case RAID5: return 5; > } > panic("pers_to_level()"); > + > + return 0; > }
panic should be marked attribute(noreturn), so gcc is being silly here by warning at all.
I do this too, because IMHO its inline and won't make things bigger just shut up the warning. But Alan will yell at you for fixing gcc bugs in the kernel source :)
Also, add a comment "fixes gcc warning" next to the code, so people know why it's there.
-- Jeff Garzik | Game called on account of naked chick Building 1024 | MandrakeSoft | - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
|  |