Re: libparted/linux.c include broken copy of linux/fs.h

[Yury V. Umanets]

On Mon, 2003-11-10 at 13:33, Sven Luther wrote:
> Hello,
> 
> I have been battling since a week or so with a problem which appeared
> now that debian glibc include a copy of the linux 2.6.0-test9 includes.
> 
> The linux.c file in the libparted directory contains its own copy of the
> some of the linux/fs.h headers, which breaks now that the linux/fs.h
> headers have changed some (to fix a bug on 32bit userland on 64bit
> kernels i think).
> 
> The incriminated defines are :
> 
> #define BLKGETLASTSECT  _IO(0x12,108) /* get last sector of block device */
> #define BLKSETLASTSECT  _IO(0x12,109) /* set last sector of block device */

> #define BLKGETSIZE64 _IOR(0x12,114,sizeof(uint64_t))      /* return device 
> size in bytes (u64 *arg) */
> 
I guess, that this define should look like the following:
#define BLKGETSIZE64 _IOR(0x12,114,uint64_t)

We have recently fased with this problem in reiser4progs. One of testers
have pointed us to this.


> In linux/fs.h, the two first have dissapeared, and this part is replaced
> by :
> 
> /* A jump here: 108-111 have been used for various private purposes. */
> #define BLKBSZGET  _IOR(0x12,112,size_t)
> #define BLKBSZSET  _IOW(0x12,113,size_t)
> #define BLKGETSIZE64 _IOR(0x12,114,size_t)      /* return device size in 
> bytes (u64 *arg) */
> 
> Now, i don't know if the 112 and 113 ioctl can be called replacement for
> the  BLKGETLASTSECT and BLKSETLASTSECT, but the above comment seems
> suspicious to me.
> 

> Anyway, i will try building with the sizeof(uint64_t) changed to size_t,
> but i wonder what should happen to the BLKGETLASTSECT and BLKSETLASTSECT
> ioctls.
> 
> Friendly,
> 
> Sven Luther
> 
> 
> 
> _______________________________________________
> Bug-parted mailing list
> address@hidden
> http://mail.gnu.org/mailman/listinfo/bug-parted
-- 
umka