On 11/10/22 10:45, Ross Philipson wrote:
While sending an earlier patch set it was discovered that there are a
number of places in early x86 code were the functions early_memremap()
and early_ioremap() are called but the returned pointer is not checked
for NULL. Since NULL can be returned for a
On 11/10/22 11:07, Dave Hansen wrote:
On 11/10/22 07:45, Ross Philipson wrote:
dt = early_memremap(initial_dtb, map_len);
+ if (!dt) {
+ pr_warn("failed to memremap initial dtb\n");
+ return;
+ }
Are all of these new pr_warn/err(
On 11/10/22 13:07, Peter Zijlstra wrote:
On Thu, Nov 10, 2022 at 03:45:21PM +, Ross Philipson wrote:
On allocation failures, panic() was used since this seemed
to be the action taken on other failures in the modules
touched by this patch.
How is the panic() more useful than the obvious
proper usage of pr_*
printing macros.
Ross Philipson (2):
x86: Check return values from early_memremap calls
x86: Check return values from early_ioremap calls
arch/x86/kernel/apic/x2apic_uv_x.c | 2 ++
arch/x86/kernel/devicetree.c | 13 ++
arch/x86/kernel/e820.c
touched by this patch.
Signed-off-by: Ross Philipson
---
arch/x86/kernel/apic/x2apic_uv_x.c | 2 ++
arch/x86/kernel/early_printk.c | 2 ++
arch/x86/kernel/vsmp_64.c | 3 +++
3 files changed, 7 insertions(+)
diff --git a/arch/x86/kernel/apic/x2apic_uv_x.c
b/arch/x86/kernel/apic
failures.
In addition to checking the return values, a bit of extra
cleanup of pr_* usages was done since the pr_fmt macro was
introduced in the modules touched by this patch.
Signed-off-by: Ross Philipson
---
arch/x86/kernel/devicetree.c | 13 +++
arch/x86/kernel/e820.c | 12
On 11/3/22 05:29, 'Jan Beulich' via trenchboot-devel wrote:
On 27.10.2022 21:37, Andrew Cooper wrote:
However, we're also very close to supporting parallel boot. The
serialising point we currently have is __high_start loading %rsp from
stack_start, because that's a single pointer adjusted by do
On 10/8/22 11:12, Borislav Petkov wrote:
Adding Xen and Jailhouse people and MLs to Cc.
Folks, thread starts here:
https://lore.kernel.org/r/1650035401-22855-1-git-send-email-ross.philip...@oracle.com
On Fri, Apr 15, 2022 at 11:10:00AM -0400, Ross Philipson wrote:
There are a number of
CK also confirms that Citrix is happy for the header files to
> be published under a BSD license in this series (which is based on [1]).
>
> Signed-off-by: Christopher Clark
> Acked-by: Lars Kurth
Other than an indentation issue in domain_rings_remove_all, this LGTM.
Reviewed-by: Ros
On 12/06/2018 10:40 AM, Daniel Kiper wrote:
> On Thu, Dec 06, 2018 at 10:37:43AM -0500, Ross Philipson wrote:
>> On 12/06/2018 08:40 AM, Daniel Kiper wrote:
>>> Xen fallout cleanup after commit ca0a4f689 (verifiers: File type for
>>> fine-grained signature-verification c
On 12/06/2018 08:40 AM, Daniel Kiper wrote:
> ARM Xen fallout cleanup after commit ca0a4f689 (verifiers: File type for
> fine-grained signature-verification controlling).
>
> Signed-off-by: Daniel Kiper
> ---
> grub-core/loader/arm64/xen_boot.c | 8
> include/grub/file.h |
On 12/06/2018 08:40 AM, Daniel Kiper wrote:
> Xen fallout cleanup after commit ca0a4f689 (verifiers: File type for
> fine-grained signature-verification controlling).
>
> Signed-off-by: Daniel Kiper
> ---
> grub-core/loader/i386/xen.c | 14 +++---
> 1 file changed, 7 insertions(+), 7 del
On 10/23/2018 12:58 PM, Andrew Cooper wrote:
> On 23/10/18 17:42, Ross Philipson wrote:
>> On 10/23/2018 11:31 AM, Jason Andryuk wrote:
>>> On Tue, Oct 23, 2018 at 10:46 AM Andrew Cooper
>>> wrote:
>>>> On 23/10/18 15:01, Jason Andryuk wrote:
>>>
On 10/23/2018 11:31 AM, Jason Andryuk wrote:
> On Tue, Oct 23, 2018 at 10:46 AM Andrew Cooper
> wrote:
>> On 23/10/18 15:01, Jason Andryuk wrote:
>>> On Tue, Oct 23, 2018 at 7:15 AM Andrew Cooper
>>> wrote:
On 23/10/18 11:59, Sergey Dyasli wrote:
> In certain scenarios, NMIs might be
14 matches
Mail list logo
