lkml.org 
[lkml]   [2007]   [Oct]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [git pull] lguest: paravirt boot code


On Tue, 23 Oct 2007, Rusty Russell wrote:
>
> Well, with that out the way, and some scatterlist fixups, please pull from
>
> git://git.kernel.org/pub/scm/linux/kernel/git/rusty/linux-2.6-lguest.git

Ok. However, please include a proper diffstat. And by proper, I mean:

git diff -M --stat --summary

because right now your diffstat doesn't contain the final "total number of
lines added/removed" (which is the easiest way for me to see that there
isn't anything strange in there), and your diffstat doesn't do renames, so
renames look like delete/create pairs. For example, you had:

> arch/x86/lguest/boot.c | 1070 ++++++++++++++++++++++
> drivers/lguest/lguest.c | 1108 ----------------------

but in reality it was not at all thousands of lines changed, but instead a
rename with much smaller changes:

drivers/lguest/lguest.c => arch/x86/lguest/boot.c | 102 +-
which is a lot more readable and tells me much more about what is going
on.

And at the end, I really do want to see the details like this:

70 files changed, 4822 insertions(+), 4401 deletions(-)
create mode 100644 arch/x86/lguest/Kconfig
create mode 100644 arch/x86/lguest/Makefile
rename drivers/lguest/lguest.c => arch/x86/lguest/boot.c (93%)
rename drivers/lguest/lguest_asm.S => arch/x86/lguest/i386_head.S (73%)
delete mode 100644 drivers/block/lguest_blk.c
create mode 100644 drivers/block/virtio_blk.c
delete mode 100644 drivers/char/hvc_lguest.c
create mode 100644 drivers/char/virtio_console.c
delete mode 100644 drivers/lguest/io.c
delete mode 100644 drivers/lguest/lguest_bus.c
create mode 100644 drivers/lguest/lguest_device.c
create mode 100644 drivers/lguest/x86/core.c
rename drivers/lguest/{switcher.S => x86/switcher_32.S} (99%)
delete mode 100644 drivers/net/lguest_net.c
create mode 100644 drivers/net/virtio_net.c
create mode 100644 drivers/virtio/Kconfig
create mode 100644 drivers/virtio/Makefile
create mode 100644 drivers/virtio/config.c
create mode 100644 drivers/virtio/virtio.c
create mode 100644 drivers/virtio/virtio_ring.c
create mode 100644 include/asm-x86/lguest.h
create mode 100644 include/asm-x86/lguest_hcall.h
delete mode 100644 include/linux/lguest_bus.h
create mode 100644 include/linux/virtio.h
create mode 100644 include/linux/virtio_blk.h
create mode 100644 include/linux/virtio_config.h
create mode 100644 include/linux/virtio_console.h
create mode 100644 include/linux/virtio_net.h
create mode 100644 include/linux/virtio_ring.h
which again gives a nice overview of what the heck is actually going on
here. All of which you get for free if you use the above git command line.

Anyway, pulled. But just for future reference,

Linus
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2007-10-23 18:13    [from the cache]
©2003-2011 Jasper Spaans