[Xen-devel] [PATCH v3 10/17] livepatch: Declare live patching as a supported feature

2017-09-11 Thread Konrad Rzeszutek Wilk
From: Ross Lagerwall See docs/features/livepatch.pandoc for the details. Reviewed-by: Jan Beulich Signed-off-by: Ross Lagerwall Signed-off-by: Konrad Rzeszutek Wilk -- v2: - Moved it into a feature document. - Clarified a few bits and pieces based on feedback. v3: - default X86 - added

[Xen-devel] [PATCH v3 17/17] livepatch: Add xen_local_symbols test-case

2017-09-11 Thread Konrad Rzeszutek Wilk
.. Signed-off-by: Konrad Rzeszutek Wilk --- v1: First edition v2: Build with mkhex version of build-id from hypervisor. --- xen/test/livepatch/Makefile| 12 +++- xen/test/livepatch/xen_local_symbols.c | 53 ++ 2 files changed, 64 insertions(+),

[Xen-devel] [PATCH v3 02/17] livepatch: Tighten alignment checks.

2017-09-11 Thread Konrad Rzeszutek Wilk
-by: Konrad Rzeszutek Wilk --- v1: Initial patch v2: Drop the check when loading it in memory Add check for alignment being anything but power of two (ignoring 0, and 1) Change dprintk to include hex values and print addr not size. v3: Change the two checks to be per Jan's recommenda

Re: [Xen-devel] [PATCH v3 07/17] livepatch/arm/x86: Strip note_depends symbol from test-cases.

2017-09-12 Thread Konrad Rzeszutek Wilk
On Tue, Sep 12, 2017 at 08:48:33AM -0600, Jan Beulich wrote: > >>> On 12.09.17 at 02:37, wrote: > > This surfaced due to "xen/livepatch/x86/arm32: Force > > .livepatch.depends section to be uint32_t aligned." which switched > > to a different way of including the build-id. > > > > Each livepatch

Re: [Xen-devel] CONFIG_SCRUB_DEBUG=y + arm64 + livepatch = Xen BUG at page_alloc.c:738

2017-09-12 Thread Konrad Rzeszutek Wilk
On Mon, Sep 11, 2017 at 08:45:02PM -0400, Boris Ostrovsky wrote: > > > On 09/11/2017 07:55 PM, Konrad Rzeszutek Wilk wrote: > > Hey, > > > > I've only been able to reproduce this on ARM64 (trying right now ARM32 > > as well), and not on x86. > > >

Re: [Xen-devel] [PATCH v3 14/17] livepatch/x86/arm: arch/x86/mm: generalize do_page_walk() and implement arch_livepatch_lookup_mfn

