All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/4] remus: remove old remus script
@ 2014-06-25  7:00 Yang Hongyang
  2014-06-25  7:00 ` [PATCH 2/4] remus: move remus README to docs directory Yang Hongyang
                   ` (4 more replies)
  0 siblings, 5 replies; 16+ messages in thread
From: Yang Hongyang @ 2014-06-25  7:00 UTC (permalink / raw)
  To: xen-devel; +Cc: rshriram, Yang Hongyang, ian.jackson

Since xend already deleted, the old remus script which based on
xend no longer functional, remove it.

Signed-off-by: Yang Hongyang <yanghy@cn.fujitsu.com>
---
 tools/remus/Makefile |  15 ----
 tools/remus/remus    | 230 ---------------------------------------------------
 2 files changed, 245 deletions(-)
 delete mode 100644 tools/remus/Makefile
 delete mode 100644 tools/remus/remus

diff --git a/tools/remus/Makefile b/tools/remus/Makefile
deleted file mode 100644
index ae82376..0000000
--- a/tools/remus/Makefile
+++ /dev/null
@@ -1,15 +0,0 @@
-XEN_ROOT=$(CURDIR)/../..
-include $(XEN_ROOT)/tools/Rules.mk
-
-SCRIPTS = remus
-
-.PHONY: all
-all: subdirs-all
-
-.PHONY: install
-install: subdirs-install
-	$(INSTALL_DIR) $(DESTDIR)$(BINDIR)
-	$(INSTALL_PYTHON_PROG) $(SCRIPTS) $(DESTDIR)$(BINDIR)
-
-.PHONY: clean
-clean: subdirs-clean
diff --git a/tools/remus/remus b/tools/remus/remus
deleted file mode 100644
index 38f0365..0000000
--- a/tools/remus/remus
+++ /dev/null
@@ -1,230 +0,0 @@
-#!/usr/bin/env python
-#
-# This is a save process which also buffers outgoing I/O between
-# rounds, so that external viewers never see anything that hasn't
-# been committed at the backup
-#
-# TODO: fencing.
-
-import optparse, os, re, select, signal, sys, time
-
-from xen.remus import save, util, vm
-from xen.remus.device import ReplicatedDisk, ReplicatedDiskException
-from xen.remus.device import BufferedNIC, BufferedNICException
-from xen.xend import XendOptions
-
-class CfgException(Exception): pass
-
-class Cfg(object):
-
-    REMUS_FLAGS_COMPRESSION = 1
-
-    def __init__(self):
-        # must be set
-        self.domid = 0
-
-        self.host = 'localhost'
-        self.nullremus = False
-        self.port = XendOptions.instance().get_xend_relocation_port()
-        self.interval = 200
-        self.netbuffer = True
-        self.flags = self.REMUS_FLAGS_COMPRESSION
-        self.timer = False
-
-        parser = optparse.OptionParser()
-        parser.usage = '%prog [options] domain [destination]'
-        parser.add_option('-i', '--interval', dest='interval', type='int',
-                          metavar='MS',
-                          help='checkpoint every MS milliseconds')
-        parser.add_option('-p', '--port', dest='port', type='int',
-                          help='send stream to port PORT', metavar='PORT')
-        parser.add_option('', '--blackhole', dest='nullremus', action='store_true',
-                          help='replicate to /dev/null (no disk checkpoints, only memory & net buffering)')
-        parser.add_option('', '--no-net', dest='nonet', action='store_true',
-                          help='run without net buffering (benchmark option)')
-        parser.add_option('', '--no-compression', dest='nocompress', action='store_true',
-                          help='run without checkpoint compression')
-        parser.add_option('', '--timer', dest='timer', action='store_true',
-                          help='force pause at checkpoint interval (experimental)')
-        self.parser = parser
-
-    def usage(self):
-        self.parser.print_help()
-
-    def getargs(self):
-        opts, args = self.parser.parse_args()
-
-        if opts.interval:
-            self.interval = opts.interval
-        if opts.port:
-            self.port = opts.port
-        if opts.nullremus:
-            self.nullremus = True
-        if opts.nonet:
-            self.netbuffer = False
-        if opts.nocompress:
-            self.flags &= ~self.REMUS_FLAGS_COMPRESSION
-        if opts.timer:
-            self.timer = True
-
-        if not args:
-            raise CfgException('Missing domain')
-        self.domid = args[0]
-        if (len(args) > 1):
-            self.host = args[1]
-
-class SignalException(Exception): pass
-
-def run(cfg):
-    closure = lambda: None
-    closure.cmd = None
-
-    def sigexception(signo, frame):
-        raise SignalException(signo)
-
-    def die():
-        # I am not sure what the best way to die is. xm destroy is another option,
-        # or we could attempt to trigger some instant reboot.
-        print "dying..."
-        print util.runcmd(['sudo', 'ifdown', 'eth2'])
-        # dangling imq0 handle on vif locks up the system
-        for buf in bufs:
-            buf.uninstall()
-        print util.runcmd(['sudo', 'xm', 'destroy', cfg.domid])
-        print util.runcmd(['sudo', 'ifup', 'eth2'])
-
-    def getcommand():
-        """Get a command to execute while running.
-        Commands include:
-          s: die prior to postsuspend hook
-          s2: die after postsuspend hook
-          r: die prior to preresume hook
-          r2: die after preresume hook
-          c: die prior to commit hook
-          c2: die after commit hook
-          """
-        r, w, x = select.select([sys.stdin], [], [], 0)
-        if sys.stdin not in r:
-            return
-
-        cmd = sys.stdin.readline().strip()
-        if cmd not in ('s', 's2', 'r', 'r2', 'c', 'c2'):
-            print "unknown command: %s" % cmd
-        closure.cmd = cmd
-
-    signal.signal(signal.SIGTERM, sigexception)
-
-    dom = vm.VM(cfg.domid)
-
-    # set up I/O buffers
-    bufs = []
-
-    # disks must commit before network can be released
-    if not cfg.nullremus:
-        for disk in dom.disks:
-            try:
-                bufs.append(ReplicatedDisk(disk))
-            except ReplicatedDiskException, e:
-                print e
-                continue
-
-    if cfg.netbuffer:
-        for vif in dom.vifs:
-            bufs.append(BufferedNIC(vif))
-
-    if cfg.nullremus:
-        fd = save.NullSocket((cfg.host, cfg.port))
-    else:
-        fd = save.MigrationSocket((cfg.host, cfg.port))
-
-    def postsuspend():
-        'Begin external checkpointing after domain has paused'
-        if not cfg.timer:
-            # when not using a timer thread, sleep until now + interval
-            closure.starttime = time.time()
-
-        if closure.cmd == 's':
-            die()
-
-        for buf in bufs:
-            buf.postsuspend()
-
-        if closure.cmd == 's2':
-            die()
-
-    def preresume():
-        'Complete external checkpointing before domain resumes'
-        if closure.cmd == 'r':
-            die()
-
-        for buf in bufs:
-            buf.preresume()
-
-        if closure.cmd == 'r2':
-            die()
-
-    def commit():
-        'commit network buffer'
-        if closure.cmd == 'c':
-            die()
-
-        print >> sys.stderr, "PROF: flushed memory at %0.6f" % (time.time())
-
-        for buf in bufs:
-            buf.commit()
-
-        if closure.cmd == 'c2':
-            die()
-
-        # Since the domain is running at this point, it's a good time to
-        # check for control channel commands
-        getcommand()
-
-        if not cfg.timer:
-            endtime = time.time()
-            elapsed = (endtime - closure.starttime) * 1000
-
-            if elapsed < cfg.interval:
-                time.sleep((cfg.interval - elapsed) / 1000.0)
-
-        # False ends checkpointing
-        return True
-
-    if cfg.timer:
-        interval = cfg.interval
-    else:
-        interval = 0
-
-    rc = 0
-
-    checkpointer = save.Saver(cfg.domid, fd, postsuspend, preresume, commit,
-                              interval, cfg.flags)
-
-    try:
-        checkpointer.start()
-    except save.CheckpointError, e:
-        print e
-        rc = 1
-    except KeyboardInterrupt:
-        pass
-    except SignalException:
-        print '*** signalled ***'
-
-    for buf in bufs:
-        buf.uninstall()
-
-    sys.exit(rc)
-
-cfg = Cfg()
-try:
-    cfg.getargs()
-except CfgException, inst:
-    print str(inst)
-    cfg.usage()
-    sys.exit(1)
-
-try:
-    run(cfg)
-except vm.VMException, inst:
-    print str(inst)
-    sys.exit(1)
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 16+ messages in thread

