linux-kernel - Re: [PATCH 29/30] microblaze_mmu

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <49F5A5B2.7000307@monstr.eu>
Date:	Mon, 27 Apr 2009 14:31:46 +0200
From:	Michal Simek <monstr@...str.eu>
To:	Arnd Bergmann <arnd@...db.de>
CC:	Christoph Hellwig <hch@...radead.org>,
	linux-kernel@...r.kernel.org, john.williams@...alogix.com
Subject: Re: [PATCH 29/30] microblaze_mmu_v1: stat.h MMU update

Arnd Bergmann wrote:
> On Monday 27 April 2009, Michal Simek wrote:
>> Christoph Hellwig wrote:
> 
>>> Given that microblaze only got merged in the 2.6.30 window I would
>>> suggest fixing up the nommu variant.
>> ok. Let's do it.
>> Here are stats structures from xtensa.
>>
>> Arnd: Is it ok for asm-generic?
> 
> I carry a (not cleaned up) tree with changes in my playground
> (git.kernel.org/pub/scm/linux/kernel/git/arnd/playground.git)
> as branch generic-microblaze, where I try to use generic
> headers for everything. See below for my version of stat.h.
> In particular, it uses 64 bit st_size and st_blocks, but 32 bit
> time fields. It also defines struct stat to be identical to
> struct stat64 on 64 bit systems. 32 bit systems should only
> provide the stat64 syscalls, so the shorter types in struct
> stat do not hurt.
> 
> 	Arnd <><
> 
> ---
> #ifndef __ASM_GENERIC_STAT_H
> #define __ASM_GENERIC_STAT_H
> 
> /*
>  * Everybody gets this wrong and has to stick with it for all
>  * eternity. Hopefully, this version gets used by new architectures
>  * so they don't fall into the same traps.
>  *
>  * stat64 is copied from powerpc64, with explicit padding added.
>  * stat is the same structure layout on 64-bit, without the 'long long'
>  * types.
>  *
>  * By convention, 64 bit architectures use the stat interface, while
>  * 32 bit architectures use the stat64 interface. Note that we don't
>  * provide an __old_kernel_stat here, which new architecture should
>  * not have to start with.
>  */
> 
> #include <asm/bitsperlong.h>
> 
> #define STAT_HAVE_NSEC 1
> 
> struct stat {
> 	unsigned long	st_dev;		/* Device.  */
> 	unsigned long	st_ino;		/* File serial number.  */
> 	unsigned int	st_mode;	/* File mode.  */
> 	unsigned int	st_nlink;	/* Link count.  */
> 	unsigned int	st_uid;		/* User ID of the file's owner.  */
> 	unsigned int	st_gid;		/* Group ID of the file's group. */
> 	unsigned long	st_rdev;	/* Device number, if device.  */
> 	unsigned long	__pad1;
> 	long		st_size;	/* Size of file, in bytes.  */

Maybe unsigned? Make more sense to me that file size is not minus.
And for some types below.

Michal


> 	int		st_blksize;	/* Optimal block size for I/O.  */


> 	int		__pad2;
> 	long		st_blocks;	/* Number 512-byte blocks allocated. */

> 	int		st_atime;	/* Time of last access.  */
> 	unsigned int	st_atime_nsec;
> 	int		st_mtime;	/* Time of last modification.  */
> 	unsigned int	st_mtime_nsec;
> 	int		st_ctime;	/* Time of last status change.  */
> 	unsigned int	st_ctime_nsec;
> 	unsigned int	__unused4;
> 	unsigned int	__unused5;
> };
> 
> #if __BITS_PER_LONG != 64
> /* This matches struct stat64 in glibc2.1. Only used for 32 bit. */
> struct stat64 {
> 	unsigned long long st_dev;	/* Device.  */
> 	unsigned long long st_ino;	/* File serial number.  */
> 	unsigned int	st_mode;	/* File mode.  */
> 	unsigned int	st_nlink;	/* Link count.  */
> 	unsigned int	st_uid;		/* User ID of the file's owner.  */
> 	unsigned int	st_gid;		/* Group ID of the file's group. */
> 	unsigned long long st_rdev;	/* Device number, if device.  */
> 	unsigned long long __pad1;
> 	long long	st_size;	/* Size of file, in bytes.  */
> 	int		st_blksize;	/* Optimal block size for I/O.  */
> 	int		__pad2;
> 	long long	st_blocks;	/* Number 512-byte blocks allocated. */
> 	int		st_atime;	/* Time of last access.  */
> 	unsigned int	st_atime_nsec;
> 	int		st_mtime;	/* Time of last modification.  */
> 	unsigned int	st_mtime_nsec;
> 	int		st_ctime;	/* Time of last status change.  */
> 	unsigned int	st_ctime_nsec;
> 	unsigned int	__unused4;
> 	unsigned int	__unused5;
> };
> #endif
> 
> #endif /* __ASM_GENERIC_STAT_H */


-- 
Michal Simek, Ing. (M.Eng)
w: www.monstr.eu p: +42-0-721842854
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/