lkml.org 
[lkml]   [2003]   [Feb]   [23]   [last100]   RSS Feed
Views: [more markup]  [less markup]  [headers]  [forward] 
 
Messages in this thread
/
DateSun, 23 Feb 2003 10:29:37 +0000
FromChristoph Hellwig <>
SubjectRe: [PATCH] PC-9800 subarch. support for 2.5.62-AC1 (6/21) FS & partiton
On Sun, Feb 23, 2003 at 06:43:25PM +0900, Osamu Tomita wrote:
> -	if (FAT_FIRST_ENT(sb, media) != first) {> +	if (FAT_FIRST_ENT(sb, media) != first> +	    && (!pc98 || media != 0xf8 || (first & 0xff) != 0xfe))
> +	{

I think this should be made unconditionally.  There's no reason why
non-pc98 linux machines shouldn't read fat filesystems created on pc98.

> +/* #ifdef CONFIG_BLK_DEV_IDEDISK */> +#include <linux/ide.h>> +/* #endif */> +> +/* #ifdef CONFIG_BLK_DEV_SD */> +#include "../../drivers/scsi/scsi.h"> +#include "../../drivers/scsi/hosts.h"> +#include <scsi/scsicam.h>> +/* #endif */

this is really ugly..

> +	int g_head, g_sect;> +	Sector sect;> +	const struct nec98_partition *part;> +	unsigned char *data;> +	int sector_size = bdev_hardsect_size(bdev);> +	int major = major(to_kdev_t(bdev->bd_dev));> +	int minor = minor(to_kdev_t(bdev->bd_dev));

use MAJOR/MINOR here.

> +> +	switch (major) {> +#if defined CONFIG_BLK_DEV_HD_ONLY> +	case HD_MAJOR:> +	{> +		extern struct hd_i_struct hd_info[2];> +> +		g_head = hd_info[minor >> 6].head;
> +		g_sect = hd_info[minor >> 6].sect;
> +		break;
> +	}
> +#endif /* CONFIG_BLK_DEV_HD_ONLY */
> +#if defined CONFIG_BLK_DEV_SD || defined CONFIG_BLK_DEV_SD_MODULE
> +	case SCSI_DISK0_MAJOR:
> +	case SCSI_DISK1_MAJOR:
> +	case SCSI_DISK2_MAJOR:
> +	case SCSI_DISK3_MAJOR:
> +	case SCSI_DISK4_MAJOR:
> +	case SCSI_DISK5_MAJOR:
> +	case SCSI_DISK6_MAJOR:
> +	case SCSI_DISK7_MAJOR:
> +	{
> +		int diskinfo[3] = { 0, 0, 0 };
> +
> +		pc98_bios_param(bdev, diskinfo);
> +
> +		if ((g_head = diskinfo[0]) <= 0)
> +			g_head = 8;
> +		if ((g_sect = diskinfo[1]) <= 0)
> +			g_sect = 17;
> +		break;
> +	}
> +#endif /* CONFIG_BLK_DEV_SD(_MODULE) */
> +#if defined CONFIG_BLK_DEV_IDEDISK || defined CONFIG_BLK_DEV_IDEDISK_MODULE
> +	case IDE0_MAJOR:
> +	case IDE1_MAJOR:
> +	case IDE2_MAJOR:
> +	case IDE3_MAJOR:
> +	case IDE4_MAJOR:
> +	case IDE5_MAJOR:
> +	case IDE6_MAJOR:
> +	case IDE7_MAJOR:
> +	case IDE8_MAJOR:
> +	case IDE9_MAJOR:
> +	{
> +		ide_drive_t *drive;
> +		unsigned int	h;
> +
> +		for (h = 0; h < MAX_HWIFS; ++h) {
> +			ide_hwif_t  *hwif = &ide_hwifs[h];
> +			if (hwif->present && major == hwif->major) {
> +				unsigned unit = minor >> PARTN_BITS;
> +				if (unit < MAX_DRIVES) {
> +					drive = &hwif->drives[unit];
> +					if (drive->present) {
> +						g_head = drive->head;
> +						g_sect = drive->sect;
> +						goto found;
> +					}
> +				}
> +				break;
> +			}
> +		}
> +	}
> +#endif /* CONFIG_BLK_DEV_IDEDISK(_MODULE) */
> +	default:
> +		printk(" unsupported disk (major = %u)\n", major);
> +		return 0;
> +	}

This is horribly ugly.  Just do an inkernel ioctl instead.  Something
like the following (untested!):

	struct hd_geometry geo;
	int err;

	err = ioctl_by_bdev(bdev, HDIO_GETGEO, &geo);
	if (err)
		return err;
	g_head = geo.heads;
	g_sect = geo.sectors;


-
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: 2005-03-22 13:33    [from the cache]
©2003-2008