* [PATCH 2/4] remus: move remus README to docs directory
  2014-06-25  7:00 [PATCH 1/4] remus: remove old remus script Yang Hongyang
@ 2014-06-25  7:00 ` Yang Hongyang
  2014-06-25  7:19   ` Shriram Rajagopalan
  2014-06-30 17:05   ` Ian Jackson
  2014-06-25  7:00 ` [PATCH 3/4] remus: add wikipage link to remus README Yang Hongyang
                   ` (3 subsequent siblings)
  4 siblings, 2 replies; 16+ messages in thread
From: Yang Hongyang @ 2014-06-25  7:00 UTC (permalink / raw)
  To: xen-devel; +Cc: rshriram, Yang Hongyang, ian.jackson

We do not need a separate directory to store remus README, just
move it to docs/ directory.

Signed-off-by: Yang Hongyang <yanghy@cn.fujitsu.com>
---
 docs/README.remus  | 4 ++++
 tools/remus/README | 4 ----
 2 files changed, 4 insertions(+), 4 deletions(-)
 create mode 100644 docs/README.remus
 delete mode 100644 tools/remus/README

diff --git a/docs/README.remus b/docs/README.remus
new file mode 100644
index 0000000..9e8140b
--- /dev/null
+++ b/docs/README.remus
@@ -0,0 +1,4 @@
+Remus provides fault tolerance for virtual machines by sending continuous
+checkpoints to a backup, which will activate if the target VM fails.
+
+See the website at http://nss.cs.ubc.ca/remus/ for details.
diff --git a/tools/remus/README b/tools/remus/README
deleted file mode 100644
index 9e8140b..0000000
--- a/tools/remus/README
+++ /dev/null
@@ -1,4 +0,0 @@
-Remus provides fault tolerance for virtual machines by sending continuous
-checkpoints to a backup, which will activate if the target VM fails.
-
-See the website at http://nss.cs.ubc.ca/remus/ for details.
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 16+ messages in thread

* [PATCH 3/4] remus: add wikipage link to remus README
  2014-06-25  7:00 [PATCH 1/4] remus: remove old remus script Yang Hongyang
  2014-06-25  7:00 ` [PATCH 2/4] remus: move remus README to docs directory Yang Hongyang
