Re: [PATCH 02/31] Constify struct address_space_operations for2.6.32 v1

From: Matthew Wilcox
Date: Fri Dec 04 2009 - 19:08:57 EST


On Fri, Dec 04, 2009 at 11:08:09PM +0100, Emese Revfy wrote:
> struct address_space_operations {
> - int (*writepage)(struct page *page, struct writeback_control *wbc);
> - int (*readpage)(struct file *, struct page *);
> - void (*sync_page)(struct page *);
> + int (* const writepage)(struct page *page, struct writeback_control *wbc);
> + int (* const readpage)(struct file *, struct page *);
> + void (* const sync_page)(struct page *);

Umm. What effect does this have? Aren't functions implicitly constant?
What changes as a result of this patch?

--
Matthew Wilcox Intel Open Source Technology Centre
"Bill, look, we understand that you're interested in selling us this
operating system, but compare it to ours. We can't possibly take such
a retrograde step."
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/