Re: 2.6.5-mm2 build problem

[Dominik Karall <dominik.karall@gmx.net>]

take a look at the main 2.6.5-mm2 thread. here is a copy of the message:

----------  Forwarded Message  ----------

Subject: Re: 2.6.5-mm2
Date: Wednesday 07 April 2004 09:10
From: Andrew Morton <akpm@osdl.org>
To: Brice Goglin <Brice.Goglin@ens-lyon.fr>
Cc: linux-kernel@vger.kernel.org, James Bottomley 
<James.Bottomley@steeleye.com>

Brice Goglin <Brice.Goglin@ens-lyon.fr> wrote:
> > ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/patches/2.6/2.6.5/2.6.5
> >-mm2/
> >
>  > - Merged up Ian Kent's autofs4 patches
>  >
>  > - Various fixes and speedups.
>
>  Hi Andrew,
>
>  When building on my Compaq EvoN600c, I get this compile error :
>
>  CC [M]  drivers/scsi/sr.o
>  	    drivers/scsi/sr.c: In function scsi_cd_get':
>  	    drivers/scsi/sr.c:128: error: structure has no member named kobj'

It looks like Mr SCSI forgot to commit his changes to sr.h.

Here's a backout patch which should get you going again.


 25-akpm/drivers/scsi/sr.c |   68
 ++++++---------------------------------------- 1 files changed, 10
 insertions(+), 58 deletions(-)

diff -puN drivers/scsi/sr.c~sr-build-fix drivers/scsi/sr.c
--- 25/drivers/scsi/sr.c~sr-build-fix	2004-04-07 00:07:47.559098560 -0700
+++ 25-akpm/drivers/scsi/sr.c	2004-04-07 00:08:02.302857168 -0700
@@ -113,28 +113,6 @@ static struct cdrom_device_ops sr_dops =
 	.generic_packet		= sr_packet,
 };

