lkml.org 
[lkml]   [2011]   [Jul]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    SubjectRe: [PATCH] block: ioctl: Fix coding style issues
    From
    Sorry, please disregard -- indentation is still bogus.

    -- Chris

    On Fri, Jul 1, 2011 at 9:09 PM, Chris Forbes <chrisf@ijw.co.nz> wrote:
    > Fixed issues as flagged by checkpatch.pl
    >
    > Signed-off-by: Chris Forbes <chrisf@ijw.co.nz>
    > ---
    >  block/ioctl.c |  105 +++++++++++++++++++++++++++++----------------------------
    >  1 files changed, 53 insertions(+), 52 deletions(-)
    >
    > diff --git a/block/ioctl.c b/block/ioctl.c
    > index 1124cd2..fa284e5 100644
    > --- a/block/ioctl.c
    > +++ b/block/ioctl.c
    > @@ -6,10 +6,10 @@
    >  #include <linux/backing-dev.h>
    >  #include <linux/buffer_head.h>
    >  #include <linux/blktrace_api.h>
    > -#include <asm/uaccess.h>
    > +#include <linux/uaccess.h>
    >
    > -static int blkpg_ioctl(struct block_device *bdev, struct blkpg_ioctl_arg __user *arg)
    > -{
    > +static int blkpg_ioctl(struct block_device *bdev,
    > +                       struct blkpg_ioctl_arg __user *arg) {
    >        struct block_device *bdevp;
    >        struct gendisk *disk;
    >        struct hd_struct *part;
    > @@ -31,28 +31,29 @@ static int blkpg_ioctl(struct block_device *bdev, struct blkpg_ioctl_arg __user
    >        partno = p.pno;
    >        if (partno <= 0)
    >                return -EINVAL;
    > +
    >        switch (a.op) {
    > -               case BLKPG_ADD_PARTITION:
    > -                       start = p.start >> 9;
    > -                       length = p.length >> 9;
    > -                       /* check for fit in a hd_struct */
    > -                       if (sizeof(sector_t) == sizeof(long) &&
    > -                           sizeof(long long) > sizeof(long)) {
    > -                               long pstart = start, plength = length;
    > -                               if (pstart != start || plength != length
    > -                                   || pstart < 0 || plength < 0)
    > -                                       return -EINVAL;
    > -                       }
    > +       case BLKPG_ADD_PARTITION:
    > +               start = p.start >> 9;
    > +               length = p.length >> 9;
    > +               /* check for fit in a hd_struct */
    > +               if (sizeof(sector_t) == sizeof(long) &&
    > +                   sizeof(long long) > sizeof(long)) {
    > +                       long pstart = start, plength = length;
    > +                       if (pstart != start || plength != length
    > +                           || pstart < 0 || plength < 0)
    > +                               return -EINVAL;
    > +               }
    >
    > -                       mutex_lock(&bdev->bd_mutex);
    > +               mutex_lock(&bdev->bd_mutex);
    >
    > -                       /* overlap? */
    > -                       disk_part_iter_init(&piter, disk,
    > -                                           DISK_PITER_INCL_EMPTY);
    > -                       while ((part = disk_part_iter_next(&piter))) {
    > -                               if (!(start + length <= part->start_sect ||
    > -                                     start >= part->start_sect + part->nr_sects)) {
    > -                                       disk_part_iter_exit(&piter);
    > +               /* overlap? */
    > +               disk_part_iter_init(&piter, disk,
    > +                                   DISK_PITER_INCL_EMPTY);
    > +               while ((part = disk_part_iter_next(&piter))) {
    > +                       if (!(start + length <= part->start_sect ||
    > +                             start >= part->start_sect + part->nr_sects)) {
    > +                               disk_part_iter_exit(&piter);
    >                                        mutex_unlock(&bdev->bd_mutex);
    >                                        return -EBUSY;
    >                                }
    > @@ -64,35 +65,35 @@ static int blkpg_ioctl(struct block_device *bdev, struct blkpg_ioctl_arg __user
    >                                             ADDPART_FLAG_NONE, NULL);
    >                        mutex_unlock(&bdev->bd_mutex);
    >                        return IS_ERR(part) ? PTR_ERR(part) : 0;
    > -               case BLKPG_DEL_PARTITION:
    > -                       part = disk_get_part(disk, partno);
    > -                       if (!part)
    > -                               return -ENXIO;
    > -
    > -                       bdevp = bdget(part_devt(part));
    > -                       disk_put_part(part);
    > -                       if (!bdevp)
    > -                               return -ENOMEM;
    > -
    > -                       mutex_lock(&bdevp->bd_mutex);
    > -                       if (bdevp->bd_openers) {
    > -                               mutex_unlock(&bdevp->bd_mutex);
    > -                               bdput(bdevp);
    > -                               return -EBUSY;
    > -                       }
    > -                       /* all seems OK */
    > -                       fsync_bdev(bdevp);
    > -                       invalidate_bdev(bdevp);
    > -
    > -                       mutex_lock_nested(&bdev->bd_mutex, 1);
    > -                       delete_partition(disk, partno);
    > -                       mutex_unlock(&bdev->bd_mutex);
    > +       case BLKPG_DEL_PARTITION:
    > +               part = disk_get_part(disk, partno);
    > +               if (!part)
    > +                       return -ENXIO;
    > +
    > +               bdevp = bdget(part_devt(part));
    > +               disk_put_part(part);
    > +               if (!bdevp)
    > +                       return -ENOMEM;
    > +
    > +               mutex_lock(&bdevp->bd_mutex);
    > +               if (bdevp->bd_openers) {
    >                        mutex_unlock(&bdevp->bd_mutex);
    >                        bdput(bdevp);
    > +                       return -EBUSY;
    > +               }
    > +               /* all seems OK */
    > +               fsync_bdev(bdevp);
    > +               invalidate_bdev(bdevp);
    >
    > -                       return 0;
    > -               default:
    > -                       return -EINVAL;
    > +               mutex_lock_nested(&bdev->bd_mutex, 1);
    > +               delete_partition(disk, partno);
    > +               mutex_unlock(&bdev->bd_mutex);
    > +               mutex_unlock(&bdevp->bd_mutex);
    > +               bdput(bdevp);
    > +
    > +               return 0;
    > +       default:
    > +               return -EINVAL;
    >        }
    >  }
    >
    > @@ -189,7 +190,7 @@ int blkdev_ioctl(struct block_device *bdev, fmode_t mode, unsigned cmd,
    >        loff_t size;
    >        int ret, n;
    >
    > -       switch(cmd) {
    > +       switch (cmd) {
    >        case BLKFLSBUF:
    >                if (!capable(CAP_SYS_ADMIN))
    >                        return -EACCES;
    > @@ -279,7 +280,7 @@ int blkdev_ioctl(struct block_device *bdev, fmode_t mode, unsigned cmd,
    >                return put_ushort(arg, queue_max_sectors(bdev_get_queue(bdev)));
    >        case BLKRASET:
    >        case BLKFRASET:
    > -               if(!capable(CAP_SYS_ADMIN))
    > +               if (!capable(CAP_SYS_ADMIN))
    >                        return -EACCES;
    >                bdi = blk_get_backing_dev_info(bdev);
    >                if (bdi == NULL)
    > --
    > 1.7.4.1
    >
    >
    --
    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/

    \
     
     \ /
      Last update: 2011-07-01 11:19    [W:0.042 / U:0.120 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site