linux-kernel - Re: [RFC][PATCH 3/3] TXT: Intel(R) TXT and tboot kernel support

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]

Date:	Thu, 9 Oct 2008 15:05:18 +0200
From:	Pavel Machek <pavel@...e.cz>
To:	"Cihula, Joseph" <joseph.cihula@...el.com>
Cc:	linux-kernel@...r.kernel.org, "Wang, Shane" <shane.wang@...el.com>,
	"Wei, Gang" <gang.wei@...el.com>,
	"Van De Ven, Arjan" <arjan.van.de.ven@...el.com>,
	"Mallick, Asit K" <asit.k.mallick@...el.com>,
	"Nakajima, Jun" <jun.nakajima@...el.com>,
	Chris Wright <chrisw@...hat.com>,
	Jan Beulich <jbeulich@...ell.com>, mingo@...e.hu, tytso@....edu
Subject: Re: [RFC][PATCH 3/3] TXT: Intel(R) TXT and tboot kernel support

Hi!

> (large) Patch inline below (and still attached for those who prefer it).
> 
> Linux kernel changes to support tboot and Intel TXT.

Please pick a better acronym. TXT is already taken.

> +struct tboot_acpi_sleep_info {
> +	u16 pm1a_cnt;
> +	u16 pm1b_cnt;
> +	u16 pm1a_evt;
> +	u16 pm1b_evt;
> +	u16 pm1a_cnt_val;
> +	u16 pm1b_cnt_val;
> +}  __attribute__ ((__packed__));
> +
> +struct tboot_shared {
> +	/* version 0x01+ fields: */
> +	struct tboot_uuid uuid; /* TBOOT_SHARED_UUID */
> +	u32  version;      	/* Version number: 2 is current */
> +	u32  log_addr;     	/* physical addr of tb_log_t log */
> +	u32  shutdown_entry32;	/* entry point for tboot shutdown from
> 32b */
> +	u32  shutdown_entry64;	/* entry point for tboot shutdown from
> 64b */

should these be function pointers?

> +	u32  shutdown_type;	/* type of shutdown (TB_SHUTDOWN_*) */
> +	u32  s3_tb_wakeup_entry;/* entry point for tboot s3 wake up */
> +	u32  s3_k_wakeup_entry;	/* entry point for kernel s3 wake up */
> +	struct tboot_acpi_sleep_info
> +		  acpi_sinfo;	/* where kernel put acpi sleep info in
> Sx */
> +	u8   reserved[52];	/* this pad is for compat with old field
> */
> +	/* version 0x02+ fields: */
> +	u32  tboot_base;	/* starting addr for tboot */
> +	u32  tboot_size;	/* size of tboot */
> +} __attribute__ ((__packed__));
> +
> +/* {663C8DFF-E8B3-4b82-AABF-19EA4D057A08} */
> +#define TBOOT_SHARED_UUID     \
> +		((struct tboot_uuid){ 0x663c8dff, 0xe8b3, 0x4b82,
> 0xaabf,   \
> +				{ 0x19, 0xea, 0x4d, 0x5, 0x7a, 0x8 } })

Can we use normal magic number like rest of kernel does?
								Pavel

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
--
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/