@ 2014-06-25  7:00 ` Yang Hongyang
  2014-06-25  7:14   ` Shriram Rajagopalan
  2014-06-30 17:05   ` Ian Jackson
  2014-06-25  7:00 ` [PATCH 4/4] MAINTAINERS: Update maintained files of REMUS part Yang Hongyang
                   ` (2 subsequent siblings)
  4 siblings, 2 replies; 16+ messages in thread
From: Yang Hongyang @ 2014-06-25  7:00 UTC (permalink / raw)
  To: xen-devel; +Cc: rshriram, Yang Hongyang, ian.jackson

add wikipage link to remus README

Signed-off-by: Yang Hongyang <yanghy@cn.fujitsu.com>
---
 docs/README.remus | 1 +
 1 file changed, 1 insertion(+)

diff --git a/docs/README.remus b/docs/README.remus
index 9e8140b..cb1bea0 100644
--- a/docs/README.remus
+++ b/docs/README.remus
@@ -2,3 +2,4 @@ Remus provides fault tolerance for virtual machines by sending continuous
 checkpoints to a backup, which will activate if the target VM fails.
 
 See the website at http://nss.cs.ubc.ca/remus/ for details.
+See also http://wiki.xen.org/wiki/Remus.
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 16+ messages in thread

* [PATCH 4/4] MAINTAINERS: Update maintained files of REMUS part
  2014-06-25  7:00 [PATCH 1/4] remus: remove old remus script Yang Hongyang
  2014-06-25  7:00 ` [PATCH 2/4] remus: move remus README to docs directory Yang Hongyang
  2014-06-25  7:00 ` [PATCH 3/4] remus: add wikipage link to remus README Yang Hongyang
@ 2014-06-25  7:00 ` Yang Hongyang
  2014-06-30 17:06   ` Ian Jackson
  2014-06-25  7:12 ` [PATCH 1/4] remus: remove old remus script Shriram Rajagopalan
  2014-06-30 17:05 ` Ian Jackson
  4 siblings, 1 reply; 16+ messages in thread
From: Yang Hongyang @ 2014-06-25  7:00 UTC (permalink / raw)
  To: xen-devel; +Cc: rshriram, Yang Hongyang, ian.jackson

delete tools/remus and add docs/README.remus

Singed-off-by: Yang Hongyang <yanghy@cn.fujitsu.com>
---
 MAINTAINERS | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 1865a84..ae5d7b3 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -258,7 +258,7 @@ T:	git git://xenbits.xen.org/qemu-upstream-*.git
 REMUS
 M:	Shriram Rajagopalan <rshriram@cs.ubc.ca>
 S:	Maintained
-F:	tools/remus/
+F:	docs/README.remus
 F:	tools/blktap2/drivers/block-remus.c
 F:	tools/blktap2/drivers/hashtable*
 
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 16+ messages in thread

* Re: [PATCH 1/4] remus: remove old remus script
  2014-06-25  7:00 [PATCH 1/4] remus: remove old remus script Yang Hongyang
                   ` (2 preceding siblings ...)
  2014-06-25  7:00 ` [PATCH 4/4] MAINTAINERS: Update maintained files of REMUS part Yang Hongyang
@ 2014-06-25  7:12 ` Shriram Rajagopalan
  2014-06-30 17:05 ` Ian Jackson
  4 siblings, 0 replies; 16+ messages in thread
From: Shriram Rajagopalan @ 2014-06-25  7:12 UTC (permalink / raw)
  To: FNST-Yang Hongyang, xen-devel


[-- Attachment #1.1: Type: text/plain, Size: 8519 bytes --]

On Jun 25, 2014 12:34 PM, "Yang Hongyang" <yanghy@cn.fujitsu.com> wrote:
>
> Since xend already deleted, the old remus script which based on
> xend no longer functional, remove it.
>
> Signed-off-by: Yang Hongyang <yanghy@cn.fujitsu.com>
> ---
>  tools/remus/Makefile |  15 ----
>  tools/remus/remus    | 230
---------------------------------------------------
>  2 files changed, 245 deletions(-)
>  delete mode 100644 tools/remus/Makefile
>  delete mode 100644 tools/remus/remus
>
> diff --git a/tools/remus/Makefile b/tools/remus/Makefile
> deleted file mode 100644
> index ae82376..0000000
> --- a/tools/remus/Makefile
> +++ /dev/null
> @@ -1,15 +0,0 @@
> -XEN_ROOT=$(CURDIR)/../..
> -include $(XEN_ROOT)/tools/Rules.mk
> -
> -SCRIPTS = remus
> -
> -.PHONY: all
> -all: subdirs-all
> -
> -.PHONY: install
> -install: subdirs-install
> -       $(INSTALL_DIR) $(DESTDIR)$(BINDIR)
> -       $(INSTALL_PYTHON_PROG) $(SCRIPTS) $(DESTDIR)$(BINDIR)
> -
> -.PHONY: clean
> -clean: subdirs-clean
> diff --git a/tools/remus/remus b/tools/remus/remus
> deleted file mode 100644
> index 38f0365..0000000
> --- a/tools/remus/remus
> +++ /dev/null
> @@ -1,230 +0,0 @@
> -#!/usr/bin/env python
> -#
> -# This is a save process which also buffers outgoing I/O between
> -# rounds, so that external viewers never see anything that hasn't
> -# been committed at the backup
> -#
> -# TODO: fencing.
> -
> -import optparse, os, re, select, signal, sys, time
> -
> -from xen.remus import save, util, vm
> -from xen.remus.device import ReplicatedDisk, ReplicatedDiskException
> -from xen.remus.device import BufferedNIC, BufferedNICException
> -from xen.xend import XendOptions
> -
> -class CfgException(Exception): pass
> -
> -class Cfg(object):
> -
> -    REMUS_FLAGS_COMPRESSION = 1
> -
> -    def __init__(self):
> -        # must be set
> -        self.domid = 0
> -
> -        self.host = 'localhost'
> -        self.nullremus = False
> -        self.port = XendOptions.instance().get_xend_relocation_port()
> -        self.interval = 200
> -        self.netbuffer = True
> -        self.flags = self.REMUS_FLAGS_COMPRESSION
> -        self.timer = False
> -
> -        parser = optparse.OptionParser()
> -        parser.usage = '%prog [options] domain [destination]'
> -        parser.add_option('-i', '--interval', dest='interval',
type='int',
> -                          metavar='MS',
> -                          help='checkpoint every MS milliseconds')
> -        parser.add_option('-p', '--port', dest='port', type='int',
> -                          help='send stream to port PORT',
metavar='PORT')
> -        parser.add_option('', '--blackhole', dest='nullremus',
action='store_true',
> -                          help='replicate to /dev/null (no disk
checkpoints, only memory & net buffering)')
> -        parser.add_option('', '--no-net', dest='nonet',
action='store_true',
> -                          help='run without net buffering (benchmark
option)')
> -        parser.add_option('', '--no-compression', dest='nocompress',
action='store_true',
> -                          help='run without checkpoint compression')
> -        parser.add_option('', '--timer', dest='timer',
action='store_true',
> -                          help='force pause at checkpoint interval
(experimental)')
> -        self.parser = parser
> -
> -    def usage(self):
> -        self.parser.print_help()
> -
> -    def getargs(self):
> -        opts, args = self.parser.parse_args()
> -
> -        if opts.interval:
> -            self.interval = opts.interval
> -        if opts.port:
> -            self.port = opts.port
> -        if opts.nullremus:
> -            self.nullremus = True
> -        if opts.nonet:
> -            self.netbuffer = False
> -        if opts.nocompress:
> -            self.flags &= ~self.REMUS_FLAGS_COMPRESSION
> -        if opts.timer:
> -            self.timer = True
> -
> -        if not args:
> -            raise CfgException('Missing domain')
> -        self.domid = args[0]
> -        if (len(args) > 1):
> -            self.host = args[1]
> -
> -class SignalException(Exception): pass
> -
> -def run(cfg):
> -    closure = lambda: None
> -    closure.cmd = None
> -
> -    def sigexception(signo, frame):
> -        raise SignalException(signo)
> -
> -    def die():
> -        # I am not sure what the best way to die is. xm destroy is
another option,
> -        # or we could attempt to trigger some instant reboot.
> -        print "dying..."
> -        print util.runcmd(['sudo', 'ifdown', 'eth2'])
> -        # dangling imq0 handle on vif locks up the system
> -        for buf in bufs:
> -            buf.uninstall()
> -        print util.runcmd(['sudo', 'xm', 'destroy', cfg.domid])
> -        print util.runcmd(['sudo', 'ifup', 'eth2'])
> -
> -    def getcommand():
> -        """Get a command to execute while running.
> -        Commands include:
> -          s: die prior to postsuspend hook
> -          s2: die after postsuspend hook
> -          r: die prior to preresume hook
> -          r2: die after preresume hook
> -          c: die prior to commit hook
> -          c2: die after commit hook
> -          """
> -        r, w, x = select.select([sys.stdin], [], [], 0)
> -        if sys.stdin not in r:
> -            return
> -
> -        cmd = sys.stdin.readline().strip()
> -        if cmd not in ('s', 's2', 'r', 'r2', 'c', 'c2'):
> -            print "unknown command: %s" % cmd
> -        closure.cmd = cmd
> -
> -    signal.signal(signal.SIGTERM, sigexception)
> -
> -    dom = vm.VM(cfg.domid)
> -
> -    # set up I/O buffers
> -    bufs = []
> -
> -    # disks must commit before network can be released
> -    if not cfg.nullremus:
> -        for disk in dom.disks:
> -            try:
> -                bufs.append(ReplicatedDisk(disk))
> -            except ReplicatedDiskException, e:
> -                print e
> -                continue
> -
> -    if cfg.netbuffer:
> -        for vif in dom.vifs:
> -            bufs.append(BufferedNIC(vif))
> -
> -    if cfg.nullremus:
> -        fd = save.NullSocket((cfg.host, cfg.port))
> -    else:
> -        fd = save.MigrationSocket((cfg.host, cfg.port))
> -
> -    def postsuspend():
> -        'Begin external checkpointing after domain has paused'
> -        if not cfg.timer:
> -            # when not using a timer thread, sleep until now + interval
> -            closure.starttime = time.time()
> -
> -        if closure.cmd == 's':
> -            die()
> -
> -        for buf in bufs:
> -            buf.postsuspend()
> -
> -        if closure.cmd == 's2':
> -            die()
> -
> -    def preresume():
> -        'Complete external checkpointing before domain resumes'
> -        if closure.cmd == 'r':
> -            die()
> -
> -        for buf in bufs:
> -            buf.preresume()
> -
> -        if closure.cmd == 'r2':
> -            die()
> -
> -    def commit():
> -        'commit network buffer'
> -        if closure.cmd == 'c':
> -            die()
> -
> -        print >> sys.stderr, "PROF: flushed memory at %0.6f" %
(time.time())
> -
> -        for buf in bufs:
> -            buf.commit()
> -
> -        if closure.cmd == 'c2':
> -            die()
> -
> -        # Since the domain is running at this point, it's a good time to
> -        # check for control channel commands
> -        getcommand()
> -
> -        if not cfg.timer:
> -            endtime = time.time()
> -            elapsed = (endtime - closure.starttime) * 1000
> -
> -            if elapsed < cfg.interval:
> -                time.sleep((cfg.interval - elapsed) / 1000.0)
> -
> -        # False ends checkpointing
> -        return True
> -
> -    if cfg.timer:
> -        interval = cfg.interval
> -    else:
> -        interval = 0
> -
> -    rc = 0
> -
> -    checkpointer = save.Saver(cfg.domid, fd, postsuspend, preresume,
commit,
> -                              interval, cfg.flags)
> -
> -    try:
> -        checkpointer.start()
> -    except save.CheckpointError, e:
> -        print e
> -        rc = 1
> -    except KeyboardInterrupt:
> -        pass
> -    except SignalException:
> -        print '*** signalled ***'
> -
> -    for buf in bufs:
> -        buf.uninstall()
> -
> -    sys.exit(rc)
> -
> -cfg = Cfg()
> -try:
> -    cfg.getargs()
> -except CfgException, inst:
> -    print str(inst)
> -    cfg.usage()
> -    sys.exit(1)
> -
> -try:
> -    run(cfg)
> -except vm.VMException, inst:
> -    print str(inst)
> -    sys.exit(1)
> --
> 1.9.1
>

Acked-by: Shriram Rajagopalan <rshriram@cs.ubc.ca>

[-- Attachment #1.2: Type: text/html, Size: 12203 bytes --]

[-- Attachment #2: Type: text/plain, Size: 126 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH 3/4] remus: add wikipage link to remus README
  2014-06-25  7:00 ` [PATCH 3/4] remus: add wikipage link to remus README Yang Hongyang
