Re: [PATCH] s390 hypfs fixes for 2.6.18-rc1-mm1

[Badari Pulavarty <pbadari@us.ibm.com>]

Andrew Morton wrote:
> On Wed, 12 Jul 2006 21:17:53 -0700
> Badari Pulavarty <pbadari@us.ibm.com> wrote:
>
>   
>> +static ssize_t hypfs_aio_read(struct kiocb *iocb, const struct iovec *iov,
>> +			      unsigned long nr_segs, loff_t offset)
>>  {
>>  	char *data;
>>  	size_t len;
>>  	struct file *filp = iocb->ki_filp;
>> +	/* XXX: temporary */
>> +	char __user *buf = iov[0].iov_base;
>> +	size_t count = iov[0].iov_len;
>> +
>> +	if (nr_segs != 1) {
>> +		count = -EINVAL;
>> +		goto out;
>> +	}
>>     
>
> err, "temporary" things tend to become permanent.  What's the real fix?
>   
I am not sure, if we really need to vectorize this method or not - 
meaning will this be ever called
with more than one items in the vector. 

Micheal, is it possible ? Can some one directly use AIO interface on 
hypfs ? If not, we can always
look at only first element and ignore rest of them. Otherwise, we need 
to iterate on all the elements
of the vector.

Thanks,
Badari