On 23.06.2022 19:50, Julien Grall wrote:
> On 11/05/2022 19:47, Oleksandr Tyshchenko wrote:
>> @@ -1505,7 +1507,23 @@ int xenmem_add_to_physmap_one(
>> }
>>
>> /* Map at new location. */
>> -rc = guest_physmap_add_entry(d, gfn, mfn, 0, t);
>
>> +if ( !p2m_is_ram(t) || !is_xe
flight 171334 xen-unstable real [real]
http://logs.test-lab.xenproject.org/osstest/logs/171334/
Failures :-/ but no regressions.
Tests which did not succeed, but are not blocking:
test-amd64-amd64-xl-qemut-win7-amd64 19 guest-stopfail like 171327
test-armhf-armhf-libvirt 16 save
On 23-06-22, 15:47, Oleksandr Tyshchenko wrote:
> Below is my understanding, which might be wrong.
>
> I am not sure about x86, there are some moments with its modes, for example
> PV guests should always use grants for virtio, but on Arm (which guest type
> is HVM):
> 1. If you run backend(s) in
On 23.06.22 20:34, Julien Grall wrote:
Hi Juergen,
On 27/05/2022 08:24, Juergen Gross wrote:
Add documentation for two new Xenstore wire commands SET_QUOTA and
GET_QUOTA used to set or query the global Xenstore quota or those of
a given domain.
Signed-off-by: Juergen Gross
---
Note that it mi
On 23.06.22 20:27, Julien Grall wrote:
Hi Juergen,
On 27/05/2022 08:24, Juergen Gross wrote:
Add documentation for two new Xenstore wire commands SET_FEATURE and
GET_FEATURE used to set or query the Xenstore features visible in the
ring page of a given domain.
Signed-off-by: Juergen Gross
---
flight 171330 linux-5.4 real [real]
http://logs.test-lab.xenproject.org/osstest/logs/171330/
Failures :-/ but no regressions.
Tests which did not succeed, but are not blocking:
test-armhf-armhf-xl-credit2 14 guest-start fail like 171224
test-amd64-amd64-xl-qemuu-win7-amd64 19
flight 171329 linux-linus real [real]
http://logs.test-lab.xenproject.org/osstest/logs/171329/
Regressions :-(
Tests which did not succeed and are blocking,
including tests which could not be run:
test-amd64-amd64-dom0pvh-xl-intel 8 xen-bootfail REGR. vs. 171277
test-amd64-amd64-xl
flight 171333 xen-unstable-smoke real [real]
http://logs.test-lab.xenproject.org/osstest/logs/171333/
Failures :-/ but no regressions.
Tests which did not succeed, but are not blocking:
test-amd64-amd64-libvirt 15 migrate-support-checkfail never pass
test-arm64-arm64-xl-xsm 1
On Thu, 23 Jun 2022, dmitry.semen...@gmail.com wrote:
> From: Dmytro Semenets
>
> When shutting down (or rebooting) the platform, Xen will call stop_cpu()
> on all the CPUs but one. The last CPU will then request the system to
> shutdown/restart.
>
> On platform using PSCI, stop_cpu() will call
Hi Bertrand,
On 24/05/2022 09:05, Bertrand Marquis wrote:
Signed-off-by: Julien Grall
I noticed that a few functions in Xen expect [start, end[. This is risky
as we may end up with end < start if the region is defined right at the
top of the address space.
I haven't yet tackle this issu
On Wed, 22 Jun 2022, Julien Grall wrote:
> On 15/06/2022 01:32, Stefano Stabellini wrote:
> > On Tue, 14 Jun 2022, Julien Grall wrote:
> > > From: Julien Grall
> > >
> > > Commit 5047cd1d5dea "xen/common: Use enhanced ASSERT_ALLOC_CONTEXT in
> > > xmalloc()" extended the checks in _xmalloc() to c
Hi Jan.
I know I will sound pedantic ;-) but an important fact about
the MISRA standards is that reading the headline alone is almost
never enough. In the specific of (advisory) Directive 4.6,
the Rationale says, among other things:
It might be desirable not to apply this guideline when
On Thu, 23 Jun 2022, Bertrand Marquis wrote:
> > On 22 Jun 2022, at 22:52, Stefano Stabellini wrote:
> >
> > On Wed, 22 Jun 2022, Bertrand Marquis wrote:
> >> Hi Stefano,
> >>
> >>> On 22 Jun 2022, at 01:00, Stefano Stabellini
> >>> wrote:
> >>>
> >>> On Fri, 10 Jun 2022, Bertrand Marquis wro
Hi Xenia,
On 22/06/2022 00:44, Stefano Stabellini wrote:
On Tue, 21 Jun 2022, Xenia Ragiadakou wrote:
Include vtimer.h so that the declarations of vtimer functions with external
linkage are visible before the function definitions.
Signed-off-by: Xenia Ragiadakou
Reviewed-by: Stefano Stabell
Hi Xenia,
On 22/06/2022 16:19, Bertrand Marquis wrote:
On 22 Jun 2022, at 16:15, Xenia Ragiadakou wrote:
Include header so that the declarations of the functions
machine_halt() and machine_restart(), which have external linkage, are visible
before the function definitions.
Signed-off-by: Xen
Hi,
On 22/06/2022 20:31, Stefano Stabellini wrote:
On Wed, 22 Jun 2022, Xenia Ragiadakou wrote:
The function __dt_n_size_cells() is referenced only in device_tree.c.
Change the linkage of the function from external to internal by adding
the storage-class specifier static to the function definit
Hi Juergen,
On 27/05/2022 08:24, Juergen Gross wrote:
Add documentation for an extension of the WATCH command used to limit
the scope of watched paths. Additionally it enables to receive more
information in the events related to special watches (@introduceDomain
or @releaseDomain).
Signed-off-b
flight 171327 xen-unstable real [real]
http://logs.test-lab.xenproject.org/osstest/logs/171327/
Failures :-/ but no regressions.
Tests which are failing intermittently (not blocking):
test-amd64-i386-libvirt 7 xen-installfail pass in 171319
test-amd64-amd64-libvirt-vhd 16
Hi Juergen,
On 27/05/2022 08:24, Juergen Gross wrote:
Add documentation for two new Xenstore wire commands SET_QUOTA and
GET_QUOTA used to set or query the global Xenstore quota or those of
a given domain.
Signed-off-by: Juergen Gross
---
Note that it might be a good idea to add support to the
Hi Juergen,
On 27/05/2022 08:24, Juergen Gross wrote:
Add documentation for two new Xenstore wire commands SET_FEATURE and
GET_FEATURE used to set or query the Xenstore features visible in the
ring page of a given domain.
Signed-off-by: Juergen Gross
---
Do we need support in the migration pro
On Thu, 23 Jun 2022, Jan Beulich wrote:
> On 23.06.2022 09:37, Roberto Bagnara wrote:
> > Rule 8.1 only applies to C90 code, as all the violating instances are
> > syntax errors in C99 and later versions of the language. So,
> > the following line does not contain a violation of Rule 8.1:
> >
> >
Hi Juergen,
On 27/05/2022 08:24, Juergen Gross wrote:
Instead of specifying the feature bits in xenstore-ring.txt as a mask
value use bit numbers. This will make the specification easier to read
when adding more features.
Signed-off-by: Juergen Gross
Reviewed-by: Julien Grall
Cheers,
--
J
Hi Oleksandr,
On 11/05/2022 19:47, Oleksandr Tyshchenko wrote:
From: Oleksandr Tyshchenko
Borrow the x86's check from p2m_remove_page() which was added
by the following commit: c65ea16dbcafbe4fe21693b18f8c2a3c5d14600e
"x86/p2m: don't assert that the passed in MFN matches for a remove"
and adju
Hi Oleksandr,
Sorry for the late reply.
On 11/05/2022 19:47, Oleksandr Tyshchenko wrote:
diff --git a/xen/arch/arm/include/asm/mm.h b/xen/arch/arm/include/asm/mm.h
+/*
+ * All accesses to the GFN portion of type_info field should always be
+ * protected by the P2M lock. In case when it is not f
flight 171328 xen-unstable-smoke real [real]
http://logs.test-lab.xenproject.org/osstest/logs/171328/
Failures :-/ but no regressions.
Tests which did not succeed, but are not blocking:
test-amd64-amd64-libvirt 15 migrate-support-checkfail never pass
test-arm64-arm64-xl-xsm 1
On 23.06.2022 17:30, Julien Grall wrote:
> On 23/06/2022 16:10, Rahul Singh wrote:
>>> On 22 Jun 2022, at 4:05 pm, Julien Grall wrote:
>>> On 22/06/2022 15:38, Rahul Singh wrote:
Guest can request the Xen to close the event channels. Ignore the
request from guest to close the static chan
On 23/06/2022 16:10, Rahul Singh wrote:
Hi Julien,
On 22 Jun 2022, at 4:05 pm, Julien Grall wrote:
Hi,
On 22/06/2022 15:38, Rahul Singh wrote:
Guest can request the Xen to close the event channels. Ignore the
request from guest to close the static channels as static event channels
shoul
Hi Julien,
> On 22 Jun 2022, at 4:05 pm, Julien Grall wrote:
>
> Hi,
>
> On 22/06/2022 15:38, Rahul Singh wrote:
>> Guest can request the Xen to close the event channels. Ignore the
>> request from guest to close the static channels as static event channels
>> should not be closed.
>
> Why do
On 23.06.2022 10:24, Roger Pau Monne wrote:
> Using cluster mode by default greatly limits the amount of vectors
> available, as then vector space is shared amongst all the CPUs in the
> logical cluster.
>
> This can lead to vector shortage issues on boxes with not a huge
> amount of CPUs but with
On 23.06.2022 10:24, Roger Pau Monne wrote:
> Allow selecting the default x2APIC destination mode from Kconfig.
> Note the default destination mode is still Logical (Cluster) mode.
>
> Signed-off-by: Roger Pau Monné
> ---
> xen/arch/x86/Kconfig | 29 +
> xen/
On 23/06/2022 13:05, xenia wrote:
Hi Ayan!
Hi Xenia,
On 6/23/22 13:02, Ayan Kumar Halder wrote:
(Resending mail, as the previous delivery failed)
On 21/06/2022 12:34, Ayan Kumar Halder wrote:
Hi,
On 19/06/2022 13:43, Xenia Ragiadakou wrote:
Direct mapping for dom0less VMs is disabled by
flight 171320 linux-5.4 real [real]
http://logs.test-lab.xenproject.org/osstest/logs/171320/
Regressions :-(
Tests which did not succeed and are blocking,
including tests which could not be run:
test-armhf-armhf-xl-credit1 14 guest-start fail REGR. vs. 171275
test-armhf-armhf-libv
On 23.06.2022 11:08, Roger Pau Monne wrote:
> Testing on a Kaby Lake box with 8 CPUs leads to the serial buffer
> being filled halfway during dom0 boot, and thus a non-trivial chunk of
> Linux boot messages are dropped.
>
> Increasing the buffer to 32K does fix the issue and Linux boot
> messages
On 23.06.2022 11:08, Roger Pau Monne wrote:
> --- a/xen/drivers/char/Kconfig
> +++ b/xen/drivers/char/Kconfig
> @@ -74,3 +74,13 @@ config HAS_EHCI
> help
> This selects the USB based EHCI debug port to be used as a UART. If
> you have an x86 based system with USB, say Y.
> +
>
On 23.06.2022 15:03, Julien Grall wrote:
>
>
> On 23/06/2022 13:59, Jan Beulich wrote:
>> On 23.06.2022 13:24, Julien Grall wrote:
>>> From: Julien Grall
>>>
>>> At the moment, corrupt() is neither checking for allocation failure
>>> nor freeing the allocated memory.
>>>
>>> Harden the code by p
Hello Anthony
[sorry for the possible format issues]
On Thu, Jun 23, 2022 at 1:24 PM Anthony PERARD
wrote:
> On Fri, Jun 17, 2022 at 07:14:31PM +0300, Oleksandr Tyshchenko wrote:
> > From: Oleksandr Tyshchenko
> >
> > This patch adds basic support for configuring and assisting virtio-mmio
> >
On 18.05.22 11:45, Juergen Gross wrote:
On 04.05.22 09:53, Juergen Gross wrote:
On 19.04.22 10:01, Juergen Gross wrote:
On 24.03.22 15:01, Juergen Gross wrote:
In order to avoid indirect function calls on the hypercall path as
much as possible this series is removing the hypercall function tab
On 23/06/2022 13:59, Jan Beulich wrote:
On 23.06.2022 13:24, Julien Grall wrote:
From: Julien Grall
At the moment, corrupt() is neither checking for allocation failure
nor freeing the allocated memory.
Harden the code by printing ENOMEM if the allocation failed and
free 'str' after the las
On 09.05.22 15:51, Oleksandr Tyshchenko wrote:
From: Oleksandr Tyshchenko
With Xen PV Display driver in use the "expected" VM_DONTEXPAND flag
is not set (neither explicitly nor implicitly), so the driver hits
the code path in drm_gem_mmap_obj() which triggers the WARNING.
Signed-off-by: Oleksa
On 17.06.22 12:30, Julien Grall wrote:
From: Julien Grall
1 << 31 is undefined. So switch to 1U << 31.
Fixes: 5ead97c84fa7 ("xen: Core Xen implementation")
Signed-off-by: Julien Grall
Pushed to xen/tip.git for-linus-5.19a
Juergen
OpenPGP_0xB0DE9DD628BF132F.asc
Description: OpenPGP publi
On 01.06.22 21:53, Jason Andryuk wrote:
When a VBD is not fully created and then closed, the kernel can have a
NULL pointer dereference:
The reproducer is trivial:
[user@dom0 ~]$ sudo xl block-attach work backend=sys-usb vdev=xvdi
target=/dev/sdz
[user@dom0 ~]$ xl block-list work
Vdev BE han
On 23.06.2022 13:24, Julien Grall wrote:
> From: Julien Grall
>
> At the moment, corrupt() is neither checking for allocation failure
> nor freeing the allocated memory.
>
> Harden the code by printing ENOMEM if the allocation failed and
> free 'str' after the last use.
>
> This is not consider
flight 171322 linux-linus real [real]
http://logs.test-lab.xenproject.org/osstest/logs/171322/
Regressions :-(
Tests which did not succeed and are blocking,
including tests which could not be run:
test-amd64-coresched-amd64-xl broken
test-amd64-coresched-amd64-xl 5 host-ins
On 22.06.22 04:27, Demi Marie Obenour wrote:
unmap_grant_pages() currently waits for the pages to no longer be used.
In https://github.com/QubesOS/qubes-issues/issues/7481, this lead to a
deadlock against i915: i915 was waiting for gntdev's MMU notifier to
finish, while gntdev was waiting for i91
On 10.06.2022 07:53, Wei Chen wrote:
> --- a/xen/arch/arm/Makefile
> +++ b/xen/arch/arm/Makefile
> @@ -1,6 +1,5 @@
> obj-$(CONFIG_ARM_32) += arm32/
> obj-$(CONFIG_ARM_64) += arm64/
> -obj-$(CONFIG_ARM_64) += efi/
> obj-$(CONFIG_ACPI) += acpi/
> obj-$(CONFIG_HAS_PCI) += pci/
> ifneq ($(CONFIG_N
Hello Viresh
[sorry for the possible format issues]
On Thu, Jun 23, 2022 at 8:48 AM Viresh Kumar
wrote:
> On 22-06-22, 18:05, Oleksandr Tyshchenko wrote:
> > Even leaving
> > aside the fact that restricted virtio memory access in the guest means
> that
> > not all of guest memory can be accesse
Hi Andrew,
On 23/06/2022 13:41, Andrew Cooper wrote:
On 23/06/2022 12:28, Juergen Gross wrote:
On 23.06.22 13:24, Julien Grall wrote:
From: Julien Grall
At the moment, corrupt() is neither checking for allocation failure
nor freeing the allocated memory.
Harden the code by printing ENOMEM i
On 23/06/2022 12:28, Juergen Gross wrote:
> On 23.06.22 13:24, Julien Grall wrote:
>> From: Julien Grall
>>
>> At the moment, corrupt() is neither checking for allocation failure
>> nor freeing the allocated memory.
>>
>> Harden the code by printing ENOMEM if the allocation failed and
>> free 'str
> On 23 Jun 2022, at 13:00, Jan Beulich wrote:
>
> On 09.12.2021 12:25, Jan Beulich wrote:
>> I did notice this anomaly in the context of IOMMU side work.
>>
>> 1: shadow: slightly consolidate sh_unshadow_for_p2m_change()
>> 2: P2M: allow 2M superpage use for shadowed guests
>
> This has been
On 6/23/22 13:01, Ayan Kumar Halder wrote:
(Resending mail, as the previous delivery failed)
On 21/06/2022 11:57, Ayan Kumar Halder wrote:
1. In README.md, please mention in 'DOM0_KERNEL' description that it
is an optional parameter. When user does not provide this, it
generates a dom0less con
Hi Ayan!
On 6/23/22 13:02, Ayan Kumar Halder wrote:
(Resending mail, as the previous delivery failed)
On 21/06/2022 12:34, Ayan Kumar Halder wrote:
Hi,
On 19/06/2022 13:43, Xenia Ragiadakou wrote:
Direct mapping for dom0less VMs is disabled by default in XEN and
can be
enabled through the '
On 13.09.2021 08:26, Jan Beulich wrote:
> asm() constraints need to fit both the intended insn(s) which the
> respective operands are going to be used with as well as the actual kind
> of value specified. "m" (alone) together with a constant, however, leads
> to gcc saying
>
> error: memory input
On 09.12.2021 12:25, Jan Beulich wrote:
> I did notice this anomaly in the context of IOMMU side work.
>
> 1: shadow: slightly consolidate sh_unshadow_for_p2m_change()
> 2: P2M: allow 2M superpage use for shadowed guests
This has been pending for over half a year. Anyone?
Thanks, Jan
By using | instead of || or (in the negated form) && chances increase
for the compiler to recognize that both predicates can actually be
folded into an expression requiring just a single branch (via OR-ing
together the respective P2M_*_TYPES constants).
Signed-off-by: Jan Beulich
---
RFC: The 3-w
Grant P2M entries, which are covered by p2m_is_any_ram(), wouldn't pass
the get_page() unless the grant was a local one. These need to take the
same path as foreign entries. Just the assertion there is not valid for
local grants, and hence it triggering needs to be avoided.
Signed-off-by: Jan Beul
While the first change is a bug fix (for, admittedly, a case which
apparently hasn't occurred in practice, or else we would have had
bug reports), it already puts in place an instance of what the 2nd
patch is proposing for perhaps wider use.
1: make p2m_get_page_from_gfn() handle grant and shared
On 14.04.2022 14:59, Jan Beulich wrote:
> On 14.04.2022 14:49, Andrew Cooper wrote:
>> On 12/04/2022 11:28, Jan Beulich wrote:
>>> Future gas versions will generate minimalistic Dwarf debug info for
>>> items annotated as functions and having their sizes specified [1].
>>> "Borrow" Arm's END() and
Hi Juergen,
On 23/06/2022 12:28, Juergen Gross wrote:
On 23.06.22 13:24, Julien Grall wrote:
From: Julien Grall
At the moment, corrupt() is neither checking for allocation failure
nor freeing the allocated memory.
Harden the code by printing ENOMEM if the allocation failed and
free 'str' aft
On 23.06.22 13:24, Julien Grall wrote:
From: Julien Grall
At the moment, corrupt() is neither checking for allocation failure
nor freeing the allocated memory.
Harden the code by printing ENOMEM if the allocation failed and
free 'str' after the last use.
This is not considered to be a securit
From: Julien Grall
At the moment, corrupt() is neither checking for allocation failure
nor freeing the allocated memory.
Harden the code by printing ENOMEM if the allocation failed and
free 'str' after the last use.
This is not considered to be a security issue because corrupt() should
only be
flight 171325 xen-unstable-smoke real [real]
http://logs.test-lab.xenproject.org/osstest/logs/171325/
Failures :-/ but no regressions.
Tests which did not succeed, but are not blocking:
test-amd64-amd64-libvirt 15 migrate-support-checkfail never pass
test-arm64-arm64-xl-xsm 1
flight 171319 xen-unstable real [real]
flight 171326 xen-unstable real-retest [real]
http://logs.test-lab.xenproject.org/osstest/logs/171319/
http://logs.test-lab.xenproject.org/osstest/logs/171326/
Failures :-/ but no regressions.
Tests which are failing intermittently (not blocking):
test-amd6
On Fri, Jun 17, 2022 at 07:14:31PM +0300, Oleksandr Tyshchenko wrote:
> From: Oleksandr Tyshchenko
>
> This patch adds basic support for configuring and assisting virtio-mmio
> based virtio-disk backend (emulator) which is intended to run out of
> Qemu and could be run in any domain.
> Although t
(Resending mail, as the previous delivery failed)
On 21/06/2022 11:57, Ayan Kumar Halder wrote:
Hi Xenia,
Thanks for the change. It looks good.
Some points :-
1. In README.md, please mention in 'DOM0_KERNEL' description that it
is an optional parameter. When user does not provide this, it
g
(Resending mail, as the previous delivery failed)
On 21/06/2022 12:34, Ayan Kumar Halder wrote:
Hi,
On 19/06/2022 13:43, Xenia Ragiadakou wrote:
Direct mapping for dom0less VMs is disabled by default in XEN and can be
enabled through the 'direct-map' property.
Add a new config parameter DOMU_D
On Thu, Jun 23, 2022 at 09:56:34AM +, George Dunlap wrote:
>
>
> > On 23 Jun 2022, at 08:43, Jan Beulich wrote:
> >
> > On 09.06.2022 17:33, Jan Beulich wrote:
> >> While mail hasn't been bouncing, Daniel has not been responding to patch
> >> submissions or otherwise interacting with the co
> On 23 Jun 2022, at 08:43, Jan Beulich wrote:
>
> On 09.06.2022 17:33, Jan Beulich wrote:
>> While mail hasn't been bouncing, Daniel has not been responding to patch
>> submissions or otherwise interacting with the community for several
>> years. Move maintainership to THE REST in kind of an u
On 23.06.2022 11:46, Juergen Gross wrote:
> --- a/arch/x86/xen/enlighten_pv.c
> +++ b/arch/x86/xen/enlighten_pv.c
> @@ -1183,15 +1183,19 @@ static void __init xen_domu_set_legacy_features(void)
> extern void early_xen_iret_patch(void);
>
> /* First C function to be called on Xen boot */
> -asml
> On 22 Jun 2022, at 17:28, Andrew Cooper wrote:
>
> On 22/06/2022 13:32, Bertrand Marquis wrote:
>> Hi Andrew and Christopher,
>>
>> I will not dig into the details of the issues you currently have
>> but it seems you are trying to re-do the work we already did
>> and have been using for quit
On 10.06.2022 10:32, Roger Pau Monne wrote:
> The loop in iommu_{un,}map() can be arbitrary large, and as such it
> needs to handle preemption. Introduce a new parameter that allow
> returning the number of pages that have been processed, and which
> presence also signals whether the function shou
The brk area needs to be zeroed initially, like the .bss section.
At the same time its memory should be covered by the ELF program
headers.
Juergen Gross (3):
x86/xen: use clear_bss() for Xen PV guests
x86: fix setup of brk area
x86: fix .brk attribute in linker script
arch/x86/include/asm
Commit e32683c6f7d2 ("x86/mm: Fix RESERVE_BRK() for older binutils")
put the brk area into the .bss..brk section (placed directly behind
.bss), causing it not to be cleared initially. As the brk area is used
to allocate early page tables, these might contain garbage in not
explicitly written entrie
Instead of clearing the bss area in assembly code, use the clear_bss()
function.
This requires to pass the start_info address as parameter to
xen_start_kernel() in order to avoid the xen_start_info being zeroed
again.
Signed-off-by: Juergen Gross
Reviewed-by: Jan Beulich
---
arch/x86/include/a
Commit e32683c6f7d2 ("x86/mm: Fix RESERVE_BRK() for older binutils")
added the "NOLOAD" attribute to the .brk section as a "failsafe"
measure.
Unfortunately this leads to the linker no longer covering the .brk
section in a program header, resulting in the kernel loader not knowing
that the memory
flight 171324 ovmf real [real]
http://logs.test-lab.xenproject.org/osstest/logs/171324/
Perfect :-)
All tests in this flight passed as required
version targeted for testing:
ovmf 4bfd668e5edb59092a8e16414b3f6632efdac4f2
baseline version:
ovmf f304308e1cb21846a79fc
On 22.06.2022 17:47, Marek Marczykowski-Górecki wrote:
> On Wed, Jun 15, 2022 at 04:25:54PM +0200, Jan Beulich wrote:
>> On 07.06.2022 16:30, Marek Marczykowski-Górecki wrote:
>>> +/* ...we found it, so parse the BAR and map the registers */
>>> +bar0 = pci_conf_read32(xue->sbdf, PCI_BASE_A
On 23.06.22 11:08, Jan Beulich wrote:
On 23.06.2022 11:01, Juergen Gross wrote:
On 23.06.22 10:47, Jan Beulich wrote:
On 23.06.2022 10:06, Juergen Gross wrote:
On 23.06.22 09:55, Jan Beulich wrote:
On 22.06.2022 18:06, Juergen Gross wrote:
A Linux kernel 5.19 can only be loaded as dom0, if i
On 23.06.2022 11:08, Juergen Gross wrote:
> On 23.06.22 11:04, Jan Beulich wrote:
>> On 23.06.2022 10:02, Juergen Gross wrote:
>>> When loading the dom0 kernel for PV mode, the first free usable memory
>>> location after the kernel needs to take segments into account, which
>>> have only the ALLOC
Take the opportunity to convert the variable to read-only after init.
No functional change intended.
Signed-off-by: Roger Pau Monné
---
xen/drivers/char/Kconfig | 10 ++
xen/drivers/char/serial.c | 3 ++-
2 files changed, 12 insertions(+), 1 deletion(-)
diff --git a/xen/drivers/char/
Testing on a Kaby Lake box with 8 CPUs leads to the serial buffer
being filled halfway during dom0 boot, and thus a non-trivial chunk of
Linux boot messages are dropped.
Increasing the buffer to 32K does fix the issue and Linux boot
messages are no longer dropped. There's no justification either
Hello,
First patch moves the setting of the default TX buffer size to Kconfig,
and shouldn't be controversial, second patch increases the buffer to
32K.
Thanks, Roger.
Roger Pau Monne (2):
console/serial: set the default transmit buffer size in Kconfig
console/serial: bump buffer from 16K to
On 23.06.2022 11:01, Juergen Gross wrote:
> On 23.06.22 10:47, Jan Beulich wrote:
>> On 23.06.2022 10:06, Juergen Gross wrote:
>>> On 23.06.22 09:55, Jan Beulich wrote:
On 22.06.2022 18:06, Juergen Gross wrote:
> A Linux kernel 5.19 can only be loaded as dom0, if it has been
> built wi
On 23.06.22 11:04, Jan Beulich wrote:
On 23.06.2022 10:02, Juergen Gross wrote:
When loading the dom0 kernel for PV mode, the first free usable memory
location after the kernel needs to take segments into account, which
have only the ALLOC flag set, but are not specified to be loaded in
the prog
On 23.06.2022 10:02, Juergen Gross wrote:
> When loading the dom0 kernel for PV mode, the first free usable memory
> location after the kernel needs to take segments into account, which
> have only the ALLOC flag set, but are not specified to be loaded in
> the program headers of the ELF file.
>
>
On 23.06.22 10:50, Jan Beulich wrote:
On 23.06.2022 10:14, Juergen Gross wrote:
On 23.06.22 10:09, Jan Beulich wrote:
On 22.06.2022 18:10, Juergen Gross wrote:
Commit e32683c6f7d2 ("x86/mm: Fix RESERVE_BRK() for older binutils")
put the brk area into the .bss segment, causing it not to be clea
On 23.06.22 10:47, Jan Beulich wrote:
On 23.06.2022 10:06, Juergen Gross wrote:
On 23.06.22 09:55, Jan Beulich wrote:
On 22.06.2022 18:06, Juergen Gross wrote:
A Linux kernel 5.19 can only be loaded as dom0, if it has been
built with CONFIG_AMD_MEM_ENCRYPT enabled. This is due to the
fact that
On 23.06.2022 10:14, Juergen Gross wrote:
> On 23.06.22 10:09, Jan Beulich wrote:
>> On 22.06.2022 18:10, Juergen Gross wrote:
>>> Commit e32683c6f7d2 ("x86/mm: Fix RESERVE_BRK() for older binutils")
>>> put the brk area into the .bss segment, causing it not to be cleared
>>> initially.
>>
>> This
On 23.06.2022 10:06, Juergen Gross wrote:
> On 23.06.22 09:55, Jan Beulich wrote:
>> On 22.06.2022 18:06, Juergen Gross wrote:
>>> A Linux kernel 5.19 can only be loaded as dom0, if it has been
>>> built with CONFIG_AMD_MEM_ENCRYPT enabled. This is due to the
>>> fact that otherwise the (relevant)
When using an APIC do not set nr_irqs based on a factor of nr_irqs_gsi
(currently x8), and instead do so exclusively based on the amount of
available vectors on the system.
There's no point in setting nr_irqs to a value higher than the
available set of vectors, as vector allocation will fail anywa
Using nr_irqs minus nr_irqs_gsi is misleading, as GSI interrupts are
not allocated unless requested by the hardware domain, so a hardware
domain could not use any GSI (or just one for the ACPI SCI), and hence
(almost) all nr_irqs will be available for MSI(-X) usage.
No functional difference, just
Current code to calculate nr_irqs assumes the APIC destination mode to
be physical, so all vectors on each possible CPU is available for use
by a different interrupt source. This is not true when using Logical
(Cluster) destination mode, where CPUs in the same cluster share the
vector space.
Fix b
Logic in ioapic_init() that sets the number of available vectors for
external interrupts requires knowing the x2APIC Destination Mode. As
such move the call after x2APIC BSP setup.
Do it as part of init_irq_data(), which is called just after x2APIC
BSP init and also makes use of nr_irqs itself.
Using cluster mode by default greatly limits the amount of vectors
available, as then vector space is shared amongst all the CPUs in the
logical cluster.
This can lead to vector shortage issues on boxes with not a huge
amount of CPUs but with a non-trivial amount of devices, there are
reports of b
Allow selecting the default x2APIC destination mode from Kconfig.
Note the default destination mode is still Logical (Cluster) mode.
Signed-off-by: Roger Pau Monné
---
xen/arch/x86/Kconfig | 29 +
xen/arch/x86/genapic/x2apic.c | 6 --
2 files changed, 33
Hello,
The following series aims to change the default x2APIC Destination mode
from Logical to Physical. This is done in order to cope with boxes that
don't have a huge amount of CPUs, but do have a non trivial amount of
PCI devices using MSI(-X).
The default x2APIC destination mode can now be s
On 23.06.22 10:09, Jan Beulich wrote:
On 22.06.2022 18:10, Juergen Gross wrote:
Commit e32683c6f7d2 ("x86/mm: Fix RESERVE_BRK() for older binutils")
put the brk area into the .bss segment, causing it not to be cleared
initially.
This reads contradictively: If the area was put in .bss, it would
On 22.06.2022 18:10, Juergen Gross wrote:
> Commit e32683c6f7d2 ("x86/mm: Fix RESERVE_BRK() for older binutils")
> put the brk area into the .bss segment, causing it not to be cleared
> initially.
This reads contradictively: If the area was put in .bss, it would be
cleared. Thing is it is put in .
On 23.06.22 09:55, Jan Beulich wrote:
On 22.06.2022 18:06, Juergen Gross wrote:
A Linux kernel 5.19 can only be loaded as dom0, if it has been
built with CONFIG_AMD_MEM_ENCRYPT enabled. This is due to the
fact that otherwise the (relevant) last section of the built
kernel has the NOLOAD flag set
On Thu, Jun 23, 2022 at 09:43:30AM +0200, Jan Beulich wrote:
> On 09.06.2022 17:33, Jan Beulich wrote:
> > While mail hasn't been bouncing, Daniel has not been responding to patch
> > submissions or otherwise interacting with the community for several
> > years. Move maintainership to THE REST in k
When loading the dom0 kernel for PV mode, the first free usable memory
location after the kernel needs to take segments into account, which
have only the ALLOC flag set, but are not specified to be loaded in
the program headers of the ELF file.
This is e.g. a problem for Linux kernels from 5.19 on
1 - 100 of 113 matches
Mail list logo
