linux-kernel - Re: [PATCH] asm-generic headers: modify stat.h in include/asm-generic to be applicable to more architectures

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: <201101090128.25371.arnd@arndb.de>
Date:	Sun, 9 Jan 2011 01:28:25 +0100
From:	Arnd Bergmann <arnd@...db.de>
To:	"Guan Xuetao" <guanxuetao@...c.pku.edu.cn>
Cc:	linux-arch@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] asm-generic headers: modify stat.h in include/asm-generic to be applicable to more architectures

On Saturday 08 January 2011, Guan Xuetao wrote:
> From: Guan Xuetao <guanxuetao@...c.pku.edu.cn>
> 
>  This patch modifies stat.h in include/asm-generic to be applicable to more architectures.
>  STAT64_HAS_BROKEN_ST_INO is defined in most architecture's asm/stat.h, and it need 
>  32-bit __st_ino member to be defined in different position of 64-bit st_ino member.
>  STAT64_PAD_BEFORE_ST_SIZE is the pad before st_size member, with default value 8 bytes.
>  STAT64_PAD_BEFORE_ST_BLOCKS is the pad before st_blocks member, with default value
>  4 bytes to align the following member to 64-bit.

I'd prefer not to apply this patch. It makes the generic header
significantly more complex, and I can't see a significant benefit.

The existing architectures would all still have to define the macros
you test and also keep defining stuff like __old_kernel_stat, while
risking to introduce bugs while changing to the common header. We've
done similar tricks in other places, where the differences between
architectures are smaller, but this one doesn't seem worth it
unless we can get to the point where we can define struct stat in
linux/stat.h for everyone and only do the macros for the architectures
that need it.

	Arnd
--
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/