Sign Up
Log In
Log In
or
Sign Up
Places
All Projects
Status Monitor
Collapse sidebar
SUSE:SLE-15:Update
xen.16553
5c0f7894-x86-dom0_mem-variants.patch
Overview
Repositories
Revisions
Requests
Users
Attributes
Meta
File 5c0f7894-x86-dom0_mem-variants.patch of Package xen.16553
# Commit 76a68b902062a5ca9053f5cf6a3ab46148cb45f2 # Date 2018-12-11 09:43:00 +0100 # Author Juergen Gross <jgross@suse.com> # Committer Jan Beulich <jbeulich@suse.com> x86: add dom0 memory sizing variants Today the memory size of dom0 can be specified only in terms of bytes (either an absolute value or "host-mem - value"). When dom0 shouldn't be auto-ballooned this requires nearly always a manual adaption of the Xen boot parameters to reflect the actual host memory size. Add more possibilities to specify memory sizes. Today we have: dom0_mem= List of ( min:<size> | max:<size> | <size> ) with <size> being a positive or negative size value (e.g. 1G). Modify that to: dom0_mem= List of ( min:<sz> | max:<sz> | <sz> ) <sz>: <size> | [<size>+]<frac>% <frac>: integer value < 100 With the following semantics: <frac>% specifies a fraction of host memory size in percent. <sz> is a percentage of host memory plus an offset. So <sz> being 1G+25% on a 256G host would result in 65G. Signed-off-by: Juergen Gross <jgross@suse.com> Reviewed-by: Jan Beulich <jbeulich@suse.com> --- a/docs/misc/xen-command-line.markdown +++ b/docs/misc/xen-command-line.markdown @@ -699,17 +699,17 @@ Set the amount of memory for the initial greater than zero. This parameter is required. ### dom0\_mem (x86) -> `= List of ( min:<size> | max:<size> | <size> )` +> `= List of ( min:<sz> | max:<sz> | <sz> )` Set the amount of memory for the initial domain (dom0). If a size is positive, it represents an absolute value. If a size is negative, it is subtracted from the total available memory. -* `<size>` specifies the exact amount of memory. -* `min:<size>` specifies the minimum amount of memory. -* `max:<size>` specifies the maximum amount of memory. +* `<sz>` specifies the exact amount of memory. +* `min:<sz>` specifies the minimum amount of memory. +* `max:<sz>` specifies the maximum amount of memory. -If `<size>` is not specified, the default is all the available memory +If `<sz>` is not specified, the default is all the available memory minus some reserve. The reserve is 1/16 of the available memory or 128 MB (whichever is smaller). @@ -717,13 +717,20 @@ The amount of memory will be at least th the maximum (i.e., `max` overrides the `min` option). If there isn't enough memory then as much as possible is allocated. -`max:<size>` also sets the maximum reservation (the maximum amount of +`max:<sz>` also sets the maximum reservation (the maximum amount of memory dom0 can balloon up to). If this is omitted then the maximum reservation is unlimited. For example, to set dom0's initial memory allocation to 512MB but allow it to balloon up as far as 1GB use `dom0_mem=512M,max:1G` +> `<sz>` is: `<size> | [<size>+]<frac>%` +> `<frac>` is an integer < 100 + +* `<frac>` specifies a fraction of host memory size in percent. + +So `<sz>` being `1G+25%` on a 256 GB host would result in 65 GB. + If you use this option then it is highly recommended that you disable any dom0 autoballooning feature present in your toolstack. See the _xl.conf(5)_ man page or [Xen Best --- a/xen/arch/x86/dom0_build.c +++ b/xen/arch/x86/dom0_build.c @@ -19,17 +19,42 @@ #include <asm/p2m.h> #include <asm/setup.h> -static long __initdata dom0_nrpages; -static long __initdata dom0_min_nrpages; -static long __initdata dom0_max_nrpages = LONG_MAX; +struct memsize { + long nr_pages; + unsigned int percent; + bool minus; +}; + +static struct memsize __initdata dom0_size; +static struct memsize __initdata dom0_min_size; +static struct memsize __initdata dom0_max_size = { .nr_pages = LONG_MAX }; + +static bool __init memsize_gt_zero(const struct memsize *sz) +{ + return !sz->minus && sz->nr_pages; +} + +static unsigned long __init get_memsize(const struct memsize *sz, + unsigned long avail) +{ + unsigned long pages; + + pages = sz->nr_pages + sz->percent * avail / 100; + return sz->minus ? avail - pages : pages; +} /* * dom0_mem=[min:<min_amt>,][max:<max_amt>,][<amt>] - * + * * <min_amt>: The minimum amount of memory which should be allocated for dom0. * <max_amt>: The maximum amount of memory which should be allocated for dom0. * <amt>: The precise amount of memory to allocate for dom0. - * + * + * The format of <min_amt>, <max_amt> and <amt> is as follows: + * <size> | <frac>% | <size>+<frac>% + * <size> is a size value like 1G (1 GByte), <frac> is percentage of host + * memory (so 1G+10% means 10 percent of host memory + 1 GByte). + * * Notes: * 1. <amt> is clamped from below by <min_amt> and from above by available * memory and <max_amt> @@ -38,19 +63,59 @@ static long __initdata dom0_max_nrpages * 4. If <amt> is not specified, it is calculated as follows: * "All of memory is allocated to domain 0, minus 1/16th which is reserved * for uses such as DMA buffers (the reservation is clamped to 128MB)." - * + * * Each value can be specified as positive or negative: * If +ve: The specified amount is an absolute value. * If -ve: The specified amount is subtracted from total available memory. */ -static long __init parse_amt(const char *s, const char **ps) +static int __init parse_amt(const char *s, const char **ps, struct memsize *sz) { - long pages = parse_size_and_unit((*s == '-') ? s+1 : s, ps) >> PAGE_SHIFT; - return (*s == '-') ? -pages : pages; + unsigned long val; + struct memsize tmp = { }; + unsigned int items = 0; + + tmp.minus = (*s == '-'); + if ( tmp.minus ) + s++; + + do + { + if ( !isdigit(*s) ) + return -EINVAL; + + val = parse_size_and_unit(s, ps); + s = *ps; + if ( *s == '%' ) + { + if ( val >= 100 ) + return -EINVAL; + tmp.percent = val; + s++; + items++; /* No other item allowed. */ + } + else + { + /* <size> item must be first one. */ + if ( items ) + return -EINVAL; + tmp.nr_pages = val >> PAGE_SHIFT; + } + items++; + } while ( *s++ == '+' && items < 2 ); + + *ps = --s; + if ( *s && *s != ',' ) + return -EINVAL; + + *sz = tmp; + + return 0; } static int __init parse_dom0_mem(const char *s) { + int ret; + /* xen-shim uses shim_mem parameter instead of dom0_mem */ if ( pv_shim ) { @@ -60,14 +125,14 @@ static int __init parse_dom0_mem(const c do { if ( !strncmp(s, "min:", 4) ) - dom0_min_nrpages = parse_amt(s+4, &s); + ret = parse_amt(s + 4, &s, &dom0_min_size); else if ( !strncmp(s, "max:", 4) ) - dom0_max_nrpages = parse_amt(s+4, &s); + ret = parse_amt(s + 4, &s, &dom0_max_size); else - dom0_nrpages = parse_amt(s, &s); - } while ( *s++ == ',' ); + ret = parse_amt(s, &s, &dom0_size); + } while ( *s++ == ',' && !ret ); - return s[-1] ? -EINVAL : 0; + return s[-1] ? -EINVAL : ret; } custom_param("dom0_mem", parse_dom0_mem); @@ -304,9 +369,9 @@ unsigned long __init dom0_compute_nr_pag (!iommu_hap_pt_share || !paging_mode_hap(d)); for ( ; ; need_paging = false ) { - nr_pages = dom0_nrpages; - min_pages = dom0_min_nrpages; - max_pages = dom0_max_nrpages; + nr_pages = get_memsize(&dom0_size, avail); + min_pages = get_memsize(&dom0_min_size, avail); + max_pages = get_memsize(&dom0_max_size, avail); /* * If allocation isn't specified, reserve 1/16th of available memory @@ -314,14 +379,9 @@ unsigned long __init dom0_compute_nr_pag * maximum of 128MB. */ if ( !nr_pages ) - nr_pages = -(pv_shim ? pv_shim_mem(avail) + nr_pages = avail - (pv_shim ? pv_shim_mem(avail) : min(avail / 16, 128UL << (20 - PAGE_SHIFT))); - /* Negative specification means "all memory - specified amount". */ - if ( (long)nr_pages < 0 ) nr_pages += avail; - if ( (long)min_pages < 0 ) min_pages += avail; - if ( (long)max_pages < 0 ) max_pages += avail; - /* Clamp according to min/max limits and available memory. */ nr_pages = max(nr_pages, min_pages); nr_pages = min(nr_pages, max_pages); @@ -335,8 +395,8 @@ unsigned long __init dom0_compute_nr_pag } if ( is_pv_domain(d) && - (parms->p2m_base == UNSET_ADDR) && (dom0_nrpages <= 0) && - ((dom0_min_nrpages <= 0) || (nr_pages > min_pages)) ) + (parms->p2m_base == UNSET_ADDR) && !memsize_gt_zero(&dom0_size) && + (!memsize_gt_zero(&dom0_min_size) || (nr_pages > min_pages)) ) { /* * Legacy Linux kernels (i.e. such without a XEN_ELFNOTE_INIT_P2M @@ -362,7 +422,7 @@ unsigned long __init dom0_compute_nr_pag { end = sizeof_long >= sizeof(end) ? 0 : 1UL << (8 * sizeof_long); nr_pages = (end - vend) / (2 * sizeof_long); - if ( dom0_min_nrpages > 0 && nr_pages < min_pages ) + if ( memsize_gt_zero(&dom0_min_size) && nr_pages < min_pages ) nr_pages = min_pages; printk("Dom0 memory clipped to %lu pages\n", nr_pages); }
Locations
Projects
Search
Status Monitor
Help
OpenBuildService.org
Documentation
API Documentation
Code of Conduct
Contact
Support
@OBShq
Terms
openSUSE Build Service is sponsored by
The Open Build Service is an
openSUSE project
.
Sign Up
Log In
Places
Places
All Projects
Status Monitor
