Re: 2.5.1-pre8 oopses on non existing acorn partition

[Jens Axboe]

--k1lZvvs/B4yU6o8G
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

On Tue, Dec 11 2001, Bas Vermeulen wrote:
> 2.5.1-pre8 oopses in adfspart_check_ICS (probably the put_dev_sector, 
> not entirely sure, since there doesn't seem to be anything wrong).
> I've enabled advanced partitions, and all the partition types.
> Disabling advanced partitions fixes it.

Please try attached patch.

-- 
Jens Axboe

--k1lZvvs/B4yU6o8G
Content-Type: text/plain; charset=us-ascii
Content-Disposition: attachment; filename=acorn-part-1

--- /opt/kernel/linux-2.5.1-pre9/fs/partitions/acorn.c	Mon Oct  1 23:03:26 2001
+++ fs/partitions/acorn.c	Tue Dec 11 06:21:54 2001
@@ -162,12 +162,12 @@
 		struct adfs_discrecord *dr;
 		unsigned int nr_sects;
 
-		if (!(minor & mask))
-			break;
-
 		data = read_dev_sector(bdev, start_blk * 2 + 6, &sect);
 		if (!data)
 			return -1;
+
+		if (!(minor & mask))
+			break;
 
 		dr = adfs_partition(hd, name, data, first_sector, minor++);
 		if (!dr)

--k1lZvvs/B4yU6o8G--