From: Jerone Young <jyoung5-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org>
To: Avi Kivity <avi-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
Cc: kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org,
kvm-ppc-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org,
Hollis Blanchard
<hollisb-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org>
Subject: Re: [kvm-ppc-devel] [PATCH 01 of 19] Move kvm_context to kvmctl.h
Date: Mon, 05 Nov 2007 14:24:17 -0600 [thread overview]
Message-ID: <1194294257.9355.4.camel@thinkpad> (raw)
In-Reply-To: <472D72D8.5010704-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
On Sun, 2007-11-04 at 09:20 +0200, Avi Kivity wrote:
> Jerone Young wrote:
> > I'll wait awhile before sending another super patch bomb to the list.
> > Perhaps I can add these changes afterward if need be.
> >
> >
>
> Please address the comments. If there is to be churn, I prefer it to be
> out-of-tree. Please verify that the patch set is bisectable.
Agreed.
>
> Note that it is too finely granular in my opinion. I'll accept it in
> the current form however, I don't want churn for churn's sake.
Well the reason for it being fine grained was to see which moves people
who disagree with. It kind of has pointed that out :-). But yes having
20+ patches churning on the list is a pain for everyone.
>
-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems? Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/
next prev parent reply other threads:[~2007-11-05 20:24 UTC|newest]
Thread overview: 37+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-11-02 18:24 [PATCH 00 of 19] [v3] Refactor libkvm Jerone Young
2007-11-02 18:24 ` [PATCH 01 of 19] Move kvm_context to kvmctl.h Jerone Young
2007-11-02 19:34 ` [kvm-ppc-devel] " Hollis Blanchard
2007-11-02 19:48 ` Jerone Young
2007-11-04 7:20 ` Avi Kivity
[not found] ` <472D72D8.5010704-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2007-11-05 20:24 ` Jerone Young [this message]
2007-11-04 7:22 ` Avi Kivity
2007-11-02 18:24 ` [PATCH 02 of 19] Make static slot & kvm_memory region funcions public Jerone Young
2007-11-02 19:36 ` [kvm-ppc-devel] " Hollis Blanchard
2007-11-02 18:24 ` [PATCH 03 of 19] Move fuction kvm_alloc_kernel_memory to libkvm-x86.c Jerone Young
2007-11-02 18:24 ` [PATCH 04 of 19] Move kvm_alloc_userspace_memory " Jerone Young
2007-11-02 18:24 ` [PATCH 05 of 19] Modify out arch specific code from kvm_create function Jerone Young
2007-11-02 18:24 ` [PATCH 06 of 19] Move kvm_create_kernel_phys_mem to libkvm-x86.c Jerone Young
2007-11-02 18:24 ` [PATCH 07 of 19] Move kvm_create_phys_mem " Jerone Young
2007-11-04 7:29 ` Avi Kivity
[not found] ` <472D74CB.8050602-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2007-11-05 21:15 ` Jerone Young
2007-11-06 1:06 ` [PATCH 07 of 19] Movekvm_create_phys_mem " Zhang, Xiantao
2007-11-06 9:10 ` [PATCH 07 of 19] Move kvm_create_phys_mem " Avi Kivity
2007-11-02 18:24 ` [PATCH 08 of 19] Move kvm_destroy_phys_mem " Jerone Young
2007-11-04 7:30 ` Avi Kivity
2007-11-02 18:24 ` [PATCH 09 of 19] Move kvm_create_memory_alias & kvm_destroy_memory_alias " Jerone Young
2007-11-02 18:24 ` [PATCH 10 of 19] Move kvm_get & kmv_set_lapci functions " Jerone Young
2007-11-02 18:25 ` [PATCH 11 of 19] Make functions in libkvm.c nonstatic Jerone Young
2007-11-04 7:32 ` Avi Kivity
2007-11-02 18:25 ` [PATCH 12 of 19] Move abi 10 functions to libkvm-x86.c Jerone Young
2007-11-02 18:25 ` [PATCH 13 of 19] Move msrs " Jerone Young
2007-11-02 18:25 ` [PATCH 14 of 19] Move print_seg & Move kvm_show_regs to kvmctl-x86.c Jerone Young
2007-11-02 18:25 ` [PATCH 15 of 19] Declare kvm_abi as a global variable in libkvm.h Jerone Young
2007-11-04 7:34 ` Avi Kivity
[not found] ` <472D75E9.6010409-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2007-11-05 22:10 ` Jerone Young
2007-11-06 9:11 ` Avi Kivity
2007-11-02 18:25 ` [PATCH 16 of 19] Move kvm_get_apic to libkvm-x86.c Jerone Young
2007-11-02 18:25 ` [PATCH 17 of 19] Move cr8 functions " Jerone Young
2007-11-02 19:46 ` [kvm-ppc-devel] " Hollis Blanchard
2007-11-02 18:25 ` [PATCH 18 of 19] Move kvm_setup_cpuid " Jerone Young
2007-11-02 19:47 ` [kvm-ppc-devel] " Hollis Blanchard
2007-11-02 18:25 ` [PATCH 19 of 19] Remove unsued inclusion of linux/kvm_parah.h in userspace libkvm.h Jerone Young
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1194294257.9355.4.camel@thinkpad \
--to=jyoung5-r/jw6+rmf7hqt0dzr+alfa@public.gmane.org \
--cc=avi-atKUWr5tajBWk0Htik3J/w@public.gmane.org \
--cc=hollisb-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org \
--cc=kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org \
--cc=kvm-ppc-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox