Date: Fri, 9 Dec 2005 23:53:21 -0800
From: Andrew Morton <akpm@osdl.org>
To: Martin Schwidefsky <schwidefsky@de.ibm.com>
Cc: cohuck@de.ibm.com, linux-kernel@vger.kernel.org
Subject: Re: [patch 15/17] s390: convert /proc/cio_ignore.
Message-Id: <20051209235321.2281b7e6.akpm@osdl.org>
In-Reply-To: <20051209152909.GP6532@skybase.boeblingen.de.ibm.com>
References: <20051209152909.GP6532@skybase.boeblingen.de.ibm.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1171
Lines: 47

Martin Schwidefsky <schwidefsky@de.ibm.com> wrote:
>
> +static void *
>  +cio_ignore_proc_seq_start(struct seq_file *s, loff_t *offset)
>  +{
>  +	struct ccwdev_iter *iter;
>  +
>  +	if (*offset > __MAX_SUBCHANNEL)
>  +		return NULL;
>  +	iter = kmalloc(sizeof(struct ccwdev_iter), GFP_KERNEL);
>  +	if (!iter)
>  +		return ERR_PTR(-ENOMEM);
>  +	memset(iter, 0, sizeof(struct ccwdev_iter));

kzalloc()

>  +	iter->devno = *offset;
>  +	return iter;
>  +}
>  +
>  +static void
>  +cio_ignore_proc_seq_stop(struct seq_file *s, void *it)
>  +{
>  +	if (!IS_ERR(it))
>  +		kfree(it);
>  +}
>  +
>  +static void *
>  +cio_ignore_proc_seq_next(struct seq_file *s, void *it, loff_t *offset)
>  +{
>  +	struct ccwdev_iter *iter;
>  +
>  +	if (*offset > __MAX_SUBCHANNEL)
>  +		return NULL;
>  +	iter = (struct ccwdev_iter *)it;

Unneeded (and undesirable) cast of void*.

>  +	iter->devno++;
>  +	(*offset)++;
>  +	return iter;
>  +}
-
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/