Re: [PATCH 24/34] PCI: use weak functions for MSI arch-specific functions
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- To: Richard Zhu <r65037@xxxxxxxxxxxxx>
- Subject: Re: [PATCH 24/34] PCI: use weak functions for MSI arch-specific functions
- From: Ralf Baechle <ralf@xxxxxxxxxxxxxx>
- Date: Thu, 21 Nov 2013 18:39:33 +0100
- Cc: Thomas Petazzoni <thomas.petazzoni@xxxxxxxxxxxxxxxxxx>, Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx>, Paul Mackerras <paulus@xxxxxxxxx>, linuxppc-dev@xxxxxxxxxxxxxxxx, Martin Schwidefsky <schwidefsky@xxxxxxxxxx>, Heiko Carstens <heiko.carstens@xxxxxxxxxx>, linux390@xxxxxxxxxx, linux-s390@xxxxxxxxxxxxxxx, Thomas Gleixner <tglx@xxxxxxxxxxxxx>, Ingo Molnar <mingo@xxxxxxxxxx>, "H. Peter Anvin" <hpa@xxxxxxxxx>, x86@xxxxxxxxxx, Russell King <linux@xxxxxxxxxxxxxxxx>, Tony Luck <tony.luck@xxxxxxxxx>, Fenghua Yu <fenghua.yu@xxxxxxxxx>, linux-ia64@xxxxxxxxxxxxxxx, linux-mips@xxxxxxxxxxxxxx, "David S. Miller" <davem@xxxxxxxxxxxxx>, sparclinux@xxxxxxxxxxxxxxx, Chris Metcalf <cmetcalf@xxxxxxxxxx>, Jason Cooper <jason@xxxxxxxxxxxxxx>
- In-reply-to: <1384915853-31006-24-git-send-email-r65037@freescale.com>
- List-id: <linux-ia64.vger.kernel.org>
- References: <1384915853-31006-1-git-send-email-r65037@freescale.com> <1384915853-31006-24-git-send-email-r65037@freescale.com>
- User-agent: Mutt/1.5.21 (2010-09-15)
On Wed, Nov 20, 2013 at 10:50:43AM +0800, Richard Zhu wrote:
Looking good,
Acked-by: Ralf Baechle <ralf@xxxxxxxxxxxxxx>
Nevertheless I'd again like to express that I'm not that fond of of the
increasing number of weak functions in the kernel. In the old days
things were such that when an a platform didn't provice a platform hook
or enable a default hook function, one would get a build error - an
unmistakable sign to the maintainer that something needs attention.
Weak functions mean default functions may result in subtly incorrect
operation. Been there, got bitten.
Ralf
--
To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
[Index of Archives]
[Linux Kernel]
[Sparc Linux]
[DCCP]
[Linux ARM]
[Yosemite News]
[Linux SCSI]
[Linux x86_64]
[Linux for Ham Radio]