lkml.org 
[lkml]   [2007]   [Aug]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] sysace: HDIO_GETGEO has it's own method for ages
On 7/30/07, Christoph Hellwig <hch@lst.de> wrote:
> The way this driver triesto implement HDIO_GETGEO it'll never be
> called. Then again on ppc it probably will never be called anyway
> because it's utterly pointless.
>
>
> Signed-off-by: Christoph Hellwig <hch@lst.de>

Heh, that's what I get for following LDD3 exactly for writing my block
dev, and not keeping up with API changes.

Signed-off-by: Grant Likely <grant.likely@secretlab.ca>

>
> Index: linux-2.6/drivers/block/xsysace.c
> ===================================================================
> --- linux-2.6.orig/drivers/block/xsysace.c 2007-07-30 22:53:11.000000000 +0200
> +++ linux-2.6/drivers/block/xsysace.c 2007-07-30 22:54:43.000000000 +0200
> @@ -902,26 +902,17 @@ static int ace_release(struct inode *ino
> return 0;
> }
>
> -static int ace_ioctl(struct inode *inode, struct file *filp,
> - unsigned int cmd, unsigned long arg)
> +static int ace_getgeo(struct block_device *bdev, struct hd_geometry *geo)
> {
> - struct ace_device *ace = inode->i_bdev->bd_disk->private_data;
> - struct hd_geometry __user *geo = (struct hd_geometry __user *)arg;
> - struct hd_geometry g;
> - dev_dbg(ace->dev, "ace_ioctl()\n");
> -
> - switch (cmd) {
> - case HDIO_GETGEO:
> - g.heads = ace->cf_id.heads;
> - g.sectors = ace->cf_id.sectors;
> - g.cylinders = ace->cf_id.cyls;
> - g.start = 0;
> - return copy_to_user(geo, &g, sizeof(g)) ? -EFAULT : 0;
> + struct ace_device *ace = bdev->bd_disk->private_data;
>
> - default:
> - return -ENOTTY;
> - }
> - return -ENOTTY;
> + dev_dbg(ace->dev, "ace_getgeo()\n");
> +
> + geo->heads = ace->cf_id.heads;
> + geo->sectors = ace->cf_id.sectors;
> + geo->cylinders = ace->cf_id.cyls;
> +
> + return 0;
> }
>
> static struct block_device_operations ace_fops = {
> @@ -930,7 +921,7 @@ static struct block_device_operations ac
> .release = ace_release,
> .media_changed = ace_media_changed,
> .revalidate_disk = ace_revalidate_disk,
> - .ioctl = ace_ioctl,
> + .getgeo = ace_getgeo,
> };
>
> /* --------------------------------------------------------------------
>


--
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.
grant.likely@secretlab.ca
(403) 399-0195
-
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: 2007-08-01 15:01    [W:0.036 / U:10.704 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site