Re: [PATCH 03/12] Adds xml entries for memory tunables

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 21 Sep 2010 18:14:51 +0100, "Daniel P. Berrange" <berrange@xxxxxxxxxx> wrote:
> On Mon, Sep 20, 2010 at 02:15:37PM +0530, Nikunj A. Dadhania wrote:
> > The patch adds xml entries to the domain.rng file.
> > 
> > +	  <!-- Minimum amount of memory required to start the VM -->
> > +	  <optional>
> > +	    <element name="min_gaurantee">
> 
> Small typo there,  s/gaur/guar/
Sure

> > +	      <ref name="memoryKB"/>
> > +	    </element>
> > +	  </optional>
> > +	  <!-- Maximum swap area the VM can use -->
> > +	  <optional>
> > +	    <element name="swap_hard_limit">
> > +	      <ref name="memoryKB"/>
> > +	    </element>
> > +	  </optional>
> > +	</element>
> > +      </optional>
> > +
> >        <optional>
> 
> Also the indentation looks a bit wacky. Tabs in the file perhaps ?
>
I am not sure why! I had used emacs and the indentation was automatic.
 
> >          <element name="vcpu">
> >            <optional>
> 
> The schema additions look fine to me though.
Cool.

Nikunj

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]