-static void sr_kobject_release(struct kobject *kobj);
-
-static struct kobj_type scsi_cdrom_kobj_type = {
-	.release = sr_kobject_release,
-};
-
-/*
- * The get and put routines for the struct scsi_cd.  Note this entity
- * has a scsi_device pointer and owns a reference to this.
- */
-static inline int scsi_cd_get(struct scsi_cd *cd)
-{
-	if (!kobject_get(&cd->kobj))
-		return -ENODEV;
-	return 0;
-}
-
-static inline void scsi_cd_put(struct scsi_cd *cd)
-{
-	kobject_put(&cd->kobj);
-}
-
 /*
  * This function checks to see if the media has been changed in the
  * CDROM drive.  It is possible that we have already sensed a change,
@@ -446,15 +424,8 @@ static int sr_block_open(struct inode *i

 static int sr_block_release(struct inode *inode, struct file *file)
 {
-	int ret;
 	struct scsi_cd *cd = scsi_cd(inode->i_bdev->bd_disk);
-	ret = cdrom_release(&cd->cdi, file);
-	if(ret)
-		return ret;
-
-	scsi_cd_put(cd);
-
-	return 0;
+	return cdrom_release(&cd->cdi, file);
 }

 static int sr_block_ioctl(struct inode *inode, struct file *file, unsigned
 cmd, @@ -496,7 +467,7 @@ static int sr_open(struct cdrom_device_i
 	struct scsi_device *sdev = cd->device;
 	int retval;

-	retval = scsi_cd_get(cd);
+	retval = scsi_device_get(sdev);
 	if (retval)
 		return retval;

@@ -518,7 +489,7 @@ static int sr_open(struct cdrom_device_i
 	return 0;

 error_out:
-	scsi_cd_put(cd);
+	scsi_device_put(sdev);
 	return retval;
 }

@@ -529,6 +500,7 @@ static void sr_release(struct cdrom_devi
 	if (cd->device->sector_size > 2048)
 		sr_set_blocklength(cd, 2048);

+	scsi_device_put(cd->device);
 }

 static int sr_probe(struct device *dev)
@@ -542,18 +514,12 @@ static int sr_probe(struct device *dev)
 	if (sdev->type != TYPE_ROM && sdev->type != TYPE_WORM)
 		goto fail;

-	if ((error = scsi_device_get(sdev)) != 0)
-		goto fail;
-
 	error = -ENOMEM;
 	cd = kmalloc(sizeof(*cd), GFP_KERNEL);
 	if (!cd)
-		goto fail_put_sdev;
+		goto fail;
 	memset(cd, 0, sizeof(*cd));

-	kobject_init(&cd->kobj);
-	cd->kobj.ktype = &scsi_cdrom_kobj_type;
-
 	disk = alloc_disk(1);
 	if (!disk)
 		goto fail_free;
@@ -622,8 +588,6 @@ fail_put:
 	put_disk(disk);
 fail_free:
 	kfree(cd);
-fail_put_sdev:
-	scsi_device_put(sdev);
 fail:
 	return error;
 }
@@ -899,32 +863,20 @@ static int sr_packet(struct cdrom_device
 	return cgc->stat;
 }

-static void sr_kobject_release(struct kobject *kobj)
+static int sr_remove(struct device *dev)
 {
-	struct scsi_cd *cd = container_of(kobj, struct scsi_cd, kobj);
-	struct scsi_device *sdev = cd->device;
+	struct scsi_cd *cd = dev_get_drvdata(dev);
+
+	del_gendisk(cd->disk);

 	spin_lock(&sr_index_lock);
 	clear_bit(cd->disk->first_minor, sr_index_bits);
 	spin_unlock(&sr_index_lock);

-	unregister_cdrom(&cd->cdi);
-
 	put_disk(cd->disk);
-
+	unregister_cdrom(&cd->cdi);
 	kfree(cd);

-	scsi_device_put(sdev);
-}
-
-static int sr_remove(struct device *dev)
-{
-	struct scsi_cd *cd = dev_get_drvdata(dev);
-
-	del_gendisk(cd->disk);
-
-	scsi_cd_put(cd);
-
 	return 0;
 }


On Wednesday 07 April 2004 12:37, Ralf Hildebrandt wrote:
> 2.6.5-mm1 builds OK, 2.6.5-mm2 does not:
>
>   CC [M]  drivers/scsi/scsicam.o
>   CC [M]  drivers/scsi/scsi_error.o
>   CC [M]  drivers/scsi/scsi_lib.o
>   CC [M]  drivers/scsi/scsi_scan.o
>   CC [M]  drivers/scsi/scsi_syms.o
>   CC [M]  drivers/scsi/scsi_sysfs.o
>   CC [M]  drivers/scsi/scsi_devinfo.o
>   CC [M]  drivers/scsi/scsi_sysctl.o
>   CC [M]  drivers/scsi/scsi_proc.o
>   CC [M]  drivers/scsi/sd.o
>   CC [M]  drivers/scsi/sr.o
> drivers/scsi/sr.c: In function scsi_cd_get':
> drivers/scsi/sr.c:128: error: structure has no member named kobj'
> drivers/scsi/sr.c: In function scsi_cd_put':
> drivers/scsi/sr.c:135: error: structure has no member named kobj'
> drivers/scsi/sr.c: In function sr_probe':
> drivers/scsi/sr.c:554: error: structure has no member named kobj'
> drivers/scsi/sr.c:555: error: structure has no member named kobj'
> drivers/scsi/sr.c: In function sr_kobject_release':
> drivers/scsi/sr.c:904: error: structure has no member named kobj'
> drivers/scsi/sr.c:904: warning: type defaults to int' in declaration of
> __mptr' drivers/scsi/sr.c:904: warning: initialization from incompatible
> pointer type drivers/scsi/sr.c:904: error: structure has no member named
> kobj'
> make[3]: *** [drivers/scsi/sr.o] Error 1
> make[2]: *** [drivers/scsi] Error 2
> make[1]: *** [drivers] Error 2
> make[1]: Leaving directory /usr/src/linux-2.6.5-mm2'
> make: *** [stamp-build] Error 2
>
> My .config is here:
> http://www.stahl.bau.tu-bs.de/~hildeb/satellitepro/kernel-config-2.6