@ 2014-06-25  7:14   ` Shriram Rajagopalan
  2014-06-25  7:14     ` Hongyang Yang
  2014-06-30 17:05   ` Ian Jackson
  1 sibling, 1 reply; 16+ messages in thread
From: Shriram Rajagopalan @ 2014-06-25  7:14 UTC (permalink / raw)
  To: FNST-Yang Hongyang; +Cc: ian.jackson, xen-devel


[-- Attachment #1.1: Type: text/plain, Size: 734 bytes --]

On Jun 25, 2014 12:34 PM, "Yang Hongyang" <yanghy@cn.fujitsu.com> wrote:
>
> add wikipage link to remus README
>
> Signed-off-by: Yang Hongyang <yanghy@cn.fujitsu.com>
> ---
>  docs/README.remus | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/docs/README.remus b/docs/README.remus
> index 9e8140b..cb1bea0 100644
> --- a/docs/README.remus
> +++ b/docs/README.remus
> @@ -2,3 +2,4 @@ Remus provides fault tolerance for virtual machines by
sending continuous
>  checkpoints to a backup, which will activate if the target VM fails.
>
>  See the website at http://nss.cs.ubc.ca/remus/ for details.

Can you please get rid of the nss.cs link? It does not exist anymore.

> +See also http://wiki.xen.org/wiki/Remus.
> --
> 1.9.1
>

[-- Attachment #1.2: Type: text/html, Size: 1152 bytes --]

[-- Attachment #2: Type: text/plain, Size: 126 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH 3/4] remus: add wikipage link to remus README
  2014-06-25  7:14   ` Shriram Rajagopalan
