On Tue, Jun 19, 2007 at 10:31:02AM -0400, Mark Johnson wrote: > >> On 6/15/07, Daniel Veillard <veillard@xxxxxxxxxx> wrote: > >> Is the suggestion to change xend_internal/xen_internal to not set rev > >> and remove the display of the hypervisor rev in virsh? Or is it > >something > >> else? > > > > Well ideally I would prefer xend to not send something crazy for > > 'release' > >but since that's unlikely, and since the hypervisor only provide > >major/minor > >anyway, let's just drop the release in src/xend_internal.c , I'm suggesting > >the enclosed patch, > > Ah, I see you applied the xend patch... Not sure if you wanted/didn't > want the virsh version patch too? > > In case you did, I've attached a slightly updated patch to not print the > rel number in virsh version. No, want to keep the 3 digit numbers in virsh. Even though Xen can only give us 2 meaningful digits, the QEMU driver does have 3. Regards, Dan. -- |=- Red Hat, Engineering, Emerging Technologies, Boston. +1 978 392 2496 -=| |=- Perl modules: http://search.cpan.org/~danberr/ -=| |=- Projects: http://freshmeat.net/~danielpb/ -=| |=- GnuPG: 7D3B9505 F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 -=|