2017-09-12 Thread Konrad Rzeszutek Wilk
On Tue, Sep 12, 2017 at 08:54:34AM -0600, Jan Beulich wrote: > >>> On 12.09.17 at 02:37, wrote: > > With this change we can use _do_page_walk() to implement > > arch_livepatch_lookup_mfn() which can be used to find out > > vmap virtual addresses (as under x86 virt_to_mfn won't work > > for vmap, b

Re: [Xen-devel] CONFIG_SCRUB_DEBUG=y + arm64 + livepatch = Xen BUG at page_alloc.c:738

2017-09-13 Thread Konrad Rzeszutek Wilk
On Tue, Sep 12, 2017 at 09:19:23PM -0400, Boris Ostrovsky wrote: > > > On 09/12/2017 08:01 PM, Konrad Rzeszutek Wilk wrote: > > On Mon, Sep 11, 2017 at 08:45:02PM -0400, Boris Ostrovsky wrote: > > > > > > > > > On 09/11/2017 07:55 PM

Re: [Xen-devel] [RFC XEN PATCH v3 12/39] tools/xen-ndctl: add NVDIMM management util 'xen-ndctl'

2017-09-13 Thread Konrad Rzeszutek Wilk
On Mon, Sep 11, 2017 at 09:35:08AM -0700, Dan Williams wrote: > On Sun, Sep 10, 2017 at 10:39 PM, Haozhong Zhang > wrote: > > On 09/10/17 22:10 -0700, Dan Williams wrote: > >> On Sun, Sep 10, 2017 at 9:37 PM, Haozhong Zhang > >> wrote: > >> > The kernel NVDIMM driver and the traditional NVDIMM ma

Re: [Xen-devel] [PATCH v3 07/17] livepatch/arm/x86: Strip note_depends symbol from test-cases.

2017-09-13 Thread Konrad Rzeszutek Wilk
On Wed, Sep 13, 2017 at 02:51:41AM -0600, Jan Beulich wrote: > >>> On 13.09.17 at 01:46, wrote: > > On Tue, Sep 12, 2017 at 08:48:33AM -0600, Jan Beulich wrote: > >> >>> On 12.09.17 at 02:37, wrote: > >> > This surfaced due to "xen/livepatch/x86/arm32: Force > >> > .livepatch.depends section to b

Re: [Xen-devel] [PATCH RFC] Add SUPPORT.md

2017-09-14 Thread Konrad Rzeszutek Wilk
On Thu, Sep 07, 2017 at 05:50:13AM -0600, Jan Beulich wrote: > >>> On 07.09.17 at 13:31, wrote: > > On 08/31/2017 01:46 PM, Jan Beulich wrote: > > On 31.08.17 at 12:27, wrote: > >>> +### Live Patching > >>> + > >>> +Status: Supported, x86 only > >>> + > >>> +Compile time disabled > >> >

Re: [Xen-devel] GRUB documentation updated

2017-09-14 Thread Konrad Rzeszutek Wilk
On Thu, Sep 07, 2017 at 09:12:33PM +0200, Daniel Kiper wrote: > Hey, > > Some people asked me about Multiboot2 Specification and other GRUB doc stuff. > So, I have put latest things at > https://www.gnu.org/software/grub/grub-documentation.html Yeey! Thank you for doing that! > > I hope that h

Re: [Xen-devel] CONFIG_SCRUB_DEBUG=y + arm64 + livepatch = Xen BUG at page_alloc.c:738

2017-09-14 Thread Konrad Rzeszutek Wilk
On Wed, Sep 13, 2017 at 02:49:41PM -0400, Boris Ostrovsky wrote: > On 09/13/2017 02:25 PM, Julien Grall wrote: > > Hi, > > > > On 09/13/2017 07:05 PM, Boris Ostrovsky wrote: > >> On 09/13/2017 11:32 AM, Konrad Rzeszutek Wilk wrote: > >> Well, that

Re: [Xen-devel] [PATCH RFC v2] Add SUPPORT.md

2017-09-15 Thread Konrad Rzeszutek Wilk
> +### Soft-reset for PV guests s/PV/HVM/ > + > +Status: Supported > + > +Soft-reset allows a new kernel to start 'from scratch' with a fresh VM > state, > +but with all the memory from the previous state of the VM intact. > +This is primarily designed to allow "crash kernels", > +wh

[Xen-devel] [PATCH v3] Left over patches from long-time ago.

2017-09-15 Thread Konrad Rzeszutek Wilk
oader/pci.c| 8 tools/firmware/hvmloader/smbios.c | 5 ++--- tools/firmware/hvmloader/util.h | 3 +++ 4 files changed, 12 insertions(+), 10 deletions(-) Konrad Rzeszutek Wilk (1): hvmloader: Use MB(x) and GB(x) macros ___ Xen-

[Xen-devel] [PATCH v3] hvmloader: Use MB(x) and GB(x) macros

2017-09-15 Thread Konrad Rzeszutek Wilk
instead of hardcoding values. We also drop the uint64_t cast as the macro uses ULL to produce the proper width types. Acked-by: Jan Beulich Signed-off-by: Konrad Rzeszutek Wilk --- Cc: Jan Beulich Cc: Andrew Cooper Cc: Ian Jackson Cc: Wei Liu v2: New submission v3: Don't use le

Re: [Xen-devel] [PATCH 1/3] python: Add binding for xs_fileno()

2017-09-15 Thread Konrad Rzeszutek Wilk
rris Reviewed-by: Konrad Rzeszutek Wilk > --- > tools/python/xen/lowlevel/xs/xs.c | 20 > 1 file changed, 20 insertions(+) > > diff --git a/tools/python/xen/lowlevel/xs/xs.c > b/tools/python/xen/lowlevel/xs/xs.c > index aba5a20..9f1b916 100644 >

Re: [Xen-devel] [PATCH] mm: Scrub pages returned back to heap if MEMF_no_scrub is set

2017-09-15 Thread Konrad Rzeszutek Wilk
On Fri, Sep 15, 2017 at 12:47:29PM -0400, Boris Ostrovsky wrote: > On 09/15/2017 12:05 PM, Jan Beulich wrote: > On 15.09.17 at 16:04, wrote: > >> Set free_heap_pages()'s need_scrub to true if alloc_domheap_pages() > >> returns pages back to heap as result of assign_pages() error when those >

Re: [Xen-devel] [PATCH v3] hvmloader: Use MB(x) and GB(x) macros

2017-09-15 Thread Konrad Rzeszutek Wilk
On Fri, Sep 15, 2017 at 05:57:50PM +0100, Wei Liu wrote: > On Fri, Sep 15, 2017 at 12:55:51PM -0400, Konrad Rzeszutek Wilk wrote: > > instead of hardcoding values. We also drop the uint64_t > > cast as the macro uses ULL to produce the proper width > > types. > >

Re: [Xen-devel] CONFIG_SCRUB_DEBUG=y + arm64 + livepatch = Xen BUG at page_alloc.c:738

2017-09-15 Thread Konrad Rzeszutek Wilk
On Thu, Sep 14, 2017 at 05:39:23PM -0400, Boris Ostrovsky wrote: > On 09/14/2017 05:26 PM, Konrad Rzeszutek Wilk wrote: > > On Wed, Sep 13, 2017 at 02:49:41PM -0400, Boris Ostrovsky wrote: > >> On 09/13/2017 02:25 PM, Julien Grall wrote: > >>> Hi, > >&g

Re: [Xen-devel] CONFIG_SCRUB_DEBUG=y + arm64 + livepatch = Xen BUG at page_alloc.c:738

2017-09-15 Thread Konrad Rzeszutek Wilk
.snip.. > (XEN) Loading dom0 DTB to 0x17e0-0x17e08265 > (XEN) init_domheap_pages: 0xb87b1->0xb87bc > (XEN) init_heap_pages: 0xb87b1 -> 0xb87bc > (XEN) init_domheap_pages: 0xb88f1->0xb98ae > (XEN) init_heap_pages: 0xb88f1 -> 0xb98ae <- so the memory is from here > > (XEN) Sc

Re: [Xen-devel] ARM64:Porting xen to new hardware

2017-09-18 Thread Konrad Rzeszutek Wilk
On Fri, Sep 08, 2017 at 10:19:55PM +0300, Oleksandr Tyshchenko wrote: > Hi Bharat > > On Thu, Sep 7, 2017 at 4:30 PM, bharat gohil wrote: > > Hello Olensandr, > > > > I able to boot xen and trying to boot dom0 but there are no console log for > > dom0. > > > > following log for xen and it stuck b

Re: [Xen-devel] [PATCH v3 06/17] xen/livepatch/x86/arm32: Force .livepatch.depends section to be uint32_t aligned.

2017-09-18 Thread Konrad Rzeszutek Wilk
> > +.PHONY: livepatch_depends.h > > +livepatch_depends.h: note.bin > > + $(shell (../../../tools/firmware/hvmloader/mkhex $(NOTE_DEPENDS) $^ > > > $@)) > > It looks quite odd to use a file in firmware/hvmloader for livepatch. Would > it be possible to move mkhex to a generic place? Like so?

Re: [Xen-devel] [PATCH v3 08/17] livepatch/tests: Make sure all .livepatch.funcs sections are read-only

2017-09-18 Thread Konrad Rzeszutek Wilk
On Tue, Sep 12, 2017 at 08:49:55AM -0600, Jan Beulich wrote: > >>> On 12.09.17 at 02:37, wrote: > > --- a/xen/test/livepatch/Makefile > > +++ b/xen/test/livepatch/Makefile > > @@ -54,6 +54,7 @@ xen_hello_world.o: config.h livepatch_depends.h > > $(LIVEPATCH): xen_hello_world_func.o xen_hello_worl

Re: [Xen-devel] [PATCH v3 09/17] livepatch/arm[32, 64]: Modify livepatch_funcs

2017-09-18 Thread Konrad Rzeszutek Wilk
On Thu, Sep 14, 2017 at 02:20:42PM +0100, Julien Grall wrote: > Hi Konrad, > > On 12/09/17 01:37, Konrad Rzeszutek Wilk wrote: > > This was found when porting livepatch-build-tools to ARM64/32. > > > > When livepatch-build-tools are built (and test-case thanks to: >

Re: [Xen-devel] [PATCH 01/27 v9] xen/arm: vpl011: Define common ring buffer helper functions in console.h

2017-09-19 Thread Konrad Rzeszutek Wilk
> Reviewed-by: Stefano Stabellini > Acked-by: Wei Liu Acked-by: Konrad Rzeszutek Wilk > --- > CC: Ian Jackson > CC: Wei Liu > CC: Konrad Rzeszutek Wilk > CC: Stefano Stabellini > CC: Julien Grall > > Changes since v4: > - Split this change in a separate patch.

Re: [Xen-devel] Feature control on PV devices

2017-09-19 Thread Konrad Rzeszutek Wilk
On Thu, Sep 14, 2017 at 05:08:18PM +0100, Joao Martins wrote: > [ Realized that I didn't CC the maintainers, > so doing that now, +Linux folks +PV interfaces czar > Sorry for the noise! ] > > On 09/08/2017 09:49 AM, Joao Martins wrote: > > [Forgot two important details regarding Xenbus states]

Re: [Xen-devel] [PATCH v2 3/5] xen/livepatch/ARM32: Don't load and crash on livepatches loaded with wrong alignment.

2017-09-19 Thread Konrad Rzeszutek Wilk
On Tue, Sep 12, 2017 at 02:57:04AM -0600, Jan Beulich wrote: > >>> On 12.09.17 at 02:22, wrote: > > On Mon, Sep 11, 2017 at 03:01:15AM -0600, Jan Beulich wrote: > >> Hmm, as long as the relocation isn't required to be against aligned > >> fields only (mandated by the processor ABI) I think the cod

Re: [Xen-devel] Ping: [PATCH 2/2] public/sysctl: drop unnecessary typedefs and handles

2017-09-19 Thread Konrad Rzeszutek Wilk
ction) >> +static int livepatch_action(struct xen_sysctl_livepatch_action >*action) >> { >> struct payload *data; >> char n[XEN_LIVEPATCH_NAME_SIZE]; >> @@ -1560,7 +1560,7 @@ static int livepatch_action(xen_sysctl_l >> return rc; >> }

Re: [Xen-devel] [PATCH v2 3/5] xen/livepatch/ARM32: Don't load and crash on livepatches loaded with wrong alignment.

2017-09-20 Thread Konrad Rzeszutek Wilk
On Tue, Sep 19, 2017 at 09:04:44AM -0600, Jan Beulich wrote: > >>> On 18.09.17 at 21:37, wrote: > > On Tue, Sep 12, 2017 at 02:57:04AM -0600, Jan Beulich wrote: > >> >>> On 12.09.17 at 02:22, wrote: > >> > If I compile the test-case under ARM32 it works OK (as the > >> > .livepatch.depends ends u

Re: [Xen-devel] [PATCH v3 06/17] xen/livepatch/x86/arm32: Force .livepatch.depends section to be uint32_t aligned.

2017-09-20 Thread Konrad Rzeszutek Wilk
On Wed, Sep 20, 2017 at 03:01:57PM +0100, Wei Liu wrote: > On Tue, Sep 19, 2017 at 12:05:16PM +0100, Julien Grall wrote: > > Hi Konrad, > > > > On 19/09/17 01:32, Konrad Rzeszutek Wilk wrote: > > > > > +.PHONY: livepatch_depends.h > > > > > +li

[Xen-devel] [PATCH v4 08/11] livepatch/arm/x86: Rename note_depends symbol from test-cases.

2017-09-20 Thread Konrad Rzeszutek Wilk
00 R xen_nop.livepatch_note_depends R xen_replace_world.livepatch_note_depends Signed-off-by: Konrad Rzeszutek Wilk --- Cc: Ross Lagerwall v3: First posting. v4: Instead of stripping the symbol (and also using --strip-debug), just rename the symbol. --- xen/test/livepatch/Makefile | 4

[Xen-devel] [PATCH v4 03/11] livepatch: Include sizes when an mismatch occurs

2017-09-20 Thread Konrad Rzeszutek Wilk
ail if the size of the .ex_table was being zero - but that is OK. Signed-off-by: Konrad Rzeszutek Wilk --- Cc: Ross Lagerwall v1: First posting. v2: - Changed to 'Multiple' per Jan's recommendation. - Folded the checks in 'check_size' function and removed all the oth

[Xen-devel] [PATCH v4 04/11] livepatch/arm[32, 64]: Don't load and crash on livepatches loaded with wrong text alignment.

2017-09-20 Thread Konrad Rzeszutek Wilk
for all as we only implement a small subset of them - as such we only check for data types that are implemented - and if the type is anything else and not aligned to 32-bit, then we error out. Signed-off-by: Konrad Rzeszutek Wilk --- Cc: Ross Lagerwall Cc: Andrew Cooper Cc: George Dunlap Cc: I

[Xen-devel] [PATCH v4 09/11] livepatch/tests: Make sure all .livepatch.funcs sections are read-only

2017-09-20 Thread Konrad Rzeszutek Wilk
Instead of being writable (.data). This mimics the behavior of what livepatch-build-tools do. Other approaches such as 'struct const livepatch_funcs' still result in the WA section attributes. Signed-off-by: Konrad Rzeszutek Wilk --- Cc: Ross Lagerwall v2: First posting. v3: Don

[Xen-devel] [PATCH v4 06/11] mkhex: Move it to tools/misc

2017-09-20 Thread Konrad Rzeszutek Wilk
It makes more sense to put a tool to be used by other subsystems to be in 'tools/misc' along 'mkrpm','mkdeb', etc. The patch titled "xen/livepatch/x86/arm32: Force .livepatch.depends section to be uint32_t aligned" uses mkhex. Suggested-by: Julien Gra

[Xen-devel] [PATCH v4] Livepatching patch set for 4.10

2017-09-20 Thread Konrad Rzeszutek Wilk
g-for-4.10.v4 Konrad Rzeszutek Wilk (10): livepatch: Expand check for safe_for_reapply if livepatch has only .rodata. livepatch: Tighten alignment checks. livepatch: Include sizes when an mismatch occurs livepatch/arm[32,64]: Don't load and crash on livepatches loaded w

[Xen-devel] [PATCH v4 01/11] livepatch: Expand check for safe_for_reapply if livepatch has only .rodata.

2017-09-20 Thread Konrad Rzeszutek Wilk
by: Konrad Rzeszutek Wilk --- Cc: Ross Lagerwall v3: First posting. --- xen/common/livepatch.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/xen/common/livepatch.c b/xen/common/livepatch.c index 66167a5573..b0dcd415ba 100644 --- a/xen/common/livepatch.c +++ b/xen/common/l

[Xen-devel] [PATCH v4 05/11] alternative/x86/arm32: Align altinstructions (and altinstr_replacement) sections.

2017-09-20 Thread Konrad Rzeszutek Wilk
/0 grow/shrink: 0/0 up/down: 0/0 (0) function old new delta Total: Before=822563, After=822563, chg +0.00% Also since the macros have the alignment coded in them there is no need to do that for the xen.lds.S anymore. Signed-off-by: Konrad Rzeszutek Wilk --- C

[Xen-devel] [PATCH v4 02/11] livepatch: Tighten alignment checks.

2017-09-20 Thread Konrad Rzeszutek Wilk
-by: Jan Beulich Signed-off-by: Konrad Rzeszutek Wilk --- Cc: Ross Lagerwall v1: Initial patch v2: Drop the check when loading it in memory Add check for alignment being anything but power of two (ignoring 0, and 1) Change dprintk to include hex values and print addr not size. v3: Change

[Xen-devel] [PATCH v4 07/11] livepatch/x86/arm[32, 64]: Force .livepatch.depends section to be uint32_t aligned.

2017-09-20 Thread Konrad Rzeszutek Wilk
ote_depends 49: 36 OBJECT GLOBAL HIDDEN17 note_depends 16: 36 OBJECT GLOBAL HIDDEN 3 note_depends 21: 36 OBJECT GLOBAL HIDDEN 6 note_depends See patch titled: "livepatch/arm/x86: Rename note_depends symbol from test-ca

[Xen-devel] [PATCH v4 11/11] livepatch: Declare live patching as a supported feature

2017-09-20 Thread Konrad Rzeszutek Wilk
From: Ross Lagerwall See docs/features/livepatch.pandoc for the details. Reviewed-by: Jan Beulich Signed-off-by: Ross Lagerwall Signed-off-by: Konrad Rzeszutek Wilk -- v2: - Moved it into a feature document. - Clarified a few bits and pieces based on feedback. v3: - default X86 - added

[Xen-devel] [PATCH v4 10/11] livepatch/arm[32, 64]: Modify .livepatch.funcs section to be RW when patching

2017-09-20 Thread Konrad Rzeszutek Wilk
the 'W' flag during loading of the .livepatch_funcs which would result the section being in writeable region from the gecko. Suggested-by: Julien Grall Signed-off-by: Konrad Rzeszutek Wilk --- Cc: Ross Lagerwall v4: First posting --- xen/arch/arm/livepatch.c| 45

Re: [Xen-devel] ARM64:Porting xen to new hardware

2017-09-22 Thread Konrad Rzeszutek Wilk
m0 > bringup issue. You can also hit 'Ctrl-a' three times and then 'd' which would give you the stack trace and EIP of dom0. That should help in figuirng where your dom0 is stuck. > > Thanks, > Bharat > > On Mon, Sep 18, 2017 at 8:16 PM, Konrad Rzeszutek W

Re: [Xen-devel] Xen Porting Inquiry

2017-09-22 Thread Konrad Rzeszutek Wilk
On Fri, Sep 22, 2017 at 09:31:26AM -0700, t...@hughes.net wrote: > Hello, > > Who can I contact at Xen to obtain an estimate of the effort required to port > Xen to IBM's OpenPOWER 9 CPU? I would recommend you speak to IBM as they would be most qualified to answer that question. ___

Re: [Xen-devel] pci-passthrough loses msi-x interrupts ability after domain destroy

2017-09-22 Thread Konrad Rzeszutek Wilk
On Fri, Sep 22, 2017 at 09:35:40AM +0200, Sander Eikelenboom wrote: > On 22/09/17 04:09, Christopher Clark wrote: > > On Thu, Sep 21, 2017 at 1:27 PM, Sander Eikelenboom > > wrote: > >> > >> On Thu, September 21, 2017, 10:39:52 AM, Roger Pau Monné wrote: > >> > >>> On Wed, Sep 20, 2017 at 03:50:35

Re: [Xen-devel] [PATCH v4 3/3] MAINTAINERS: xen, kvm: track pvclock-abi.h changes

2017-09-27 Thread Konrad Rzeszutek Wilk
eader file, hence this commit simply enforces it more > explicitly and have both peers noticed when such changes happen. > > Signed-off-by: Joao Martins > Acked-by: Juergen Gross Reviewed-by: Konrad Rzeszutek Wilk > --- > In the end, I choose the originally posted because t

Re: [Xen-devel] ARM Hikey 960 - Missing Image Files for Xen

2017-09-28 Thread Konrad Rzeszutek Wilk
On Thu, Sep 28, 2017 at 08:29:27AM -0400, John P. McDermott (USN Civilian) wrote: > Xen Developers, > > The guidance for running Xen on a HiKey 960 identifies files that should be > flashed onto the board, as part of the process for installing Xen. > > https://wiki.xen.org/wiki/HiKey960 >

[Xen-devel] Tips on how to debug EFI code (iPXE) from within KVM after ipxe.efi has crashed with #GP?

2017-09-28 Thread Konrad Rzeszutek Wilk
Hey, I am hitting an interesting issue with the latest versions of iPXE, edk2, and GRUB wherein I am trying to boot GRUB (via iPXE), and then load Xen, I get this: !!! X64 Exception Type - 0D(#GP - General Protection) CPU Apic ID - ExceptionData - RIP - BE

[Xen-devel] GRUB2 missing multiboot2 patches?Re: Only 1 CPU was detected

2017-09-28 Thread Konrad Rzeszutek Wilk
On Thu, Sep 28, 2017 at 02:45:38PM +, Hongjiang Zhang wrote: > >> (XEN) ACPI Error (tbxfroot-0218): A valid RSDP was not found > >> [20070126] > > > >Uuh, that is rather bad, I guess. I am going to assume this is due to not having: b4d709b6e Use grub-file to figure out whether multiboot2 shou

Re: [Xen-devel] GRUB2 missing multiboot2 patches?Re: Only 1 CPU was detected

2017-09-29 Thread Konrad Rzeszutek Wilk
On Fri, Sep 29, 2017 at 11:49:39AM +0100, Michael Young wrote: > > > On Thu, 28 Sep 2017, Hongjiang Zhang wrote: > > > > -Original Message- > > > From: Stefan Bader [mailto:stefan.ba...@canonical.com] > > > Sent: Friday, September 29, 2017

Re: [Xen-devel] Formal vote on Unicore Proposal (deadline Frid, Oct 6th)

2017-09-29 Thread Konrad Rzeszutek Wilk
On Fri, Sep 29, 2017 at 12:07:47PM +, Lars Kurth wrote: > Dear committers, > > in accordance with https://www.xenproject.org/governance.html, I need the > leadership teams of the two mature projects – the Hypervisor and the XAPI > project – to vote on this proposal. > > The Advisory Board i

[Xen-devel] (Offlist)Re: ARM64 HiKey 960 Xen BUG at page_alloc.c:738

2017-09-29 Thread Konrad Rzeszutek Wilk
On Fri, Sep 29, 2017 at 10:57:11AM -0400, John P. McDermott (USN Civilian) wrote: > Xen Developers, > > Trying to run Xen ARM64 on HiKey 960. Following the guidance on the wiki, > building on Ubuntu 17, everything works as expected until Xen boots. The > interesting tail of the serial console o

Re: [Xen-devel] [PATCH v3 0/1] netif: staging grants for I/O requests

2017-09-29 Thread Konrad Rzeszutek Wilk
this and applicability for other > use-cases alongside the necessary changes. This proposal is an RFC and also s/is an RFC and also/ And with that Reviewed-by: Konrad Rzeszutek Wilk ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v4 1/1] public/io/netif.h: add gref mapping control messages

2017-09-29 Thread Konrad Rzeszutek Wilk
On Wed, Sep 20, 2017 at 08:23:20AM +, Paul Durrant wrote: > > -Original Message- > > From: Joao Martins [mailto:joao.m.mart...@oracle.com] > > Sent: 19 September 2017 20:08 > > To: Xen-devel > > Cc: Wei Liu ; Paul Durrant ; > > Konrad Rzeszutek

Re: [Xen-devel] [PATCH v3 9/9] xen/arm: mm: Use memory flags for modify_xen_mappings rather than custom one

2017-10-02 Thread Konrad Rzeszutek Wilk
; > --- > Reviewed-by: Konrad Rzeszutek Wilk > Cc: Ross Lagerwall > > arch_livepatch_secure is now the same as on x86. It might be > possible to combine both, but I left that alone for now. > > Changes in v3: > - Add Stefano's review

Re: [Xen-devel] [PATCH 0/2 v2] xen/acpi: upload PM state from init-domain to Xen

2017-03-24 Thread Konrad Rzeszutek Wilk
is a > chance that we may miss this. I am also copying Konrad since since his > R-b was not given on the public list. Whoops. Reviewed-by: Konrad Rzeszutek Wilk here > > -boris ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] maybe revert commit c275a57f5ec3 "xen/balloon: Set balloon's initial state to number of existing RAM pages"

2017-03-24 Thread Konrad Rzeszutek Wilk
On Fri, Mar 24, 2017 at 04:34:23PM -0400, Dan Streetman wrote: > On Wed, Mar 22, 2017 at 10:13 PM, Boris Ostrovsky > wrote: > > > > > > On 03/22/2017 05:16 PM, Dan Streetman wrote: > >> > >> I have a question about a problem introduced by this commit: > >> c275a57f5ec3056f732843b11659d892235faff7

Re: [Xen-devel] [PATCH v3] xen/arm: alternative: Register re-mapped Xen area as a temporary virtual region

2017-03-27 Thread Konrad Rzeszutek Wilk
a value in the > re-mapped area. But we haven't registered this area as an active kernel text. > So the check of is_active_kernel_text will always return false. > > We have to register the re-mapped Xen area as a virtual region temporarily to > solve this problem. > &g

Re: [Xen-devel] raisin and minios stubdom

2017-03-27 Thread Konrad Rzeszutek Wilk
On Mon, Mar 27, 2017 at 09:28:14PM +0200, Gémes Géza wrote: > Hi, > > Currently the xen build system has optional support for building a minios > (+needed libraries and tools) based stubdom. > > What is your opinion about moving support for building this into raisin and > once that is stable drop

Re: [Xen-devel] raisin and minios stubdom

2017-03-31 Thread Konrad Rzeszutek Wilk
On Thu, Mar 30, 2017 at 07:42:48PM +0200, Gémes Géza wrote: > > > On Mon, Mar 27, 2017 at 09:28:14PM +0200, Gémes Géza wrote: > > > Hi, > > > > > > Currently the xen build system has optional support for building a minios > > > (+needed libraries and tools) based stubdom. > > > > > > What is you

Re: [Xen-devel] Legacy PCI interrupt {de}assertion count

2017-03-31 Thread Konrad Rzeszutek Wilk
On Fri, Mar 31, 2017 at 04:46:27AM -0600, Jan Beulich wrote: > >>> On 31.03.17 at 10:07, wrote: > > On Fri, Mar 31, 2017 at 05:05:44AM +, Tian, Kevin wrote: > >> > From: Jan Beulich [mailto:jbeul...@suse.com] > >> > Sent: Monday, March 27, 2017 4:00 PM > >> > > >> > >>> On 24.03.17 at 17:54,

Re: [Xen-devel] [PATCH v2] xen: make functions in xen-acpi-processor return void

2017-03-31 Thread Konrad Rzeszutek Wilk
On Fri, Mar 31, 2017 at 04:40:56PM +0200, Juergen Gross wrote: > There are several functions in xen-acpi-processor which either always > return the same value or where the returned value is never checked. > > Make the functions return void. Well, we could actually check it and do some extra error

Re: [Xen-devel] [RFC XEN PATCH v2 00/15] Add vNVDIMM support to HVM domains

2017-04-01 Thread Konrad Rzeszutek Wilk
..snip.. > >> Is there a resource I can read more about why the hypervisor needs to > >> have this M2P mapping for nvdimm support? > > > > M2P is basically an array of frame numbers. It's indexed by the host > > page frame number, or the machine frame number (MFN) in Xen's > > definition. The n'th

Re: [Xen-devel] [RFC XEN PATCH v2 15/15] tools/misc: add xen-ndctl

2017-04-01 Thread Konrad Rzeszutek Wilk
On Thu, Mar 30, 2017 at 03:58:25PM +0800, Haozhong Zhang wrote: > On 03/29/17 21:11 -0700, Dan Williams wrote: > > On Sun, Mar 19, 2017 at 5:09 PM, Haozhong Zhang > > wrote: > > > xen-ndctl is a tool for users in Dom0 to setup the host pmem with Xen > > > hypervisor. It's used to specify the stora

Re: [Xen-devel] [RFC XEN PATCH v2 00/15] Add vNVDIMM support to HVM domains

2017-04-01 Thread Konrad Rzeszutek Wilk
On Mon, Mar 20, 2017 at 08:09:34AM +0800, Haozhong Zhang wrote: > This is v2 RFC patch series to add vNVDIMM support to HVM domains. > v1 can be found at > https://lists.xenproject.org/archives/html/xen-devel/2016-10/msg00424.html. > > No label and no _DSM except function 0 "query implemented fun

Re: [Xen-devel] [RFC QEMU PATCH v2 01/10] nvdimm xen: disable label support on Xen

2017-04-01 Thread Konrad Rzeszutek Wilk
On Mon, Mar 20, 2017 at 08:12:40AM +0800, Haozhong Zhang wrote: > If xen_enabled(), memory_region_get_ram_ptr() always returns NULL and > nvdimm_realize() cannot get the correct pointer to the label area. This > commit disables the label support for Xen accelerator to workaround > this issue. > >

Re: [Xen-devel] Legacy PCI interrupt {de}assertion count

2017-04-04 Thread Konrad Rzeszutek Wilk
On Mon, Apr 03, 2017 at 02:22:36PM +0200, Sander Eikelenboom wrote: > On 31/03/17 16:38, Konrad Rzeszutek Wilk wrote: > > On Fri, Mar 31, 2017 at 04:46:27AM -0600, Jan Beulich wrote: > >>>>> On 31.03.17 at 10:07, wrote: > >>> On Fri, Mar 31, 2017

Re: [Xen-devel] [RFC XEN PATCH v2 00/15] Add vNVDIMM support to HVM domains

2017-04-04 Thread Konrad Rzeszutek Wilk
On Sat, Apr 01, 2017 at 08:45:45AM -0700, Dan Williams wrote: > On Sat, Apr 1, 2017 at 4:54 AM, Konrad Rzeszutek Wilk > wrote: > > ..snip.. > >> >> Is there a resource I can read more about why the hypervisor needs to > >> >> have this M2P mappin

Re: [Xen-devel] [RFC XEN PATCH v2 00/15] Add vNVDIMM support to HVM domains

2017-04-04 Thread Konrad Rzeszutek Wilk
On Tue, Apr 04, 2017 at 10:16:41AM -0700, Dan Williams wrote: > On Tue, Apr 4, 2017 at 10:00 AM, Konrad Rzeszutek Wilk > wrote: > > On Sat, Apr 01, 2017 at 08:45:45AM -0700, Dan Williams wrote: > >> On Sat, Apr 1, 2017 at 4:54 AM, Konrad Rzeszutek Wilk > >> wrote:

Re: [Xen-devel] [RFC XEN PATCH v2 00/15] Add vNVDIMM support to HVM domains

2017-04-04 Thread Konrad Rzeszutek Wilk
On Tue, Apr 04, 2017 at 10:59:01AM -0700, Dan Williams wrote: > On Tue, Apr 4, 2017 at 10:34 AM, Konrad Rzeszutek Wilk > wrote: > > On Tue, Apr 04, 2017 at 10:16:41AM -0700, Dan Williams wrote: > >> On Tue, Apr 4, 2017 at 10:00 AM, Konrad Rzeszutek Wilk > >> wrote:

[Xen-devel] [PATCH v2 3/5] tmem: By default to join an shared pool it must be authorized.

2017-04-04 Thread Konrad Rzeszutek Wilk
Having an off by default option allowing guests to join _any_ shared pool is not very secure. Lets eliminate tmem_shared_auth bootup option (which was disabled by default) and have the code force this by default. Signed-off-by: Konrad Rzeszutek Wilk --- docs/misc/tmem-internals.html | 6

[Xen-devel] [PATCH v2 4/5] tmem: Fix tmem-shared-auth 'auth' values

2017-04-04 Thread Konrad Rzeszutek Wilk
The hypervisor code (tmemc_shared_pool_auth) since the inception would consider auth values of: 0 - to disable authentication! 1 - to enable authentication for the given UUID. The docs have it the other way around, so lets fix it. Acked-by: Wei Liu Signed-off-by: Konrad Rzeszutek Wilk

[Xen-devel] [PATCH v2 5/5] tmem: Parse UUIDs correctly.

2017-04-04 Thread Konrad Rzeszutek Wilk
A simple xl tmem-shared-auth -u --000A--0001 -A 0 0 resulted in uuid_low = 1 (correct) and uuid_high = 0 (umm?). The issue was that for hex values above 'A' (or 'a') we forgot to add 10. Acked-by: Wei Liu Signed-off-by: Konrad Rzeszutek Wilk ---

[Xen-devel] [PATCH v2 1/5] xen/libcx/tmem: Replace TMEM_RESTORE_NEW with XEN_SYSCTL_TMEM_OP_SET_POOLS

2017-04-04 Thread Konrad Rzeszutek Wilk
t set to NULL and would be initialized via the guest do_tmem() hypercalls. Presumarily that was the initial reason that TMEM_RESTORE_NEW was in the guest accessible hypercalls. Acked-by: Wei Liu [libxc change] [libxc change] [libxc change] [libxc change] Signed-off-by: Konrad Rzeszutek Wilk

[Xen-devel] [PATCH v2 2/5] xen/libxc: Move TMEM_AUTH to XEN_SYSCTL_TMEM_OP_SET_AUTH

2017-04-04 Thread Konrad Rzeszutek Wilk
emaining issues. We also have to call client_create as the guest at this point may not have done any tmem hypercalls - and hence the '->tmem' from 'struct domain' is still NULL. Us calling client_create fixes this. Signed-off-by: Konrad Rzeszutek Wilk --- Cc: Ian Jackson Cc: W

[Xen-devel] [PATCH v2] Tmem fixes for v4.9.

2017-04-04 Thread Konrad Rzeszutek Wilk
public/sysctl.h | 15 -- xen/include/public/tmem.h | 8 +-- xen/include/xen/tmem_control.h | 6 +++ xen/include/xen/tmem_xen.h | 9 12 files changed, 173 insertions(+), 98 deletions(-) Konrad Rzeszutek Wilk (5): xen/libcx/tmem: Replace TMEM_R

Re: [Xen-devel] [PATCH v4] displif: add ABI for para-virtual display

2017-04-04 Thread Konrad Rzeszutek Wilk
Hey! I only had minor comments - and if you are OK with them I can also put them in the patch when checking them in (or if you want you can repost it right away and I will check in the new version). > > --- > Changes since v3: > * add clarification on buffer's size/depth/format and >connec

Re: [Xen-devel] [PATCH v8 1/4] ring.h: introduce macros to handle monodirectional rings with multiple req sizes

2017-04-04 Thread Konrad Rzeszutek Wilk
On Fri, Mar 31, 2017 at 12:15:16PM -0700, Stefano Stabellini wrote: > This patch introduces macros, structs and functions to handle rings in > the format described by docs/misc/pvcalls.markdown and > docs/misc/9pfs.markdown. The index page (struct __name##_data_intf) > contains the indexes and the

Re: [Xen-devel] [PATCH v8 3/4] Introduce the Xen 9pfs transport header

2017-04-04 Thread Konrad Rzeszutek Wilk
se.com > CC: konrad.w...@oracle.com Reviewed-by: Konrad Rzeszutek Wilk > --- > xen/include/Makefile | 4 +++- > xen/include/public/io/9pfs.h | 49 > > 2 files changed, 52 insertions(+), 1 deletion(-) > create mode 100

Re: [Xen-devel] [PATCH v8 4/4] Introduce the pvcalls header

2017-04-04 Thread Konrad Rzeszutek Wilk
CC: jbeul...@suse.com > CC: konrad.w...@oracle.com Reviewed-by: Konrad Rzeszutek Wilk > --- > xen/include/Makefile| 3 +- > xen/include/public/io/pvcalls.h | 153 > > 2 files changed, 155 insertions(+), 1 deletion(-) >

Re: [Xen-devel] [PATCH v4] displif: add ABI for para-virtual display

2017-04-04 Thread Konrad Rzeszutek Wilk
.snip.. > > + *- Protocol version > > -- > > + * > > + * version > > + * Values: > > + * > > + * Protocol version, chosen among the ones supported by the backend. > > + * ..snip.. > > +#define XENDISPL_FIELD_BE_VERSIONS

Re: [Xen-devel] Wondering about cirris and stdvga

2017-04-04 Thread Konrad Rzeszutek Wilk
On Mon, Mar 20, 2017 at 02:25:16PM +, Roger Pau Monne wrote: > On Mon, Mar 20, 2017 at 02:21:50PM +, Paul Durrant wrote: > > > -Original Message- > > > From: Roger Pau Monne > > > Sent: 20 March 2017 14:14 > > > To: Konrad Rzeszutek Wilk

Re: [Xen-devel] [PATCH v5] xen/displif: add ABI for para-virtual display

2017-04-05 Thread Konrad Rzeszutek Wilk
On Wed, Apr 05, 2017 at 09:11:43AM +0300, Oleksandr Andrushchenko wrote: > For your convenience, PFA diff between v4..v5 Fantastic! Thank you! ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v5] displif: add ABI for para-virtual display

2017-04-05 Thread Konrad Rzeszutek Wilk
On Wed, Apr 05, 2017 at 09:09:07AM +0300, Oleksandr Andrushchenko wrote: > From: Oleksandr Andrushchenko .. snip.. > > --- ..snip.. > Signed-off-by: Oleksandr Grytsov > Signed-off-by: Oleksandr Andrushchenko Please in the future to put these SoB above the ---. Otherwise when I do 'git am' it

Re: [Xen-devel] [PATCH v2 3/5] tmem: By default to join an shared pool it must be authorized.

2017-04-05 Thread Konrad Rzeszutek Wilk
On Wed, Apr 05, 2017 at 03:36:51AM -0600, Jan Beulich wrote: > >>> On 04.04.17 at 21:10, wrote: > > @@ -1530,7 +1529,8 @@ int do_tmem_new_pool(domid_t this_cli_id, > > pool->shared = 0; > > goto out; > > } > > -if ( client->shared_auth_required && !tmem_g

Re: [Xen-devel] [PATCH v9 1/4] ring.h: introduce macros to handle monodirectional rings with multiple req sizes

2017-04-05 Thread Konrad Rzeszutek Wilk
> Signed-off-by: Stefano Stabellini > CC: konrad.w...@oracle.com Reviewed-by: Konrad Rzeszutek Wilk ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v1 0/2] libxl: add PV display device driver interface

2017-04-05 Thread Konrad Rzeszutek Wilk
On Wed, Apr 05, 2017 at 02:49:26PM +0100, Ian Jackson wrote: > al1img . writes ("Re: [PATCH v1 0/2] libxl: add PV display device driver > interface"): > > On Wed, Apr 5, 2017 at 2:05 PM, Ian Jackson > > wrote: > > > Sorry to show my ignorance, but what is a "PV display device" ? > > > Where is t

Re: [Xen-devel] [PATCH v9 1/4] ring.h: introduce macros to handle monodirectional rings with multiple req sizes

2017-04-05 Thread Konrad Rzeszutek Wilk
to define another macro with a > variable name. For this reason, this patch introduces static inline > functions instead, that are not C89 compliant. Additionally, the macro > defines a struct with a variable sized array, which is also not C89 > compliant. > > Signed-off-by: Stefano St

Re: [Xen-devel] [PATCH] tools/golang: Add myself as maintainer

2017-04-06 Thread Konrad Rzeszutek Wilk
On Thu, Apr 06, 2017 at 03:57:06PM +0100, George Dunlap wrote: > Signed-off-by: George Dunlap > --- > Andrew Cooper > Ian Jackson > Jan Beulich > Konrad Rzeszutek Wilk > Stefano Stabellini > Tim Deegan > Wei Liu > --- > MAINTAINERS | 7 +++ > 1 fil

Re: [Xen-devel] [For Linux 1/4] xen/kbdif: update protocol description

2017-04-07 Thread Konrad Rzeszutek Wilk
On Fri, Apr 07, 2017 at 01:29:27PM +0200, Juergen Gross wrote: > On 07/04/17 10:30, Oleksandr Andrushchenko wrote: > > From: Oleksandr Andrushchenko > > > > The patch clarifies the protocol that is used by the PV keyboard > > drivers. > > > > Reviewed-by:

Re: [Xen-devel] [PATCH] tools/golang: Add myself as maintainer

2017-04-10 Thread Konrad Rzeszutek Wilk
On Mon, Apr 10, 2017 at 12:21:01PM +0100, George Dunlap wrote: > On 10/04/17 11:37, George Dunlap wrote: > > On 06/04/17 20:52, Konrad Rzeszutek Wilk wrote: > >> On Thu, Apr 06, 2017 at 03:57:06PM +0100, George Dunlap wrote: > >>> Signed-off-by: George Dunlap

Re: [Xen-devel] PCI passthrough will cause unknown device appearance in device manager of Windows OS via xenstore keys

2017-04-10 Thread Konrad Rzeszutek Wilk
libxl-Don-t-insert-PCI-device-into-xenstore-for-HVM-.patch Attaching the flr one (inline) and attachemnt >From 314a0ff66af9380de4cd4c8a7f4f648c6f593bb8 Mon Sep 17 00:00:00 2001 From: Konrad Rzeszutek Wilk Date: Wed, 3 Dec 2014 15:47:02 -0500 Subject: [PATCH] xen/pciback: Implement PCI reset

Re: [Xen-devel] PCI passthrough will cause unknown device appearance in device manager of Windows OS via xenstore keys

2017-04-10 Thread Konrad Rzeszutek Wilk
On Mon, Apr 10, 2017 at 02:28:58PM +0100, Wei Liu wrote: > On Mon, Apr 10, 2017 at 01:46:20PM +0100, Andrew Cooper wrote: > > On 10/04/17 12:24, lidonglin wrote: > > > > > > Hi all: > > > > > > I have one question about PCI passthrough. I found > > > that if I created a VM with host

Re: [Xen-devel] PCI passthrough will cause unknown device appearance in device manager of Windows OS via xenstore keys

2017-04-10 Thread Konrad Rzeszutek Wilk
On Mon, Apr 10, 2017 at 03:21:41PM +0100, Andrew Cooper wrote: > On 10/04/17 15:12, Konrad Rzeszutek Wilk wrote: > > On Mon, Apr 10, 2017 at 02:28:58PM +0100, Wei Liu wrote: > >> On Mon, Apr 10, 2017 at 01:46:20PM +0100, Andrew Cooper wrote: > >>> On 10/04/17 12:24,

Re: [Xen-devel] [RFC XEN PATCH v2 00/15] Add vNVDIMM support to HVM domains

2017-04-11 Thread Konrad Rzeszutek Wilk
On Tue, Apr 04, 2017 at 11:59:03AM -0700, Dan Williams wrote: > >> I don't think KVM has the same issue, but honestly I don't have the > >> full mental model of how KVM supports mmap. I've at least been able to > >> run a guest where the "pmem" is just dynamic page cache on the host > >> side so th

Re: [Xen-devel] [PATCH] blkfront: add uevent for size change

2017-04-11 Thread Konrad Rzeszutek Wilk
On Tue, Apr 11, 2017 at 12:24:09PM -0700, Marc Olson wrote: > When a blkfront device is resized from dom0, emit a KOBJ_CHANGE uevent to > notify the guest about the change. This allows for custom udev rules, such > as automatically resizing a filesystem, when an event occurs. Looks pretty reasonab

Re: [Xen-devel] PCI passthrough will cause unknown device appearance in device manager of Windows OS via xenstore keys

2017-04-12 Thread Konrad Rzeszutek Wilk
On Wed, Apr 12, 2017 at 04:46:33PM +0100, Wei Liu wrote: > On Mon, Apr 10, 2017 at 09:43:13AM -0400, Konrad Rzeszutek Wilk wrote: > [...] > > > > .. Except that we need some way of doing FLR and Pciback > > is the one doing it. > > > > The right way would

Re: [Xen-devel] PCI passthrough will cause unknown device appearance in device manager of Windows OS via xenstore keys

2017-04-13 Thread Konrad Rzeszutek Wilk
On April 13, 2017 4:41:03 AM EDT, Wei Liu wrote: >On Wed, Apr 12, 2017 at 04:21:42PM -0400, Konrad Rzeszutek Wilk wrote: >> On Wed, Apr 12, 2017 at 04:46:33PM +0100, Wei Liu wrote: >> > On Mon, Apr 10, 2017 at 09:43:13AM -0400, Konrad Rzeszutek Wilk >wrote: >> >

Re: [Xen-devel] [PATCH] pass-through: drop pointless uses of __func__

2016-08-24 Thread Konrad Rzeszutek Wilk
On Wed, Aug 24, 2016 at 01:51:13AM -0600, Jan Beulich wrote: > Non-debugging message text should be (and is in the cases here) > distinguishable without also logging function names. Additionally log > the PCI device coordinates for alloc_pdev() failure. > Reviewed-by: Konrad Rz

Re: [Xen-devel] [PATCH] VT-d: drop pointless uses of __func__

2016-08-24 Thread Konrad Rzeszutek Wilk
On Wed, Aug 24, 2016 at 01:54:08AM -0600, Jan Beulich wrote: > Debugging message text already includes file name and line number, so > also logging function names is redundant. And you also fixed one style guide violation :-) > > Signed-off-by: Jan Beulich Reviewed-by: Konrad Rz

<    4   5   6   7   8   9   10   11   12   13   >