@ 2014-06-25  7:14     ` Hongyang Yang
  2014-06-25  7:22       ` Shriram Rajagopalan
  0 siblings, 1 reply; 16+ messages in thread
From: Hongyang Yang @ 2014-06-25  7:14 UTC (permalink / raw)
  To: rshriram; +Cc: ian.jackson, xen-devel

On 06/25/2014 03:14 PM, Shriram Rajagopalan wrote:
>
> On Jun 25, 2014 12:34 PM, "Yang Hongyang" <yanghy@cn.fujitsu.com
> <mailto:yanghy@cn.fujitsu.com>> wrote:
>  >
>  > add wikipage link to remus README
>  >
>  > Signed-off-by: Yang Hongyang <yanghy@cn.fujitsu.com
> <mailto:yanghy@cn.fujitsu.com>>
>  > ---
>  >  docs/README.remus | 1 +
>  >  1 file changed, 1 insertion(+)
>  >
>  > diff --git a/docs/README.remus b/docs/README.remus
>  > index 9e8140b..cb1bea0 100644
>  > --- a/docs/README.remus
>  > +++ b/docs/README.remus
>  > @@ -2,3 +2,4 @@ Remus provides fault tolerance for virtual machines by
> sending continuous
>  >  checkpoints to a backup, which will activate if the target VM fails.
>  >
>  >  See the website at http://nss.cs.ubc.ca/remus/ for details.
>
> Can you please get rid of the nss.cs link? It does not exist anymore.

Ok, does it moved to anywhere else, or should I just delete it...

>
>  > +See also http://wiki.xen.org/wiki/Remus.
>  > --
>  > 1.9.1
>  >
>

-- 
Thanks,
Yang.

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH 2/4] remus: move remus README to docs directory
  2014-06-25  7:00 ` [PATCH 2/4] remus: move remus README to docs directory Yang Hongyang
@ 2014-06-25  7:19   ` Shriram Rajagopalan
  2014-06-25  7:38     ` Hongyang Yang
  2014-06-30 17:05   ` Ian Jackson
  1 sibling, 1 reply; 16+ messages in thread
From: Shriram Rajagopalan @ 2014-06-25  7:19 UTC (permalink / raw)
  To: FNST-Yang Hongyang; +Cc: ian.jackson, xen-devel


