Andrew Morton writes:
> On Wed, 10 Oct 2012 16:34:37 -0700
> Jeremy Fitzhardinge wrote:
>
>> On 10/09/2012 06:14 PM, Andrew Morton wrote:
>> > On Wed, 10 Oct 2012 00:09:12 + KY Srinivasan
>> > wrote:
>> >
>> + if (!pg) {
>> + *alloc_error =
Andrew Morton a...@linux-foundation.org writes:
On Wed, 10 Oct 2012 16:34:37 -0700
Jeremy Fitzhardinge jer...@goop.org wrote:
On 10/09/2012 06:14 PM, Andrew Morton wrote:
On Wed, 10 Oct 2012 00:09:12 + KY Srinivasan k...@microsoft.com
wrote:
+ if (!pg) {
+
On Wed, 10 Oct 2012 16:34:37 -0700
Jeremy Fitzhardinge wrote:
> On 10/09/2012 06:14 PM, Andrew Morton wrote:
> > On Wed, 10 Oct 2012 00:09:12 + KY Srinivasan wrote:
> >
> +if (!pg) {
> +*alloc_error = true;
> +
On 10/09/2012 06:14 PM, Andrew Morton wrote:
> On Wed, 10 Oct 2012 00:09:12 + KY Srinivasan wrote:
>
+ if (!pg) {
+ *alloc_error = true;
+ return i * alloc_unit;
+ }
+
+ totalram_pages -= alloc_unit;
On 10/09/2012 09:44 PM, Andrew Morton wrote:
> On Sun, 7 Oct 2012 16:59:46 -0700
> "K. Y. Srinivasan" wrote:
>
>> Add the basic balloon driver.
>
> hm, how many balloon drivers does one kernel need?
>
> Although I see that the great majority of this code is hypervisor-specific.
Much like
On 10/09/2012 09:44 PM, Andrew Morton wrote:
On Sun, 7 Oct 2012 16:59:46 -0700
K. Y. Srinivasan k...@microsoft.com wrote:
Add the basic balloon driver.
hm, how many balloon drivers does one kernel need?
Although I see that the great majority of this code is hypervisor-specific.
Much
On 10/09/2012 06:14 PM, Andrew Morton wrote:
On Wed, 10 Oct 2012 00:09:12 + KY Srinivasan k...@microsoft.com wrote:
+ if (!pg) {
+ *alloc_error = true;
+ return i * alloc_unit;
+ }
+
+ totalram_pages -= alloc_unit;
Well, I'd
On Wed, 10 Oct 2012 16:34:37 -0700
Jeremy Fitzhardinge jer...@goop.org wrote:
On 10/09/2012 06:14 PM, Andrew Morton wrote:
On Wed, 10 Oct 2012 00:09:12 + KY Srinivasan k...@microsoft.com wrote:
+if (!pg) {
+*alloc_error = true;
+
a...@canonical.com;
> a...@firstfloor.org
> Subject: Re: [PATCH 2/2] Drivers: hv: Add Hyper-V balloon driver
>
> On Wed, 10 Oct 2012 00:09:12 + KY Srinivasan wrote:
>
> > > > + if (!pg) {
> > > > + *alloc_error = tr
On Wed, 10 Oct 2012 00:09:12 + KY Srinivasan wrote:
> > > + if (!pg) {
> > > + *alloc_error = true;
> > > + return i * alloc_unit;
> > > + }
> > > +
> > > + totalram_pages -= alloc_unit;
> >
> > Well, I'd consider totalram_pages to be
a...@canonical.com;
> a...@firstfloor.org
> Subject: Re: [PATCH 2/2] Drivers: hv: Add Hyper-V balloon driver
>
> On Sun, 7 Oct 2012 16:59:46 -0700
> "K. Y. Srinivasan" wrote:
>
> > Add the basic balloon driver.
>
> hm, how many balloon drivers does one kernel need
On Sun, 7 Oct 2012 16:59:46 -0700
"K. Y. Srinivasan" wrote:
> Add the basic balloon driver.
hm, how many balloon drivers does one kernel need?
Although I see that the great majority of this code is hypervisor-specific.
> Windows hosts dynamically manage the guest
> memory allocation via a
On Sun, 7 Oct 2012 16:59:46 -0700
K. Y. Srinivasan k...@microsoft.com wrote:
Add the basic balloon driver.
hm, how many balloon drivers does one kernel need?
Although I see that the great majority of this code is hypervisor-specific.
Windows hosts dynamically manage the guest
memory
Subject: Re: [PATCH 2/2] Drivers: hv: Add Hyper-V balloon driver
On Sun, 7 Oct 2012 16:59:46 -0700
K. Y. Srinivasan k...@microsoft.com wrote:
Add the basic balloon driver.
hm, how many balloon drivers does one kernel need?
Although I see that the great majority of this code
On Wed, 10 Oct 2012 00:09:12 + KY Srinivasan k...@microsoft.com wrote:
+ if (!pg) {
+ *alloc_error = true;
+ return i * alloc_unit;
+ }
+
+ totalram_pages -= alloc_unit;
Well, I'd consider totalram_pages to be an
Subject: Re: [PATCH 2/2] Drivers: hv: Add Hyper-V balloon driver
On Wed, 10 Oct 2012 00:09:12 + KY Srinivasan k...@microsoft.com wrote:
+ if (!pg) {
+ *alloc_error = true;
+ return i * alloc_unit
nux-foundation.org; a...@firstfloor.org
> Cc: KY Srinivasan
> Subject: Re: [PATCH 2/2] Drivers: hv: Add Hyper-V balloon driver
>
> "K. Y. Srinivasan" writes:
> > +static int hot_add;
> > +
> > +module_param(hot_add, int, S_IRUGO);
> > +MODULE_PARM_DESC(hot
; a...@firstfloor.org
Cc: KY Srinivasan
Subject: Re: [PATCH 2/2] Drivers: hv: Add Hyper-V balloon driver
K. Y. Srinivasan k...@microsoft.com writes:
+static int hot_add;
+
+module_param(hot_add, int, S_IRUGO);
+MODULE_PARM_DESC(hot_add, If set attempt memory hot_add);
I think this should
"K. Y. Srinivasan" writes:
> +static int hot_add;
> +
> +module_param(hot_add, int, S_IRUGO);
> +MODULE_PARM_DESC(hot_add, "If set attempt memory hot_add");
I think this should be a 'bool', but I can't tell, since it's not used
in this patch.
Cheers,
Rusty.
--
To unsubscribe from this list:
; a...@firstfloor.org
> Subject: Re: [PATCH 2/2] Drivers: hv: Add Hyper-V balloon driver
>
> On Sun, Oct 07, 2012 at 04:59:46PM -0700, K. Y. Srinivasan wrote:
> > +config HYPERV_BALLOON
> > + tristate "Microsoft Hyper-V Balloon driver"
> > + depends on
On Sun, Oct 07, 2012 at 04:59:46PM -0700, K. Y. Srinivasan wrote:
> +config HYPERV_BALLOON
> + tristate "Microsoft Hyper-V Balloon driver"
> + depends on HYPERV
> + help
> + Select this option to enable the Hyper-V Utilities.
Your help text is wrong :(
> --- /dev/null
> +++
On Sun, Oct 07, 2012 at 04:59:46PM -0700, K. Y. Srinivasan wrote:
+config HYPERV_BALLOON
+ tristate Microsoft Hyper-V Balloon driver
+ depends on HYPERV
+ help
+ Select this option to enable the Hyper-V Utilities.
Your help text is wrong :(
--- /dev/null
+++
Subject: Re: [PATCH 2/2] Drivers: hv: Add Hyper-V balloon driver
On Sun, Oct 07, 2012 at 04:59:46PM -0700, K. Y. Srinivasan wrote:
+config HYPERV_BALLOON
+ tristate Microsoft Hyper-V Balloon driver
+ depends on HYPERV
+ help
+ Select this option to enable the Hyper-V Utilities
K. Y. Srinivasan k...@microsoft.com writes:
+static int hot_add;
+
+module_param(hot_add, int, S_IRUGO);
+MODULE_PARM_DESC(hot_add, If set attempt memory hot_add);
I think this should be a 'bool', but I can't tell, since it's not used
in this patch.
Cheers,
Rusty.
--
To unsubscribe from this
24 matches
Mail list logo
