Hi Ryusuke,
On 2014-02-05 20:25, Ryusuke Konishi wrote:
>> /**
>> + * nilfs_set_suinfo - sets segment usage info
>> + * @nilfs: nilfs object
>> + * @sup: an array of nilfs_suinfo_update structs
>> + * @nsup: number of elements in sup
>> + *
>> + * Description: Takes an array of nilfs_suinfo_update structs and updates
>> + * segment usage info accordingly. Only the fields indicated by sup_flags
>> + * are updated.
>> + *
>> + * Return Value: On success, 0 is returned. On error, -1 is returned.
>> + */
>> +ssize_t nilfs_set_suinfo(struct nilfs *nilfs,
>> + struct nilfs_suinfo_update *sup, size_t nsup)
>
> The type of return value of this function should be "int" because it
> doesn't return the count of obtained suinfo items.
>
>> +{
>> + struct nilfs_argv argv;
>> +
>> + if (nilfs->n_iocfd < 0) {
>> + errno = EBADF;
>> + return -1;
>> + }
>> +
>> + argv.v_base = (unsigned long)sup;
>> + argv.v_nmembs = nsup;
>> + argv.v_size = sizeof(struct nilfs_suinfo_update);
>> + argv.v_index = 0;
>> + argv.v_flags = 0;
>> + if (ioctl(nilfs->n_iocfd, NILFS_IOCTL_SET_SUINFO, &argv) < 0) {
>> + if (errno == ENOTTY)
>> + nilfs_opt_clear_set_suinfo(nilfs);
>
> This invalidation should be done in utils or upper library.
> Simply return here as follows:
>
> return ioctl(nilfs->n_iocfd, NILFS_IOCTL_SET_SUINFO, &argv);
> }
Thanks for your review. I have fixed everything you pointed out. Sorry
for all the missing stuff. I will wait with sending in a new version of
the patch set, until you finished your review. I suspect you will also
find some things in patch 5, since it is the crucial part of the
implementation.
Best regards,
Andreas Rohner
--
To unsubscribe from this list: send the line "unsubscribe linux-nilfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
