Re: [PATCH] fs: replace int param with size_t for seq_open_private()

From: Rob Jones
Date: Mon Sep 01 2014 - 11:53:50 EST




On 01/09/14 16:36, Al Viro wrote:
On Mon, Sep 01, 2014 at 02:17:08PM +0100, Rob Jones wrote:

void *__seq_open_private(struct file *f, const struct seq_operations *ops,
- int psize)
+ size_t psize)

<sarcasm>
It is a horrible limitation to impose, indeed. Why, a lousy
2 gigabytes per line in procfs file - that's intolerable...
</sarcasm>


<shrug>

Consistency?

kmalloc() expects a size_t and I haven't heard anyone whingeing about that.

Ultimately, this function calls kmalloc().



--
Rob Jones
Codethink Ltd
mailto:rob.jones@xxxxxxxxxxxxxxx
tel:+44 161 236 5575
--
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/