at the end of the list
Signed-off-by: Michał Żygowski
Signed-off-by: Tomasz Żyjewski
Signed-off-by: Krystian Hebel
Signed-off-by: Sergii Dmytruk
---
grub-core/loader/multiboot.c | 11 ++
grub-core/loader/multiboot_elfxx.c| 91 +++-
grub-core/loader/multiboot_mbi2.c
Unallocated memory is cleared before loading an image. This is meant to
make measurements of DLME more deterministic in the face of firmware
that doesn't bother to zero area containing padding.
Signed-off-by: Sergii Dmytruk
---
grub-core/loader/efi/chainloader.c
duplication legacy boot is performed through the stub as well,
but GRUB calls it directly and DLME is started only once.
Signed-off-by: Ross Philipson
Signed-off-by: Sergii Dmytruk
---
grub-core/loader/slaunch/x86_dlstub.c | 93
grub-core/loader/slaunch/x86_dltrampoline.S
From: Ross Philipson
Signed-off-by: Ross Philipson
Signed-off-by: Daniel Kiper
Signed-off-by: Sergii Dmytruk
---
include/grub/i386/txt.h | 720
1 file changed, 720 insertions(+)
create mode 100644 include/grub/i386/txt.h
diff --git a/include/grub
: Daniel Kiper
Signed-off-by: Krystian Hebel
Signed-off-by: Sergii Dmytruk
Reviewed-by: Alec Brown
---
include/grub/i386/cpuid.h | 13 +++
include/grub/i386/crfr.h | 190 ++
include/grub/i386/mmio.h | 74 +++
include/grub/i386/msr.h | 63
Signed-off-by: Sergii Dmytruk
---
grub-core/loader/slaunch/slrt.c | 37 -
include/grub/slaunch.h | 10 +
2 files changed, 46 insertions(+), 1 deletion(-)
diff --git a/grub-core/loader/slaunch/slrt.c b/grub-core/loader/slaunch/slrt.c
index
From: Ross Philipson
Signed-off-by: Ross Philipson
Signed-off-by: Daniel Kiper
Signed-off-by: Sergii Dmytruk
---
grub-core/loader/slaunch/acmod.c | 575 +++
1 file changed, 575 insertions(+)
create mode 100644 grub-core/loader/slaunch/acmod.c
diff --git a/grub
From: Ross Philipson
Signed-off-by: Ross Philipson
Signed-off-by: Sergii Dmytruk
---
grub-core/Makefile.am | 3 ++
include/grub/slaunch.h | 108 +
2 files changed, 111 insertions(+)
create mode 100644 include/grub/slaunch.h
diff --git a/grub-core
f the amount of extra memory that follows it.
Signed-off-by: Sergii Dmytruk
---
grub-core/loader/i386/multiboot_mbi.c | 4 +++-
grub-core/loader/multiboot.c | 3 ++-
grub-core/loader/multiboot_mbi2.c | 6 --
include/grub/multiboot.h | 2 +-
include/grub/multibo
Convert part of grub_vbe_enable_mtrr () in video/i386/pc/vbe.c to an
inline function in grub/i386/cpuid.h to enable its reuse.
Signed-off-by: Sergii Dmytruk
---
grub-core/video/i386/pc/vbe.c | 11 +--
include/grub/i386/cpuid.h | 14 ++
2 files changed, 15 insertions
From: Ross Philipson
Signed-off-by: Ross Philipson
Signed-off-by: Daniel Kiper
Signed-off-by: Sergii Dmytruk
---
grub-core/loader/slaunch/verify.c | 297 ++
1 file changed, 297 insertions(+)
create mode 100644 grub-core/loader/slaunch/verify.c
diff --git a/grub
From: Ross Philipson
This is for Linux with its EFI stub, but the same Secure Launch code can
work with something like Xen once chainloader is made aware of Secure
Launch.
Signed-off-by: Ross Philipson
Signed-off-by: Sergii Dmytruk
---
grub-core/Makefile.core.def| 2 +
grub-core
: Daniel Kiper
Signed-off-by: Sergii Dmytruk
---
docs/grub.texi| 9 +
grub-core/Makefile.core.def | 13 ++
grub-core/lib/i386/relocator32.S | 8 +
grub-core/loader/i386/bsd.c | 3 +
grub-core/loader/i386/linux.c | 85 -
grub
From: Ross Philipson
Introduce the Secure Launch Resource Table which forms the formal
interface between the pre and post launch code.
Signed-off-by: Ross Philipson
Signed-off-by: Sergii Dmytruk
---
grub-core/Makefile.am| 3 +
include/grub/slr_table.h | 297
: Sergii Dmytruk
---
docs/grub.texi| 8 ++
grub-core/Makefile.core.def | 6 ++
grub-core/commands/i386/tpm.c | 151 ++
include/grub/i386/tpm.h | 35
4 files changed, 200 insertions(+)
create mode 100644 grub-core/commands/i386
From: Ross Philipson
Replace UEFI tpm measuring command with TPM logging function, allowing
the removal of the tpm command file.
Specifying the license in commands/efi/tpm.c to make sure the code still
builds.
Signed-off-by: Ross Philipson
Signed-off-by: Sergii Dmytruk
---
grub-core
From: Daniel Kiper
The functions find the lowest and highest values with regard to the
passed in limit. Passing a low limit of 0 or a high limit of ~0
calculates lowest and highest available RAM addresses respectively.
Signed-off-by: Daniel Kiper
Signed-off-by: Sergii Dmytruk
Reviewed-by
ons header file
slaunch/txt: Add Intel TXT core implementation
slaunch/txt: Add Intel TXT ACM module support
slaunch/txt: Add Intel TXT verification routines
i386/efi: Add DL stub as common DL event module
efi/linux: Add Intel TXT Secure Launch support
Sergii Dmytruk (4):
i38
From: Ross Philipson
Signed-off-by: Ross Philipson
Signed-off-by: Daniel Kiper
Signed-off-by: Michał Żygowski
Signed-off-by: Krystian Hebel
Signed-off-by: Sergii Dmytruk
---
grub-core/loader/slaunch/txt.c | 1114
include/grub/i386/memory.h |5 +
2
From: Aleksandr Burmashev
This avoids naming collision with TPM TIS and CRB driver introduced
by subsequent patch and characterizes the functionality of the module
better.
Signed-off-by: Daniel Kiper
Signed-off-by: Sergii Dmytruk
---
docs/grub.texi| 21 +++---
grub-core
From: Ross Philipson
Signed-off-by: Ross Philipson
Signed-off-by: Sergii Dmytruk
---
grub-core/loader/slaunch/slrt.c | 292
1 file changed, 292 insertions(+)
create mode 100644 grub-core/loader/slaunch/slrt.c
diff --git a/grub-core/loader/slaunch/slrt.c b
From: Ross Philipson
Extend code introduced for legacy boot with AMD Secure Launch to support
EFI.
This is for Linux with its EFI stub and also for anything else that can
be loaded by chainloader.
Signed-off-by: Ross Philipson
Signed-off-by: Sergii Dmytruk
---
grub-core/loader/efi
From: Michał Żygowski
Signed-off-by: Michał Żygowski
Signed-off-by: Tomasz Żyjewski
Signed-off-by: Krystian Hebel
Signed-off-by: Sergii Dmytruk
---
grub-core/loader/multiboot_mbi2.c | 14 +-
grub-core/loader/slaunch/skl.c| 2 +-
2 files changed, 14 insertions(+), 2
From: Ross Philipson
Add core implementation for AMD SKINIT Secure Launch in GRUB based on
Linux legacy boot protocol.
Signed-off-by: Ross Philipson
Signed-off-by: Sergii Dmytruk
---
grub-core/Makefile.core.def | 2 +
grub-core/lib/i386/relocator32.S | 6 +
grub-core
From: Ross Philipson
A few MSR/CPUID/PCI x86 definitions added.
Signed-off-by: Ross Philipson
Signed-off-by: Sergii Dmytruk
---
include/grub/i386/cpuid.h | 7 ---
include/grub/i386/msr.h | 6 ++
include/grub/i386/pci.h | 1 +
3 files changed, 11 insertions(+), 3 deletions
From: Alec Brown
Add base code for PSP discovery in grub.
Signed-off-by: Alec Brown
Signed-off-by: Sergii Dmytruk
---
grub-core/Makefile.core.def| 1 +
grub-core/loader/slaunch/psp.c | 133 +
include/grub/i386/psp.h| 42 +++
3 files
From: Alec Brown
TMRs are setup and used to protect ranges of memory from outside
access like DMA. Setting them up to cover all memory protects from
DMA during the establishment of the DRTM environment.
Signed-off-by: Alec Brown
Signed-off-by: Sergii Dmytruk
---
grub-core/loader/slaunch
From: Alec Brown
Add various core DRTM commands for the PSP.
Signed-off-by: Alec Brown
Signed-off-by: Sergii Dmytruk
---
grub-core/loader/slaunch/psp.c | 235 -
include/grub/i386/psp.h| 48 +++
2 files changed, 281 insertions(+), 2 deletions
This is a refresh of [rfc-v1] (no substantial changes) to accompany v4 of the
main patch set, see more details there. Giving this a try and providing
feedback is appreciated.
The changes can also be viewed on GitHub at [TrenchBoot/grub].
-
[rfc-v1]: https://lists.gnu.org/archive/html/grub-d
From: Ross Philipson
Changes built on the AMD Secure Launch base support for legacy Linux
this allows booting through the kernel's EFI stub and dlstub to start a
measured launch on AMD platforms.
Signed-off-by: Ross Philipson
Signed-off-by: Sergii Dmytruk
---
grub-core/loade
From: Michał Żygowski
Signed-off-by: Michał Żygowski
Signed-off-by: Tomasz Żyjewski
Signed-off-by: Krystian Hebel
Signed-off-by: Sergii Dmytruk
---
grub-core/loader/multiboot_mbi2.c | 14 +-
grub-core/loader/slaunch/skl.c| 2 +-
2 files changed, 14 insertions(+), 2
From: Ross Philipson
Add core implementation for AMD SKINIT Secure Launch in GRUB based on
Linux legacy boot protocol.
Signed-off-by: Ross Philipson
Signed-off-by: Sergii Dmytruk
---
grub-core/Makefile.core.def | 2 +
grub-core/lib/i386/relocator32.S | 6 +
grub-core
NOTE: this patch set follows up on Intel TXT DRTM patches that are
currently under review in their 3rd version [v3]; therefore, it is not
standalone! This targets v11 of Linux patches [linux-v11] and
corresponding RFC patch set for AMD SKINIT on top of it [linux-amd-rfc].
The publication of the pa
From: Alec Brown
Add base code for PSP discovery in grub.
Signed-off-by: Alec Brown
Signed-off-by: Sergii Dmytruk
---
grub-core/Makefile.core.def| 1 +
grub-core/loader/slaunch/psp.c | 133 +
include/grub/i386/psp.h| 42 +++
3 files
From: Alec Brown
Add various core DRTM commands for the PSP.
Signed-off-by: Alec Brown
Signed-off-by: Sergii Dmytruk
---
grub-core/loader/slaunch/psp.c | 235 -
include/grub/i386/psp.h| 48 +++
2 files changed, 281 insertions(+), 2 deletions
From: Ross Philipson
A few MSR/CPUID/PCI x86 definitions added.
Signed-off-by: Ross Philipson
Signed-off-by: Sergii Dmytruk
---
include/grub/i386/cpuid.h | 7 ---
include/grub/i386/msr.h | 6 ++
include/grub/i386/pci.h | 1 +
3 files changed, 11 insertions(+), 3 deletions
From: Alec Brown
TMRs are setup and used to protect ranges of memory from outside
access like DMA. Setting them up to cover all memory protects from
DMA during the establishment of the DRTM environment.
Signed-off-by: Alec Brown
Signed-off-by: Sergii Dmytruk
---
grub-core/loader/slaunch
: Daniel Kiper
Signed-off-by: Sergii Dmytruk
---
docs/grub.texi| 9 ++
grub-core/Makefile.core.def | 13 ++
grub-core/lib/i386/relocator32.S | 8 +
grub-core/loader/i386/bsd.c | 3 +
grub-core/loader/i386/linux.c | 87
Signed-off-by: Sergii Dmytruk
---
grub-core/loader/slaunch/slrt.c | 37 -
include/grub/slaunch.h | 10 +
2 files changed, 46 insertions(+), 1 deletion(-)
diff --git a/grub-core/loader/slaunch/slrt.c b/grub-core/loader/slaunch/slrt.c
index
From: Ross Philipson
Signed-off-by: Ross Philipson
Signed-off-by: Daniel Kiper
Signed-off-by: Sergii Dmytruk
---
grub-core/loader/slaunch/txt.c | 935 +
include/grub/i386/linux.h | 5 +-
include/grub/i386/memory.h | 5 +
3 files changed, 944
From: Ross Philipson
Signed-off-by: Ross Philipson
Signed-off-by: Sergii Dmytruk
---
grub-core/loader/efi/dltrampoline.S | 94 +
grub-core/loader/slaunch/dlstub.c | 93
2 files changed, 187 insertions(+)
create mode 100644 grub-core
From: Ross Philipson
Introduce the Secure Launch Resource Table which forms the formal
interface between the pre and post launch code.
Signed-off-by: Ross Philipson
Signed-off-by: Sergii Dmytruk
---
grub-core/Makefile.am| 3 +
include/grub/slr_table.h | 296
From: Ross Philipson
Signed-off-by: Ross Philipson
Signed-off-by: Daniel Kiper
Signed-off-by: Sergii Dmytruk
---
grub-core/loader/slaunch/verify.c | 297 ++
1 file changed, 297 insertions(+)
create mode 100644 grub-core/loader/slaunch/verify.c
diff --git a/grub
From: Ross Philipson
Signed-off-by: Ross Philipson
Signed-off-by: Daniel Kiper
Signed-off-by: Sergii Dmytruk
---
grub-core/loader/slaunch/acmod.c | 575 +++
1 file changed, 575 insertions(+)
create mode 100644 grub-core/loader/slaunch/acmod.c
diff --git a/grub
: Sergii Dmytruk
---
docs/grub.texi| 8 ++
grub-core/Makefile.core.def | 6 ++
grub-core/commands/i386/tpm.c | 151 ++
include/grub/i386/tpm.h | 35
4 files changed, 200 insertions(+)
create mode 100644 grub-core/commands/i386
From: Michał Żygowski
Signed-off-by: Michał Żygowski
Signed-off-by: Krystian Hebel
Signed-off-by: Sergii Dmytruk
---
grub-core/loader/slaunch/txt.c | 68 +-
include/grub/i386/txt.h| 33 +
2 files changed, 100 insertions(+), 1 deletion
f the amount of extra memory that follows it.
Signed-off-by: Sergii Dmytruk
---
grub-core/loader/i386/multiboot_mbi.c | 4 +++-
grub-core/loader/multiboot.c | 3 ++-
grub-core/loader/multiboot_mbi2.c | 6 --
include/grub/multiboot.h | 2 +-
include/grub/multibo
From: Ross Philipson
Signed-off-by: Ross Philipson
Signed-off-by: Daniel Kiper
Signed-off-by: Sergii Dmytruk
---
include/grub/i386/txt.h | 673
1 file changed, 673 insertions(+)
create mode 100644 include/grub/i386/txt.h
diff --git a/include/grub
From: Aleksandr Burmashev
This avoids naming collision with TPM TIS and CRB driver introduced
by subsequent patch and characterizes the functionality of the module
better.
Signed-off-by: Daniel Kiper
Signed-off-by: Sergii Dmytruk
---
docs/grub.texi| 21 +++---
grub-core
From: Ross Philipson
Signed-off-by: Ross Philipson
Signed-off-by: Sergii Dmytruk
---
grub-core/loader/slaunch/slrt.c | 285
1 file changed, 285 insertions(+)
create mode 100644 grub-core/loader/slaunch/slrt.c
diff --git a/grub-core/loader/slaunch/slrt.c b
: Daniel Kiper
Signed-off-by: Krystian Hebel
Signed-off-by: Sergii Dmytruk
Reviewed-by: Alec Brown
---
include/grub/i386/cpuid.h | 13 +++
include/grub/i386/crfr.h | 190 ++
include/grub/i386/mmio.h | 74 +++
include/grub/i386/msr.h | 61
From: Ross Philipson
Signed-off-by: Ross Philipson
Signed-off-by: Sergii Dmytruk
---
grub-core/Makefile.am | 3 ++
include/grub/slaunch.h | 96 ++
2 files changed, 99 insertions(+)
create mode 100644 include/grub/slaunch.h
diff --git a/grub-core
Signed-off-by: Krystian Hebel
Signed-off-by: Sergii Dmytruk
---
grub-core/loader/multiboot.c | 11 +++
grub-core/loader/multiboot_elfxx.c | 90 +++-
grub-core/loader/multiboot_mbi2.c | 132 +
grub-core/loader/slaunch/dlstub.c | 2 +-
include
From: Ross Philipson
Replace UEFI tpm measuring command with TPM logging function, allowing the
removal of the tpm command file.
Signed-off-by: Ross Philipson
Signed-off-by: Sergii Dmytruk
---
grub-core/Makefile.core.def | 1 -
grub-core/commands/efi/tpm.c | 2 +-
grub-core/commands
odule
efi: Add Secure Launch support for efi/linux boot through EFI stub
Sergii Dmytruk (2):
multiboot: Make GRUB_MULTIBOOT(make_mbi) return MBI's size
slaunch: Introduce a hook for filling SLRT policy
docs/grub.texi | 32 +-
grub-core/M
From: Ross Philipson
Signed-off-by: Ross Philipson
Signed-off-by: Sergii Dmytruk
---
grub-core/Makefile.core.def | 1 +
grub-core/loader/efi/linux.c | 16 ++
grub-core/loader/slaunch/x86_efi_linux.c | 212 +++
include/grub/slaunch.h
From: Daniel Kiper
The functions find the lowest and highest values with regard to the
passed in limit. Passing a low limit of 0 or a high limit of ~0
calculates lowest and highest available RAM addresses respectively.
Signed-off-by: Daniel Kiper
Signed-off-by: Sergii Dmytruk
Reviewed-by
From: Krystian Hebel
GRUB_PAGE_MASK is intentionally signed to make use of sign extension.
Subsequent patches will use them.
Signed-off-by: Krystian Hebel
Signed-off-by: Sergii Dmytruk
Reviewed-by: Ross Philipson
---
include/grub/i386/memory.h | 5 +
1 file changed, 5 insertions
GRUB_MULTIBOOT(get_mbi_size) returns an upper bound on MBI's size
instead of the actual size. It could also be used for measurements
after zeroing unused parts of the buffer, but using an actual size seems
like a better option as the same MBI will always have the same hash.
Signed-off-by: S
Simply returning from `grub_cmd_linux()` doesn't free `file` resource
nor calls `grub_dl_ref(my_mod)`. Jump to `fail` label for proper
cleanup like other error checks do.
Signed-off-by: Sergii Dmytruk
---
grub-core/loader/i386/linux.c | 8 +---
1 file changed, 5 insertions(+), 3 dele
From: Michał Żygowski
This extends the code to support TPM 1.2 in addition to already
available TPM 2.0.
Signed-off-by: Michał Żygowski
Signed-off-by: Krystian Hebel
Signed-off-by: Sergii Dmytruk
---
grub-core/loader/i386/txt/txt.c | 62 +++--
include/grub/i386
check error reporting status)
- EFER (LME, LMA, SVEM (AMD-V))
- AMD: SVM control
CPUID:
- flags for availability of vendor, features
- Intel: VMX, SMX
- AMD: SVM
Signed-off-by: Ross Philipson
Signed-off-by: Daniel Kiper
Signed-off-by: Krystian Hebel
Signed-off-by: Sergii Dmytruk
Reviewed
From: Krystian Hebel
Hook up this platform in all the places which previously were specific
to Intel TXT.
Signed-off-by: Krystian Hebel
Signed-off-by: Sergii Dmytruk
---
grub-core/loader/multiboot.c | 11 +--
grub-core/loader/multiboot_elfxx.c | 22 ++---
grub-core/loader
ecure launch framework and commands
i386/linux: Add support for AMD SKINIT
Sergii Dmytruk (1):
multiboot: Make GRUB_MULTIBOOT(make_mbi) return MBI's size
docs/grub.texi | 32 +-
grub-core/Makefile.am|6 +
grub-core/Makefil
-off-by: Krystian Hebel
Signed-off-by: Sergii Dmytruk
---
grub-core/loader/multiboot.c | 23 ++-
grub-core/loader/multiboot_elfxx.c | 88 ++-
grub-core/loader/multiboot_mbi2.c | 96 ++
include/grub/multiboot2.h | 2 +
4 files
From: Ross Philipson
Signed-off-by: Ross Philipson
Signed-off-by: Daniel Kiper
Signed-off-by: Sergii Dmytruk
---
grub-core/loader/i386/linux.c | 57 ---
1 file changed, 40 insertions(+), 17 deletions(-)
diff --git a/grub-core/loader/i386/linux.c b/grub-core
From: Daniel Kiper
The functions find the lowest and highest values with regard to the
passed in limit. Passing a low limit of 0 or a high limit of ~0
calculates lowest and highest available RAM addresses respectively.
Signed-off-by: Daniel Kiper
Signed-off-by: Sergii Dmytruk
Reviewed-by
From: Krystian Hebel
Extend slaunch command to work on AMD CPUs and validate input binary to
reject obviously incorrect ones.
Signed-off-by: Krystian Hebel
Signed-off-by: Sergii Dmytruk
---
grub-core/lib/i386/relocator32.S | 6
grub-core/loader/i386/skinit.c | 58
From: Daniel Kiper
This avoids naming collision with TPM TIS and CRB driver introduced
by subsequent patch and characterizes the functionality of the module
better.
Signed-off-by: Daniel Kiper
Signed-off-by: Sergii Dmytruk
---
docs/grub.texi | 21
Hi,
These are generally useful x86-related changes which were originally posted as
part of DRTM patchset [0]. The changes can also be viewed on GitHub [1].
Best regards,
Sergii
Changes in v2:
- updated commit messages
- added missing Signed-off-by
Changes in v3:
- patches 1-3: added reviewe
From: Krystian Hebel
GRUB_PAGE_MASK is intentionally signed to make use of sign extension.
Subsequent patches will use them.
Signed-off-by: Krystian Hebel
Signed-off-by: Sergii Dmytruk
Reviewed-by: Ross Philipson
---
include/grub/i386/memory.h | 5 +
1 file changed, 5 insertions
From: Daniel Kiper
Use more obvious names which match corresponding instructions:
* grub_msr_read() => grub_rdmsr()
* grub_msr_write() => grub_wrmsr()
Signed-off-by: Daniel Kiper
Signed-off-by: Sergii Dmytruk
Reviewed-by: Ross Philipson
Reviewed-by: Daniel Kiper
---
grub-core/co
From: Daniel Kiper
It does not make sense to have separate headers for individual static
functions. So, make one common place to store them.
Signed-off-by: Daniel Kiper
Signed-off-by: Sergii Dmytruk
Reviewed-by: Ross Philipson
Reviewed-by: Daniel Kiper
---
grub-core/commands/i386/rdmsr.c
From: Daniel Kiper
This fixes naming inconsistency that goes against coding style as well
as helps to avoid potential conflicts and confusion as this constant is
used in multiple places.
Signed-off-by: Daniel Kiper
Signed-off-by: Sergii Dmytruk
Reviewed-by: Ross Philipson
Reviewed-by: Daniel
, use GRUB_ERR_BAD_DEVICE instead of GRUB_ERR_BUG to signal
an error because errors encountered by this new routine are not bugs.
Signed-off-by: Daniel Kiper
Signed-off-by: Sergii Dmytruk
Reviewed-by: Ross Philipson
Reviewed-by: Daniel Kiper
---
grub-core/commands/i386/rdmsr.c | 21
From: Daniel Kiper
This is an x86-specific thing and should be available globally.
Signed-off-by: Daniel Kiper
Signed-off-by: Sergii Dmytruk
Reviewed-by: Ross Philipson
Reviewed-by: Daniel Kiper
---
grub-core/loader/i386/xen.c | 35 +--
include/grub/i386
Ping. 7 other patches of this series already have Reviewed-by.
Best regards,
Sergii
On Sun, Sep 22, 2024 at 08:17:54PM +0300, Sergii Dmytruk wrote:
> From: Ross Philipson
>
> Control registers and flags:
> - CR0 read/write and flags (PE, MP, EM, TS, PG, NE, WP, AM, NW, CD)
> -
From: Krystian Hebel
GRUB_PAGE_MASK is intentionally signed to make use of sign extension.
Subsequent patches will use them.
Signed-off-by: Krystian Hebel
Signed-off-by: Sergii Dmytruk
---
include/grub/i386/memory.h | 5 +
1 file changed, 5 insertions(+)
diff --git a/include/grub/i386
, use GRUB_ERR_BAD_DEVICE instead of GRUB_ERR_BUG to signal
an error because errors encountered by this new routine are not bugs.
Signed-off-by: Daniel Kiper
Signed-off-by: Sergii Dmytruk
Reviewed-by: Ross Philipson
---
grub-core/commands/i386/rdmsr.c | 21 +
grub-core
From: Daniel Kiper
This fixes naming inconsistency that goes against coding style as well
as helps to avoid potential conflicts and confusion.
Signed-off-by: Daniel Kiper
Signed-off-by: Sergii Dmytruk
---
grub-core/lib/i386/xen/relocator.S | 6 +++---
grub-core/lib/x86_64/xen/relocator.S
From: Daniel Kiper
It does not make sense to have separate headers for individual static
functions. Additionally, we have to add some constants with MSR
addresses in subsequent patches. So, make one common place to store
them.
Signed-off-by: Daniel Kiper
Signed-off-by: Sergii Dmytruk
Reviewed
From: Daniel Kiper
Use more obvious names which match corresponding instructions:
* grub_msr_read() => grub_rdmsr()
* grub_msr_write() => grub_wrmsr()
Signed-off-by: Daniel Kiper
Signed-off-by: Sergii Dmytruk
Reviewed-by: Ross Philipson
---
grub-core/commands/i386/rdmsr.c | 2 +-
From: Daniel Kiper
Subsequent patches will use that constant.
Signed-off-by: Daniel Kiper
Signed-off-by: Sergii Dmytruk
---
grub-core/loader/i386/xen.c | 35 +--
include/grub/i386/memory.h | 1 +
2 files changed, 18 insertions(+), 18 deletions(-)
diff --git
From: Daniel Kiper
The functions find the lowest and highest values with regard to the
passed in limit. Passing a low limit of 0 or a high limit of ~0
calculates lowest and highest available RAM addresses respectively.
Signed-off-by: Daniel Kiper
Signed-off-by: Sergii Dmytruk
---
grub-core
check error reporting status)
- EFER (LME, LMA, SVEM (AMD-V))
- AMD: SVM control
CPUID:
- flags for availability of vendor, features
- Intel: VMX, SMX
- AMD: SVM
Signed-off-by: Ross Philipson
Signed-off-by: Daniel Kiper
Signed-off-by: Krystian Hebel
Signed-off-by: Sergii Dmytruk
Hi,
These are generally useful x86-related changes which were originally posted as
part of DRTM patchset [0]. The changes can also be viewed on GitHub [1].
Best regards,
Sergii
Changes in v2:
- updated commit messages
- added missing Signed-off-by
Changes in v3:
- patch 6: updated commit me
On Fri, Sep 20, 2024 at 02:40:22PM +0100, Frediano Ziglio via Grub-devel wrote:
> On Thu, Sep 19, 2024 at 11:03 PM Sergii Dmytruk
> wrote:
> >
> > From: Krystian Hebel
> >
> > Subsequent patches will use those macros and constant.
> >
>
> Minor, but &quo
Sent 0-7 and part of 11 separately taking all the comments into account:
https://lists.gnu.org/archive/html/grub-devel/2024-09/msg00194.html
Regards,
Sergii
On Tue, Sep 17, 2024 at 11:06:51AM -0700, Ross Philipson via Grub-devel wrote:
> On 8/26/24 5:44 AM, Sergii Dmytruk wrote:
>
From: Daniel Kiper
The functions calculate lowest and highest available RAM
addresses respectively.
Signed-off-by: Daniel Kiper
Signed-off-by: Sergii Dmytruk
---
grub-core/mmap/mmap.c | 83 +++
include/grub/memory.h | 3 ++
2 files changed, 86
From: Daniel Kiper
This fixes naming inconsistency that goes against coding style as well
as helps to avoid potential conflicts and confusion.
Signed-off-by: Daniel Kiper
Signed-off-by: Sergii Dmytruk
---
grub-core/lib/i386/xen/relocator.S | 6 +++---
grub-core/lib/x86_64/xen/relocator.S
From: Ross Philipson
Signed-off-by: Ross Philipson
Signed-off-by: Daniel Kiper
Signed-off-by: Krystian Hebel
Signed-off-by: Sergii Dmytruk
---
include/grub/i386/cpuid.h | 11
include/grub/i386/crfr.h | 127 ++
include/grub/i386/mmio.h | 72
From: Daniel Kiper
Subsequent patches will use that constant.
Signed-off-by: Daniel Kiper
Signed-off-by: Sergii Dmytruk
---
grub-core/loader/i386/xen.c | 35 +--
include/grub/i386/memory.h | 1 +
2 files changed, 18 insertions(+), 18 deletions(-)
diff --git
From: Daniel Kiper
It does not make sense to have separate headers for individual static
functions. Additionally, we have to add some constants with MSR
addresses in subsequent patches. So, make one common place to store
them.
Signed-off-by: Daniel Kiper
Signed-off-by: Sergii Dmytruk
Hi,
These are generally useful x86-related changes which were originally posted as
part of DRTM patchset [0]. This version took comments there into account,
hence v2. The changes can also be viewed on GitHub [1].
Best regards,
Sergii
[0]: https://lists.gnu.org/archive/html/grub-devel/2024-08/m
From: Krystian Hebel
Subsequent patches will use those macros and constant.
Signed-off-by: Krystian Hebel
Signed-off-by: Sergii Dmytruk
---
include/grub/i386/memory.h | 5 +
1 file changed, 5 insertions(+)
diff --git a/include/grub/i386/memory.h b/include/grub/i386/memory.h
index
From: Daniel Kiper
Use more obvious names which match corresponding instructions:
* grub_msr_read() => grub_rdmsr()
* grub_msr_write() => grub_wrmsr()
Signed-off-by: Daniel Kiper
Signed-off-by: Sergii Dmytruk
---
grub-core/commands/i386/rdmsr.c | 2 +-
grub-core/commands/i386/wrmsr
, use GRUB_ERR_BAD_DEVICE instead of GRUB_ERR_BUG to signal
an error because errors encountered by this new routine are not bugs.
Signed-off-by: Daniel Kiper
Signed-off-by: Sergii Dmytruk
---
grub-core/commands/i386/rdmsr.c | 21 +
grub-core/commands/i386/wrmsr.c | 21
Ping.
Tip of the day: patches 01-07 and 18 are no-brainers.
Initial submission with all the patches can be seen in the archive at
https://lists.gnu.org/archive/html/grub-devel/2024-08/msg00088.html
On Mon, Aug 26, 2024 at 03:44:10PM +0300, Sergii Dmytruk wrote:
> Hello,
>
> [
GRUB_MULTIBOOT(get_mbi_size) doesn't look like an accurate source of the
final size, more like a minimal memory buffer size.
Signed-off-by: Sergii Dmytruk
---
grub-core/loader/i386/multiboot_mbi.c | 4 +++-
grub-core/loader/multiboot.c | 3 ++-
grub-core/loader/multiboot_mbi2.c
From: Ross Philipson
Signed-off-by: Ross Philipson
Signed-off-by: Daniel Kiper
Signed-off-by: Michał Żygowski
Signed-off-by: Krystian Hebel
Signed-off-by: Sergii Dmytruk
---
grub-core/loader/i386/txt/txt.c | 1052 +++
include/grub/i386/slaunch.h | 14 +-
2
1 - 100 of 124 matches
Mail list logo