> - of standard dom0 handled console */
Please keep this comment somewhere, probably here:
> CONFIG_BALLOON ?= n
> +CONFIG_USE_XEN_CONSOLE ?= n
apart from that,
Acked-by: Samuel Thibault
Samuel
___
Xen-devel mailing list
Xen-devel@lists.xe
Wei Liu, on mer. 04 oct. 2017 14:51:26 +0100, wrote:
> Starting from Xen commit af8d93564, it is required to include string.h
> before console.h.
>
> Signed-off-by: Wei Liu
Reviewed-by: Samuel Thibault
> ---
> Cc: Ian Jackson
> Cc: Julien Grall
> Cc: Samuel Thiba
Wei Liu, on mer. 04 oct. 2017 14:33:36 +0100, wrote:
> Starting from Xen commit af8d93564, it is required to include string.h
> before console.h.
>
> Signed-off-by: Wei Liu
Reviewed-by: Samuel Thibault
> ---
> Cc: Ian Jackson
> Cc: Julien Grall
> Cc: Samuel Thiba
Hello,
Anil Madhavapeddy, on jeu. 14 sept. 2017 15:38:54 +0100, wrote:
> On 13 Sep 2017, at 17:13, Samuel Thibault
> wrote:
> > Anil Madhavapeddy, on mer. 13 sept. 2017 11:11:03 +0100, wrote:
> >> Maintaining a forked MiniOS has been a multi-year source of a mainten
Hello,
Anil Madhavapeddy, on mer. 13 sept. 2017 11:11:03 +0100, wrote:
> Maintaining a forked MiniOS has been a multi-year source of a maintenance
> burden for MirageOS,
I'm just wondering why this happened?
The mini-os repository is open for development, it's just a matter of
agreeing on how t
Hello,
I'm all for it :)
I won't have time to spend for contributions, but I can probably answer
questions about the existing source code, challenges with stubdomains
etc.
The only thing I'd worry about is to be clear on the scope of Unicore:
we don't want to just create another Linux kernel. Co
Wei Liu, on mer. 09 août 2017 13:15:48 +0100, wrote:
> Some code compiled by gcc 7 requires this.
>
> Signed-off-by: Wei Liu
Reviewed-by: Samuel Thibault
> ---
> lib/math.c | 10 ++
> 1 file changed, 10 insertions(+)
>
> diff --git a/lib/math.c b/lib/math.c
n targets for tools/include.
>
> The new arrangement ensures tools build gets the correct headers
> because XEN_OS is set to host os when building tools/include. As for
> stubdom, it explicitly links to the mini-os directory without relying
> on XEN_OS so it should fine.
>
> Repor
using absolute paths in tools/include Makefile. Provide
> mechanism to override the prefix in paths. Use that mechanism in
> stubdom build to reduce code duplication.
>
> Reported-by: Steven Haigh
> Signed-off-by: Wei Liu
Acked-by: Samuel Thibault
___
Hello,
Felix Schmoll, on mer. 29 mars 2017 20:53:14 +0200, wrote:
> -While implementing our own kernel last semester me and my team-mate
> came to believe that pusha/popa were faster that pushing/popping the
> individual registers, since it is just a single command. The Mini-OS
> kernel however do
gned-off-by: Paul Durrant
> Cc: Samuel Thibault
Acked-by: Samuel Thibault
> ---
> Makefile | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/Makefile b/Makefile
> index 74f2c31..ef8559b 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -134,
mit;h=41c9f2ae
> > > > >
> > >
> > > Need an ack from Samuel
Acked-by: Samuel Thibault
Samuel
___
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
fied by each "make tools" call.
>
> This can be avoided by writing stubdom/ioemu/linkfarm.stamp only if
> a source file beneath tools/qemu-xen-traditional-dir has been added
> or removed.
>
> Signed-off-by: Juergen Gross
> Acked-by: Wei Liu
Reviewed-by: Samuel Thibau
> > Signed-off-by: Wei Liu
>
> Reviewed-by: Juergen Gross
Acked-by: Samuel Thibault
___
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
Hello,
Jan Beulich, on Wed 07 Dec 2016 09:28:52 -0700, wrote:
> > Right. Jan, could you please include the patch for 4.7.2? Upstream
> > commit was 9714f6b87e19b32d3a6663a20df6610265c4bfe5.
>
> Samuel, could you confirm that this is okay to backport.
For 4.7, yes.
> In which case the question t
move flags which are meant for tools, e.g. "-m64"
> > +export EXTRA_CFLAGS_XEN_TOOLS=
> > +export EXTRA_CFLAGS_QEMU_TRADITIONAL=
> > +
>
> I can't think of a better way to deal with this. This is in line with
> how it is done in firmware build, so
>
> Ack
inked Makefile, too.
>
> At the same time don't repeat the same link pattern 7 times but use a
> make macro to do the linking.
>
> Signed-off-by: Juergen Gross
Reviewed-by: Samuel Thibault
> ---
> stubdom/Makefile | 77
> ++-
Juergen Gross, on Fri 28 Oct 2016 16:53:20 +0200, wrote:
> make distclean tries to remove stubdom/gmp-4.3.2.tar.gz, while the
> downloaded file is stubdom/gmp-4.3.2.tar.bz2
>
> Signed-off-by: Juergen Gross
Acked-by: Samuel Thibault
> ---
> stubdom/Makefile | 2 +-
&g
or another architecture.
> >
> > This should fix the issue that GMP doesn't compile with gcc 6, because
> > configure script won't try to test the host environment anymore.
>
> Acked-by: Ian Jackson
Acked-by: Samuel Thibault
Jan Beulich, on Mon 24 Oct 2016 07:12:27 -0600, wrote:
> And btw - is it really qemu that does this?
It's pciaccess which does it.
> I didn't think it would be a user space app doing any such sizing; I
> thought this would get done elsewhere in stubdom/mini-os.
mini-os just lets libpciaccess use
Hello,
Eric Shelton, on Fri 21 Oct 2016 09:01:43 -0400, wrote:
> ERROR: PCI region size must be pow2 type=0x8, size=0xdf08
> u32 u = pci_read_long(d, reg);
> if (u != 0x)
> - d->rom_base_addr = u;
> +{
> + d->rom_base_addr = u;
>
Hello,
Marek Marczykowski-Górecki, on Tue 18 Oct 2016 03:53:30 +0200, wrote:
> This series is follow up to previous attempts to fix this. Related threads:
> - http://markmail.org/thread/dwjcdfk3y7s5c5kl "PCI passthrough with
> stubdomain"
> - http://xen.markmail.org/thread/l7tvqcxbiyc2grvr "Jso
te.h.
>
> Fix this by defining the macros only if compiling a minios source.
>
> Signed-off-by: Juergen Gross
Acked-by: Samuel Thibault
> ---
> include/x86/os.h | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/include/x86/os.h b/include
gen Gross
Acked-by: Samuel Thibault
> ---
> stubdom/grub/kexec.c | 22 +++---
> 1 file changed, 11 insertions(+), 11 deletions(-)
>
> diff --git a/stubdom/grub/kexec.c b/stubdom/grub/kexec.c
> index 8fd9ff9..2ed4f6c 100644
> --- a/stubdom/grub/kexec.c
> +++
for linking mini-os_app.o to avoid
> a linker warning.
>
> Signed-off-by: Juergen Gross
Acked-by: Samuel Thibault
> ---
> Makefile | 11 +--
> 1 file changed, 9 insertions(+), 2 deletions(-)
>
> diff --git a/Makefile b/Makefile
> index 81b936f..1d2324c 1
Marek Marczykowski-Górecki, on Fri 23 Sep 2016 20:56:43 +0200, wrote:
> 1. How to do this? ;) I.e. what synchronization primitives are available
> in mini-os? Just pthread_mutex_lock/unlock?
pthread_mutex_lock are nops :o) because we don't have pthread_create.
But for mini-os itself there are sync
Marek Marczykowski-Górecki, on Fri 23 Sep 2016 16:25:41 +0200, wrote:
> On Fri, Sep 23, 2016 at 03:27:07PM +0200, Samuel Thibault wrote:
> > Marek Marczykowski-Górecki, on Fri 23 Sep 2016 10:48:14 +0200, wrote:
> > > I'm still trying to get PCI passthrough working with stubd
Marek Marczykowski-Górecki, on Fri 23 Sep 2016 10:48:14 +0200, wrote:
> I'm still trying to get PCI passthrough working with stubdomain and
> without qemu in dom0 (even for just vfb/vkbd backends). How is this
> supposed to work?
Just as I recall from my memory:
> 1. Should xen-pcifront in the ta
Wei Liu, on Mon 05 Sep 2016 15:43:21 +0100, wrote:
> Signed-off-by: Wei Liu
Acked-by: Samuel Thibault
> ---
> See:
> https://travis-ci.org/liuw/mini-os/builds/157653746
>
> Cc: Samuel Thibault
> Cc: Juergen Gross
> Cc: Doug Goldstein
>
> IRC notification is
Juergen Gross, on Mon 05 Sep 2016 13:43:30 +0200, wrote:
> Add some comment in Config.mk what to do in case of adding new config
> options.
>
> Signed-off-by: Juergen Gross
Reviewed-by: Samuel Thibault
> ---
> Config.mk | 5 +
> 1 file changed, 5 insertions(+)
>
Juergen Gross, on Fri 02 Sep 2016 10:56:47 +0200, wrote:
> Add some notes to README regarding configuration of Mini-OS via config
> files.
>
> Signed-off-by: Juergen Gross
Reviewed-by: Samuel Thibault
> + Please update the current configuration files when adding a new CONFIG
nd there are built in
> a local directory.
Please also document just above the CONFIG_* lines in Config.mk that new
CONFIG options should be set in all-* testbuilds.
> Signed-off-by: Juergen Gross
Reviewed-by: Samuel Thibault
Samuel
___
X
Juergen Gross, on Fri 02 Sep 2016 10:56:45 +0200, wrote:
> Some config settings won't build standalone. Fix the following cases:
>
> - all CONFIG_* set to "n"
> - standard config with latest Xen interface version
>
> Signed-off-by: Juergen Gross
Reviewed-by: S
Juergen Gross, on Fri 02 Sep 2016 07:42:36 +0200, wrote:
> I think I'll add a new make target "test" which will test different
> build configurations (PARAVIRT y/n, BALLOON y/n, 32/64 bit, Xen
> interface versions, all/no frontends). This can be easily added to
> OSStest.
Good, thanks.
Samuel
__
defines according to their Mini-OS configuration.
> >
> > Add a new make target "config" for that purpose creating a Makefile
> > snipplet $(CONFIG_FILE) (defaults to ./minios-config.mk) containing
> > the needed information.
> >
> > Signed-off-by: J
Hello,
Juergen Gross, on Thu 01 Sep 2016 08:21:33 +0200, wrote:
> I stumbled over the problem with xenstore-stubdom: xenstore is using
> __XEN_LATEST_INTERFACE_VERSION__ when being compiled. This produced a
> build error with Mini-OS (console_evtchn in include/console.h was
> #define'd to console.
Hello,
Juergen Gross, on Tue 30 Aug 2016 13:51:23 +0200, wrote:
> @@ -51,7 +51,7 @@ endif
>
> libc = $(stubdom)
>
> -XEN_INTERFACE_VERSION := 0x00030205
> +XEN_INTERFACE_VERSION ?= 0x00030205
Why making it overridable? AIUI changing the version would change the
xen headers API, and thus the
ck.c), and
applications may want to implement some such...
> >> Signed-off-by: Juergen Gross
> >
> > Reviewed-by: Wei Liu
Acked-by: Samuel Thibault
___
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
ry_t must be changed
> > to grant_entry_v1_t. In balloon.c we omit initializing elements of
> > struct xen_memory_reservation with 0 to avoid problems with different
> > named structure elements in different Xen interface versions.
> >
> > Signed-off-by: Juergen Gro
the start_info structure, but make it dependent on
> CONFIG_PARAVIRT.
>
> Signed-off-by: Juergen Gross
Reviewed-by: Samuel Thibault
> ---
> arch/x86/setup.c | 12
> include/hypervisor.h | 13 -
> 2 files changed, 24 insertions(+), 1 deletion(-)
Samuel Thibault, on Mon 29 Aug 2016 22:49:07 +0200, wrote:
> Juergen Gross, on Mon 29 Aug 2016 16:18:26 +0200, wrote:
> > +union start_info_union
> > +{
> > +start_info_t start_info;
> > +char padding[512];
> > +};
>
> Why defining a union? 512 is ac
Hello,
Juergen Gross, on Mon 29 Aug 2016 16:18:26 +0200, wrote:
> +union start_info_union
> +{
> +start_info_t start_info;
> +char padding[512];
> +};
Why defining a union? 512 is actually not enough for start_info_t.
Samuel
___
Xen-devel mail
Wei Liu, on Mon 29 Aug 2016 12:11:21 +0100, wrote:
> On Mon, Aug 29, 2016 at 01:01:09PM +0200, Juergen Gross wrote:
> > get_xenbus() should be called only if CONFIG_XENBUS is set.
> >
> > Signed-off-by: Juergen Gross
>
> Reviewed-by: Wei Liu
Juergen Gross, on Mon 29 Aug 2016 15:07:20 +0200, wrote:
> On 29/08/16 15:01, Wei Liu wrote:
> > There is no SCCS file.
> >
> > Signed-off-by: Wei Liu
>
> Reviewed-by: Juergen Gross
Acked-by: Samuel Thibault
___
Xen-d
Juergen Gross, on Mon 29 Aug 2016 15:07:55 +0200, wrote:
> On 29/08/16 15:01, Wei Liu wrote:
> > Signed-off-by: Wei Liu
>
> Reviewed-by: Juergen Gross
Acked-by: Samuel Thibault
___
Xen-devel mailing list
Xen-devel@list
Juergen Gross, on Mon 29 Aug 2016 15:07:39 +0200, wrote:
> On 29/08/16 15:01, Wei Liu wrote:
> > Use GNU Global to generate source code index.
> >
> > Signed-off-by: Wei Liu
>
> Reviewed-by: Juergen Gross
Acked-by: Samuel Thibault
__
of CONFIG_PARAVIRT defined.
> >
> > Remove the parameter from HYPERVISOR_IRET macro as it is used with
> > 0 only.
> >
> > Signed-off-by: Juergen Gross
>
> Reviewed-by: Wei Liu
Acked-by: Samuel Thibault
___
Xen-d
Juergen Gross, on Fri 26 Aug 2016 16:35:36 +0200, wrote:
> sched.c contains some functions nobody is using. Remove them.
>
> Signed-off-by: Juergen Gross
Reviewed-by: Samuel Thibault
> ---
> sched.c | 48
> 1 file cha
Juergen Gross, on Fri 26 Aug 2016 16:35:34 +0200, wrote:
> arch/x86/x86_32.S has some superfluous instructions. Remove them.
>
> Signed-off-by: Juergen Gross
These are indeed remnants from the past.
Reviewed-by: Samuel Thibault
> ---
> arch/x86/x86_32.S | 7 +--
>
entries
For the whole series,
Acked-by: Samuel Thibault
___
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
S
>
> This enables Mini-OS to start the boot process in HVMlite mode until it
> enters C code. This is working for 32- and for 64-bit mode.
>
> Signed-off-by: Juergen Gross
> ---
> V2: add some comments as requested by Samuel Thibault
Way clearer, thanks!
Reviewed-by:
: Juergen Gross
Reviewed-by: Samuel Thibault
> ---
> arch/x86/setup.c | 20 ++--
> include/kernel.h | 3 +++
> kernel.c | 1 +
> main.c | 5 +++--
> 4 files changed, 25 insertions(+), 4 deletions(-)
>
> diff --git a/arch/x86/setup.c b/ar
r in the timer handler as there is no
> guarantee the original timer event we are waiting for won't fire
> between enabling interrupts and calling "hlt".
>
> Signed-off-by: Juergen Gross
Reviewed-by: Samuel Thibault
> ---
> arch/x86/time.c | 11 +++
> 1
with a maximum
> memory size.
>
> Signed-off-by: Juergen Gross
Reviewed-by: Samuel Thibault
> ---
> arch/arm/mm.c | 8 ++
> arch/x86/mm.c | 72 ++
> arch/x86/setup.c
", (unsigned long)si->modlist_paddr);
It would be useful to also print nr_modules.
> +printk(" flags: 0x%x\n", (unsigned int)si->flags);
> +printk("cmd_line: %s\n", cmdline);
> +printk(" stack: %p-%p
> Signed-off-by: Juergen Gross
Reviewed-by: Samuel Thibault
> ---
> arch/arm/setup.c | 8
> arch/x86/setup.c | 8
> daytime.c| 2 +-
> include/hypervisor.h | 11 ---
> kernel.c | 6 +++---
> mai
Juergen Gross, on Tue 23 Aug 2016 17:16:04 +0200, wrote:
> Add a service function to map the shared info page on a
> non-paravirtualized system. The code is already existing on ARM side,
> just move it to hypervisor.c.
>
> Signed-off-by: Juergen Gross
Reviewed-by:
de a similar paravirtualized function.
>
> Signed-off-by: Juergen Gross
Reviewed-by: Samuel Thibault
> ---
> arch/arm/setup.c | 16 +---
> arch/x86/setup.c | 1 +
> events.c | 3 +--
> include/xenbus.h | 3 +++
> xenbus/xenbus.c | 40 +++
de a similar paravirtualized function.
>
> Signed-off-by: Juergen Gross
Reviewed-by: Samuel Thibault
> ---
> arch/arm/setup.c | 16 +---
> arch/x86/setup.c | 1 +
> console/xencons_ring.c | 38 ++
> events.c
make this area
> available even in case of undefined CONFIG_BALLOON.
>
> Signed-off-by: Juergen Gross
Reviewed-by: Samuel Thibault
> ---
> arch/arm/balloon.c| 2 --
> arch/arm/mm.c | 5 +
> arch/x86/balloon.c| 13 -
> arch/x86/mm.c |
Juergen Gross, on Tue 23 Aug 2016 17:15:58 +0200, wrote:
> For support of HVMlite don't use mmu_update hypercalls, but write the
> page table entries directly.
>
> Signed-off-by: Juergen Gross
Reviewed-by: Samuel Thibault
> ---
> arch
bles needed for doing the mapping.
>
> Signed-off-by: Juergen Gross
Reviewed-by: Samuel Thibault
> ---
> arch/x86/mm.c | 11 +++
> arch/x86/x86_64.S | 7 +++
> 2 files changed, 18 insertions(+)
>
> diff --git a/arch/x86/mm.c b/arch/x86/mm.c
> index 054
avirtualized environments. Instead of using
> references to start_info use dedicated variables initialized from
> start_info on boot.
>
> Signed-off-by: Juergen Gross
Reviewed-by: Samuel Thibault
> ---
> arch/arm/balloon.c| 5 ---
> arch/arm/mm.c
Juergen Gross, on Tue 23 Aug 2016 17:15:55 +0200, wrote:
> Trap handling in HVMlite domain is different from pv one.
>
> Signed-off-by: Juergen Gross
Reviewed-by: Samuel Thibault
> ---
> arch/x86/traps.c | 2 --
> arch/x86/x86_32.S | 19 ++--
> a
Juergen Gross, on Tue 23 Aug 2016 17:15:54 +0200, wrote:
> Trap handling for HVMlite domains requires an initialized IDT.
>
> Signed-off-by: Juergen Gross
Reviewed-by: Samuel Thibault
> ---
> arch/x86/setup.c | 15 --
> arch
Juergen Gross, on Tue 23 Aug 2016 17:15:53 +0200, wrote:
> Support the HYPERVISOR_hvm_op() hypercall which is needed for HVMlite.
> Add convenience functions hvm_get_parameter() and hvm_set_parameter().
>
> Signed-off-by: Juergen Gross
Reviewed-by: Samuel Thibault
> ---
>
ot;Unsupported architecture"
> -#endif
> #endif /* ifdef __INSIDE_MINIOS */
Then please move it after
> /* common i386 and x86_64 /
This line :)
Apart from that,
Reviewed-by: Samuel Thibault
Samuel
___
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
Hello,
It looks good, there is just:
Juergen Gross, on Tue 23 Aug 2016 17:15:51 +0200, wrote:
> +.macro PTES prot, addr, incr, idx, end
Please document what prot, addr, incr, idx, end represent precisely.
> +.ifgt \end-\idx-32
> +PTES \prot, \addr, \incr, \idx, "(\idx+32
osen as the arm port can then drop some dummy routines needed for
> para-virtualization only.
>
> Add include/paravirt.h for furute support of paravirt specific
> handling.
future*
> Signed-off-by: Juergen Gross
Reviewed-by: Samuel Thibault
> ---
> Config.mk |
corrected nevertheless.
Oops, indeed!
> Signed-off-by: Juergen Gross
Reviewed-by: Samuel Thibault
> ---
> balloon.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/balloon.c b/balloon.c
> index 8d2f151..8669edb 100644
> --- a/balloo
e.
>
> Hide some C prototypes when in assembler environment.
>
> Signed-off-by: Juergen Gross
Reviewed-by: Samuel Thibault
> ---
> include/x86/arch_mm.h | 56
> +++
> 1 file changed, 34 insertions(+), 22 deletions(-)
>
Juergen Gross, on Tue 23 Aug 2016 17:15:48 +0200, wrote:
> dump_regs() will result in page fault in early boot as there is no
> current thread pointer. Handle this case.
>
> Signed-off-by: Juergen Gross
Reviewed-by: Samuel Thibault
> ---
> arch/x86/traps.c | 2 +-
&g
Juergen Gross, on Tue 23 Aug 2016 17:15:47 +0200, wrote:
> Use the latest Xen headers.
>
> Signed-off-by: Juergen Gross
Reviewed-by: Samuel Thibault
> ---
> include/xen/arch-x86/hvm/start_info.h | 98
> +++
> include/xen/elfnote.h
Juergen Gross, on Wed 17 Aug 2016 15:39:59 +0200, wrote:
> Fix two issues discovered by coverity.
Thanks for processing mini-os through coverity :)
Samuel
___
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
rity:
>
> 1. properl mark one switch case as fall-through
> 2. unroll a loop that only executes once
>
> CID: 1369623
> CID: 1019001
>
>
> >
> > Signed-off-by: Juergen Gross
>
> Reviewed-by: Wei Liu
Acked-by: Samuel Thibault
> > ---
>
top-level directory to cope with the change.
> > Ideally the build system should also be made more elegant, but
> > overhauling the build system is out of scope of this patch.
> >
> > Signed-off-by: Wei Liu
>
> Reviewed-by: Juergen Gross
Acked-by: Samuel Thibault
Wei Liu, on Thu 18 Aug 2016 11:15:25 +0100, wrote:
> Use the newer ELF note interface. The generated ELF notes results in
> equivalent configuration.
>
> Also need to modify linker script to provide a note section.
>
> Signed-off-by: Wei Liu
> Reviewed-by: Juergen Gro
Juergen Gross, on Thu 18 Aug 2016 12:34:48 +0200, wrote:
> On 18/08/16 12:15, Wei Liu wrote:
> > Ported from Xen Test Framework project (BSD license).
> >
> > Signed-off-by: Wei Liu
>
> Reviewed-by: Juergen Gross
Acked-by: Samuel Thibault
Hello,
Wei Liu, on Wed 17 Aug 2016 13:35:12 +0100, wrote:
> Ported from xtf.git.
What is xtf.git? Does it use the same BSD licencing?
To my knowledge this is coming from the linux kernel source.
> Signed-off-by: Wei Liu
> ---
> include/asm_macros.h | 36
Wei Liu, on Wed 17 Aug 2016 13:35:11 +0100, wrote:
> Signed-off-by: Wei Liu
Reviewed-by: Samuel Thibault
> ---
> .gitignore | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/.gitignore b/.gitignore
> index 1c655a0..f21cc46 100644
> --- a/.gitignore
> +++
;s gnttab library.
>
> Ideally BUILD_BUG_ON should be moved to a private header, but there is
> actually no user of it in mini-os tree, just remove it.
>
> Signed-off-by: Wei Liu
Acked-by: Samuel Thibault
> ---
> include/lib.h | 9 -
> 1 file changed, 9 delet
nction inlined won't buy us much and is causing error in
> vtpm manager stubdom build.
>
> Signed-off-by: Wei Liu
Acked-by: Samuel Thibault
> ---
> arch/x86/mm.c | 9 +
> include/x86/arch_mm.h | 11 +++
> 2 files changed, 12 insertions(+), 8 d
the used ones
> into Config.mk and delete the rest.
>
> The CONFIG_* defines should be set for assembler sources, too.
>
> Signed-off-by: Juergen Gross
Acked-by: Samuel Thibault
> ---
> Config.mk | 93
> +++
>
Juergen Gross, on Thu 11 Aug 2016 13:06:36 +0200, wrote:
> In case of CONFIG_BALLOON the page allocator's bitmap needs some space
> to be able to grow. Remap it to kernel virtual area if the preallocated
> area isn't large enough.
>
> Signed-off-by: Juergen Gross
Rev
Juergen Gross, on Thu 11 Aug 2016 12:19:20 +0200, wrote:
> On 11/08/16 11:40, Samuel Thibault wrote:
> > Juergen Gross, on Thu 11 Aug 2016 11:18:19 +0200, wrote:
> >> +extern unsigned long *mm_bitmap;
> >> +extern unsigned long mm_bitmap_size;
> >
> &
Juergen Gross, on Thu 11 Aug 2016 11:18:19 +0200, wrote:
> +extern unsigned long *mm_bitmap;
> +extern unsigned long mm_bitmap_size;
Ah I was thinking to have these use mm_malloc_ too. "mm_bitmap" seems
short for namespace pollution.
Samuel
___
Xen-dev
Hello,
Juergen Gross, on Fri 05 Aug 2016 19:35:44 +0200, wrote:
> Support ballooning Mini-OS automatically up in case of memory shortage.
Thanks!
Samuel
___
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
y;
I would have written it
> +if ( !chk_free_pages(1UL << order) )
> +goto no_memory;
"If there are not enough free pages for this amount, go out".
Otherwise it looks good,
Reviewed-by: Samuel Thibault
Samuel
___
Xen
BALLOON_FRAMES )
> +n_pages = N_BALLOON_FRAMES;
> +
> +/* Resize alloc_bitmap if necessary. */
> +if ( alloc_bitmap_size * 8 < nr_mem_pages + n_pages )
To be on the safe side, I'd say use a while here, even if the value of
N_BALLOON_FRAMES is so that there'll alway
It looks a bit odd to be remapping something that was just allocated,
but I guess it makes portability easier.
Juergen Gross, on Fri 05 Aug 2016 19:36:00 +0200, wrote:
> diff --git a/balloon.c b/balloon.c
> index 4c18c5c..75b87c8 100644
> --- a/balloon.c
> +++ b/balloon.c
> @@ -44,3 +44,20 @@ void
Juergen Gross, on Fri 05 Aug 2016 19:35:59 +0200, wrote:
> +void arch_remap_p2m(unsigned long max_pfn)
> +{
...
> +
> +virt_kernel_area_end += PAGE_SIZE * p2m_pages(nr_max_pages);
I'd however rather see an assertion against hitting VIRT_DEMAND_AREA
here, even if the figures happen to make it i
space at the end.
>
> Signed-off-by: Juergen Gross
Reviewed-by: Samuel Thibault
> ---
> arch/arm/balloon.c| 2 ++
> arch/x86/balloon.c| 24
> arch/x86/mm.c | 3 +++
> include/balloon.h | 3 +++
> include/x86/arch_mm.h |
, as all levels share the
> same entry format (a plain mfn). So reduce the number of macros
> accordingly.
>
> Add some macros to get the indices into p2m pages from a pfn and make
> use of them in current p2m code.
>
> Signed-off-by: Juergen Gross
Reviewed-by: Samu
Juergen Gross, on Fri 05 Aug 2016 19:35:57 +0200, wrote:
> Add a function to map one physical frame to a specified virtual
> address as read/write. This will be used later multiple times.
>
> Signed-off-by: Juergen Gross
Reviewed-by: Samuel Thibault
> ---
> include
that purpose.
>
> Signed-off-by: Juergen Gross
Reviewed-by: Samuel Thibault
> ---
> arch/x86/mm.c | 14 --
> 1 file changed, 4 insertions(+), 10 deletions(-)
>
> diff --git a/arch/x86/mm.c b/arch/x86/mm.c
> index 12f7fe4..e10c2c5 100644
> --- a/arch/x86/
ions to succeed.
>
> Signed-off-by: Juergen Gross
Reviewed-by: Samuel Thibault
> ---
> arch/arm/mm.c | 8
> arch/x86/mm.c | 26 +++---
> include/mm.h | 2 +-
> 3 files changed, 28 insertions(+), 8 deletions(-)
>
> diff --git a/arch/arm/mm.c b/arc
Juergen Gross, on Fri 05 Aug 2016 19:35:54 +0200, wrote:
> mem_test() isn't used anywhere and its value is rather questionable
> with mini-os being in a mature state. Remove the function.
>
> Signed-off-by: Juergen Gross
Reviewed-by: Samuel Thibault
> ---
&
in order to keep the heap at about 1GB.
>
> The kernel virtual mappings are a new area needed for being able to
> grow the p2m list without having to relocate it in physical memory.
>
> Modify the placement of the demand mappings and heap and adjust the
> memory lay
Juergen Gross, on Fri 05 Aug 2016 19:35:52 +0200, wrote:
> Add support for obtaining the maximum memory size from the hypervisor.
> This will make it possible to support ballooning.
>
> Signed-off-by: Juergen Gross
Acked-by: Samuel Thibault
> diff --git a/mm.c b/mm.c
> inde
Juergen Gross, on Fri 05 Aug 2016 19:35:51 +0200, wrote:
> Add CONFIG_BALLOON defaulting to 'n' as a config item to Mini-OS.
>
> Add balloon.c, balloon.h and arch/*/balloon.c for future use.
>
> Signed-off-by: Juergen Gross
Not much to be against here :)
Acked-by: Sam
1 - 100 of 199 matches
Mail list logo