[-- Attachment #1.1: Type: text/plain, Size: 1284 bytes --]

On Jun 25, 2014 12:34 PM, "Yang Hongyang" <yanghy@cn.fujitsu.com> wrote:
>
> We do not need a separate directory to store remus README, just
> move it to docs/ directory.
>
> Signed-off-by: Yang Hongyang <yanghy@cn.fujitsu.com>
> ---
>  docs/README.remus  | 4 ++++
>  tools/remus/README | 4 ----
>  2 files changed, 4 insertions(+), 4 deletions(-)
>  create mode 100644 docs/README.remus
>  delete mode 100644 tools/remus/README
>
> diff --git a/docs/README.remus b/docs/README.remus
> new file mode 100644
> index 0000000..9e8140b
> --- /dev/null
> +++ b/docs/README.remus
> @@ -0,0 +1,4 @@
> +Remus provides fault tolerance for virtual machines by sending continuous
> +checkpoints to a backup, which will activate if the target VM fails.
> +
> +See the website at http://nss.cs.ubc.ca/remus/ for details.
> diff --git a/tools/remus/README b/tools/remus/README
> deleted file mode 100644
> index 9e8140b..0000000
> --- a/tools/remus/README
> +++ /dev/null
> @@ -1,4 +0,0 @@
> -Remus provides fault tolerance for virtual machines by sending continuous
> -checkpoints to a backup, which will activate if the target VM fails.
> -
> -See the website at http://nss.cs.ubc.ca/remus/ for details.
> --
> 1.9.1
>

Fine by me. But please do remove the nss.cs link as I mentioned in patch 3.

[-- Attachment #1.2: Type: text/html, Size: 1803 bytes --]

[-- Attachment #2: Type: text/plain, Size: 126 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH 3/4] remus: add wikipage link to remus README
  2014-06-25  7:14     ` Hongyang Yang
@ 2014-06-25  7:22       ` Shriram Rajagopalan
  0 siblings, 0 replies; 16+ messages in thread
From: Shriram Rajagopalan @ 2014-06-25  7:22 UTC (permalink / raw)
  To: FNST-Yang Hongyang; +Cc: ian.jackson, xen-devel


[-- Attachment #1.1: Type: text/plain, Size: 1158 bytes --]

On Jun 25, 2014 12:47 PM, "Hongyang Yang" <yanghy@cn.fujitsu.com> wrote:
>
> On 06/25/2014 03:14 PM, Shriram Rajagopalan wrote:
>>
>>
>> On Jun 25, 2014 12:34 PM, "Yang Hongyang" <yanghy@cn.fujitsu.com
>> <mailto:yanghy@cn.fujitsu.com>> wrote:
>>  >
>>  > add wikipage link to remus README
>>  >
>>  > Signed-off-by: Yang Hongyang <yanghy@cn.fujitsu.com
>> <mailto:yanghy@cn.fujitsu.com>>
>>
>>  > ---
>>  >  docs/README.remus | 1 +
>>  >  1 file changed, 1 insertion(+)
>>  >
>>  > diff --git a/docs/README.remus b/docs/README.remus
>>  > index 9e8140b..cb1bea0 100644
>>  > --- a/docs/README.remus
>>  > +++ b/docs/README.remus
>>  > @@ -2,3 +2,4 @@ Remus provides fault tolerance for virtual machines by
>> sending continuous
>>  >  checkpoints to a backup, which will activate if the target VM fails.
>>  >
>>  >  See the website at http://nss.cs.ubc.ca/remus/ for details.
>>
>> Can you please get rid of the nss.cs link? It does not exist anymore.
>
>
> Ok, does it moved to anywhere else, or should I just delete it...
>

Delete it please.

>
>>
>>  > +See also http://wiki.xen.org/wiki/Remus.
>>  > --
>>  > 1.9.1
>>  >
>>
>
> --
> Thanks,
> Yang.
>

[-- Attachment #1.2: Type: text/html, Size: 2091 bytes --]

[-- Attachment #2: Type: text/plain, Size: 126 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH 2/4] remus: move remus README to docs directory
  2014-06-25  7:19   ` Shriram Rajagopalan
@ 2014-06-25  7:38     ` Hongyang Yang
  2014-06-25  7:50       ` Shriram Rajagopalan
  0 siblings, 1 reply; 16+ messages in thread
From: Hongyang Yang @ 2014-06-25  7:38 UTC (permalink / raw)
  To: rshriram; +Cc: ian.jackson, xen-devel

Hi Shriram,

On 06/25/2014 03:19 PM, Shriram Rajagopalan wrote:
>
> On Jun 25, 2014 12:34 PM, "Yang Hongyang" <yanghy@cn.fujitsu.com
> <mailto:yanghy@cn.fujitsu.com>> wrote:
>  >
>  > We do not need a separate directory to store remus README, just
>  > move it to docs/ directory.
>  >
>  > Signed-off-by: Yang Hongyang <yanghy@cn.fujitsu.com
> <mailto:yanghy@cn.fujitsu.com>>
>  > ---
>  >  docs/README.remus  | 4 ++++
>  >  tools/remus/README | 4 ----
>  >  2 files changed, 4 insertions(+), 4 deletions(-)
>  >  create mode 100644 docs/README.remus
>  >  delete mode 100644 tools/remus/README
>  >
>  > diff --git a/docs/README.remus b/docs/README.remus
>  > new file mode 100644
>  > index 0000000..9e8140b
>  > --- /dev/null
>  > +++ b/docs/README.remus
>  > @@ -0,0 +1,4 @@
>  > +Remus provides fault tolerance for virtual machines by sending continuous
>  > +checkpoints to a backup, which will activate if the target VM fails.
>  > +
>  > +See the website at http://nss.cs.ubc.ca/remus/ for details.
>  > diff --git a/tools/remus/README b/tools/remus/README
>  > deleted file mode 100644
>  > index 9e8140b..0000000
>  > --- a/tools/remus/README
>  > +++ /dev/null
>  > @@ -1,4 +0,0 @@
>  > -Remus provides fault tolerance for virtual machines by sending continuous
>  > -checkpoints to a backup, which will activate if the target VM fails.
>  > -
>  > -See the website at http://nss.cs.ubc.ca/remus/ for details.
>  > --
>  > 1.9.1
>  >
>
> Fine by me. But please do remove the nss.cs link as I mentioned in patch 3.
>

After the first patch, there is only README file left in tools/remus directory,
so I moved it to docs/

-- 
Thanks,
Yang.

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH 2/4] remus: move remus README to docs directory
  2014-06-25  7:38     ` Hongyang Yang
@ 2014-06-25  7:50       ` Shriram Rajagopalan
  2014-06-30  5:56         ` Hongyang Yang
  0 siblings, 1 reply; 16+ messages in thread
From: Shriram Rajagopalan @ 2014-06-25  7:50 UTC (permalink / raw)
  To: FNST-Yang Hongyang; +Cc: ian.jackson, xen-devel


[-- Attachment #1.1: Type: text/plain, Size: 1997 bytes --]

On Jun 25, 2014 1:13 PM, "Hongyang Yang" <yanghy@cn.fujitsu.com> wrote:
>
> Hi Shriram,
>
>
> On 06/25/2014 03:19 PM, Shriram Rajagopalan wrote:
>>
>>
>> On Jun 25, 2014 12:34 PM, "Yang Hongyang" <yanghy@cn.fujitsu.com
>> <mailto:yanghy@cn.fujitsu.com>> wrote:
>>  >
>>  > We do not need a separate directory to store remus README, just
>>  > move it to docs/ directory.
>>  >
>>  > Signed-off-by: Yang Hongyang <yanghy@cn.fujitsu.com
>> <mailto:yanghy@cn.fujitsu.com>>
>>
>>  > ---
>>  >  docs/README.remus  | 4 ++++
>>  >  tools/remus/README | 4 ----
>>  >  2 files changed, 4 insertions(+), 4 deletions(-)
>>  >  create mode 100644 docs/README.remus
>>  >  delete mode 100644 tools/remus/README
>>  >
>>  > diff --git a/docs/README.remus b/docs/README.remus
>>  > new file mode 100644
>>  > index 0000000..9e8140b
>>  > --- /dev/null
>>  > +++ b/docs/README.remus
>>  > @@ -0,0 +1,4 @@
>>  > +Remus provides fault tolerance for virtual machines by sending
continuous
>>  > +checkpoints to a backup, which will activate if the target VM fails.
>>  > +
>>  > +See the website at http://nss.cs.ubc.ca/remus/ for details.
>>  > diff --git a/tools/remus/README b/tools/remus/README
>>  > deleted file mode 100644
>>  > index 9e8140b..0000000
>>  > --- a/tools/remus/README
>>  > +++ /dev/null
>>  > @@ -1,4 +0,0 @@
>>  > -Remus provides fault tolerance for virtual machines by sending
continuous
>>  > -checkpoints to a backup, which will activate if the target VM fails.
>>  > -
>>  > -See the website at http://nss.cs.ubc.ca/remus/ for details.
>>  > --
>>  > 1.9.1
>>  >
>>
>> Fine by me. But please do remove the nss.cs link as I mentioned in patch
3.
>>
>
> After the first patch, there is only README file left in tools/remus
directory,
> so I moved it to docs/
>

Yep. That's fine.
Acked-by: Shriram Rajagopalan <rshriram@cs.ubc.ca>
> --
> Thanks,
> Yang.
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> http://lists.xen.org/xen-devel

[-- Attachment #1.2: Type: text/html, Size: 3305 bytes --]

[-- Attachment #2: Type: text/plain, Size: 126 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH 2/4] remus: move remus README to docs directory
  2014-06-25  7:50       ` Shriram Rajagopalan
@ 2014-06-30  5:56         ` Hongyang Yang
  0 siblings, 0 replies; 16+ messages in thread
From: Hongyang Yang @ 2014-06-30  5:56 UTC (permalink / raw)
  To: rshriram; +Cc: ian.jackson, xen-devel

Hi Shriram,

   Since there's already v2 in list, would you please give an ack on v2
patch?

On 06/25/2014 03:50 PM, Shriram Rajagopalan wrote:
>
> On Jun 25, 2014 1:13 PM, "Hongyang Yang" <yanghy@cn.fujitsu.com
> <mailto:yanghy@cn.fujitsu.com>> wrote:
>  >
>  > Hi Shriram,
>  >
>  >
>  > On 06/25/2014 03:19 PM, Shriram Rajagopalan wrote:
>  >>
>  >>
>  >> On Jun 25, 2014 12:34 PM, "Yang Hongyang" <yanghy@cn.fujitsu.com
> <mailto:yanghy@cn.fujitsu.com>
>  >> <mailto:yanghy@cn.fujitsu.com <mailto:yanghy@cn.fujitsu.com>>> wrote:
>  >>  >
>  >>  > We do not need a separate directory to store remus README, just
>  >>  > move it to docs/ directory.
>  >>  >
>  >>  > Signed-off-by: Yang Hongyang <yanghy@cn.fujitsu.com
> <mailto:yanghy@cn.fujitsu.com>
>  >> <mailto:yanghy@cn.fujitsu.com <mailto:yanghy@cn.fujitsu.com>>>
>  >>
>  >>  > ---
>  >>  >  docs/README.remus  | 4 ++++
>  >>  >  tools/remus/README | 4 ----
>  >>  >  2 files changed, 4 insertions(+), 4 deletions(-)
>  >>  >  create mode 100644 docs/README.remus
>  >>  >  delete mode 100644 tools/remus/README
>  >>  >
>  >>  > diff --git a/docs/README.remus b/docs/README.remus
>  >>  > new file mode 100644
>  >>  > index 0000000..9e8140b
>  >>  > --- /dev/null
>  >>  > +++ b/docs/README.remus
>  >>  > @@ -0,0 +1,4 @@
>  >>  > +Remus provides fault tolerance for virtual machines by sending continuous
>  >>  > +checkpoints to a backup, which will activate if the target VM fails.
>  >>  > +
>  >>  > +See the website at http://nss.cs.ubc.ca/remus/ for details.
>  >>  > diff --git a/tools/remus/README b/tools/remus/README
>  >>  > deleted file mode 100644
>  >>  > index 9e8140b..0000000
>  >>  > --- a/tools/remus/README
>  >>  > +++ /dev/null
>  >>  > @@ -1,4 +0,0 @@
>  >>  > -Remus provides fault tolerance for virtual machines by sending continuous
>  >>  > -checkpoints to a backup, which will activate if the target VM fails.
>  >>  > -
>  >>  > -See the website at http://nss.cs.ubc.ca/remus/ for details.
>  >>  > --
>  >>  > 1.9.1
>  >>  >
>  >>
>  >> Fine by me. But please do remove the nss.cs link as I mentioned in patch 3.
>  >>
>  >
>  > After the first patch, there is only README file left in tools/remus directory,
>  > so I moved it to docs/
>  >
>
> Yep. That's fine.
> Acked-by: Shriram Rajagopalan <rshriram@cs.ubc.ca <mailto:rshriram@cs.ubc.ca>>
>  > --
>  > Thanks,
>  > Yang.
>  >
>  > _______________________________________________
>  > Xen-devel mailing list
>  > Xen-devel@lists.xen.org <mailto:Xen-devel@lists.xen.org>
>  > http://lists.xen.org/xen-devel
>

-- 
Thanks,
Yang.

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH 1/4] remus: remove old remus script
  2014-06-25  7:00 [PATCH 1/4] remus: remove old remus script Yang Hongyang
                   ` (3 preceding siblings ...)
  2014-06-25  7:12 ` [PATCH 1/4] remus: remove old remus script Shriram Rajagopalan
@ 2014-06-30 17:05 ` Ian Jackson
  4 siblings, 0 replies; 16+ messages in thread
From: Ian Jackson @ 2014-06-30 17:05 UTC (permalink / raw)
  To: Yang Hongyang; +Cc: rshriram, xen-devel

Yang Hongyang writes ("[PATCH 1/4] remus: remove old remus script"):
> Since xend already deleted, the old remus script which based on
> xend no longer functional, remove it.

Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH 2/4] remus: move remus README to docs directory
  2014-06-25  7:00 ` [PATCH 2/4] remus: move remus README to docs directory Yang Hongyang
  2014-06-25  7:19   ` Shriram Rajagopalan
@ 2014-06-30 17:05   ` Ian Jackson
  1 sibling, 0 replies; 16+ messages in thread
From: Ian Jackson @ 2014-06-30 17:05 UTC (permalink / raw)
  To: Yang Hongyang; +Cc: rshriram, xen-devel

Yang Hongyang writes ("[PATCH 2/4] remus: move remus README to docs directory"):
> We do not need a separate directory to store remus README, just
> move it to docs/ directory.

Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH 3/4] remus: add wikipage link to remus README
  2014-06-25  7:00 ` [PATCH 3/4] remus: add wikipage link to remus README Yang Hongyang
  2014-06-25  7:14   ` Shriram Rajagopalan
@ 2014-06-30 17:05   ` Ian Jackson
  1 sibling, 0 replies; 16+ messages in thread
From: Ian Jackson @ 2014-06-30 17:05 UTC (permalink / raw)
  To: Yang Hongyang; +Cc: rshriram, xen-devel

Yang Hongyang writes ("[PATCH 3/4] remus: add wikipage link to remus README"):
> add wikipage link to remus README

Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH 4/4] MAINTAINERS: Update maintained files of REMUS part
  2014-06-25  7:00 ` [PATCH 4/4] MAINTAINERS: Update maintained files of REMUS part Yang Hongyang
@ 2014-06-30 17:06   ` Ian Jackson
  0 siblings, 0 replies; 16+ messages in thread
From: Ian Jackson @ 2014-06-30 17:06 UTC (permalink / raw)
  To: Yang Hongyang; +Cc: rshriram, xen-devel

Yang Hongyang writes ("[PATCH 4/4] MAINTAINERS: Update maintained files of REMUS part"):
> delete tools/remus and add docs/README.remus

Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>

^ permalink raw reply	[flat|nested] 16+ messages in thread

end of thread, other threads:[~2014-06-30 17:06 UTC | newest]

Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-06-25  7:00 [PATCH 1/4] remus: remove old remus script Yang Hongyang
2014-06-25  7:00 ` [PATCH 2/4] remus: move remus README to docs directory Yang Hongyang
2014-06-25  7:19   ` Shriram Rajagopalan
2014-06-25  7:38     ` Hongyang Yang
2014-06-25  7:50       ` Shriram Rajagopalan
2014-06-30  5:56         ` Hongyang Yang
2014-06-30 17:05   ` Ian Jackson
2014-06-25  7:00 ` [PATCH 3/4] remus: add wikipage link to remus README Yang Hongyang
2014-06-25  7:14   ` Shriram Rajagopalan
2014-06-25  7:14     ` Hongyang Yang
2014-06-25  7:22       ` Shriram Rajagopalan
2014-06-30 17:05   ` Ian Jackson
2014-06-25  7:00 ` [PATCH 4/4] MAINTAINERS: Update maintained files of REMUS part Yang Hongyang
2014-06-30 17:06   ` Ian Jackson
2014-06-25  7:12 ` [PATCH 1/4] remus: remove old remus script Shriram Rajagopalan
2014-06-30 17:05 ` Ian Jackson

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.