All of lore.kernel.org
 help / color / mirror / Atom feed
From: MINAI Katsuhito <minai@jp.fujitsu.com>
To: Akio Takebe <takebe_akio@jp.fujitsu.com>
Cc: Hans-Christian Armingeon <mog.johnny@gmx.net>,
	xen-devel@lists.xensource.com,
	Mark Williamson <mark.williamson@cl.cam.ac.uk>
Subject: [PATCH] [0/4] put xen console message into syslog but xm dmesg is not affected
Date: Fri, 07 Jul 2006 17:15:55 +0900	[thread overview]
Message-ID: <20060707164914.E3C6.MINAI@jp.fujitsu.com> (raw)
In-Reply-To: <D1C69080344B8Ftakebe_akio@jp.fujitsu.com>

Hi Akio and all

Thank you for your comments. I'm sorry that the answer is late.
I will send the following 4 patches following this mail.

[1/4] add read pointer to the xen console ring
[2/4] add xenlogd daemon for xen console logging
[3/4] modify xend for automatically starting xenlogd
[4/4] README of xenlogd


> This xenlog tools clear a reading part of console buffer.
> I think xenlogd which don't clear the buffer is more cool.
> But this fix might modify xen's code.
> So this fix is trade off.

I solved this problem by adding another read pointer of the console 
buffer of Xen for xenlogd daemon.
The pointer is used when a negative value is specified 
for the fourth argument(named "clear") of the xc_readconsolering() function. 

The console can be logged independently by this correction 
with the operation of the xm dmesg command. 



> 	You should add -Werror -g option into CFLAGS
> 	And you must add .PHONY, build, install section,
> 	because of "make tools" and "make install-tools"
> 	So you may add "SUBDIRS-y += xenlogd" into tools/Makefile 

I decided to put xenlogd on tools/misc in this patch though I made a 
new directory in the last patch for xenlogd. 

And, xenlogd was automatically started from xend python script.
So you have nothing to do except applying the patch.


Best regards,
Katsuhito Minai


On Thu, 15 Jun 2006 22:32:53 +0900
Akio Takebe <takebe_akio@jp.fujitsu.com> wrote:
> Hi, Minai
> 
> I think this is cool tool!
> I want you to add xenlogd into /etc/init.d/xend.
> 
> And I have small comments.
> See the below.
> 
> diff -r 4f1e39ec05d6 tools/xenlogd/Makefile
> --- /dev/null	Thu Jan 01 00:00:00 1970 +0000
> +++ b/tools/xenlogd/Makefile	Fri Jun 09 15:40:36 2006 +0900
> @@ -0,0 +1,7 @@
> +CC=gcc
> +CFLAGS=-lxenctrl
> +
> +xenlogd:
> +
> +clean:
> +	rm -f xenlogd
> 	You should add -Werror -g option into CFLAGS
> 	And you must add .PHONY, build, install section,
> 	because of "make tools" and "make install-tools"
> 	So you may add "SUBDIRS-y += xenlogd" into tools/Makefile 
> 
> 
> diff -r 4f1e39ec05d6 tools/xenlogd/xenlogd.c
> --- /dev/null	Thu Jan 01 00:00:00 1970 +0000
> +++ b/tools/xenlogd/xenlogd.c	Fri Jun 09 15:40:36 2006 +0900
> @@ -0,0 +1,283 @@
> +#include <sys/stat.h>
> +#include <sys/types.h>
> +#include <sys/wait.h>
> +#include <stdio.h>
> +#include <stdlib.h>
> +#include <syslog.h>
> +#include <signal.h>
> +#include <errno.h>
> +#include <stdarg.h>
> +#include <fcntl.h>
> +#include <string.h>
> +
> +
> +#define XCDSIZE (4096)
> +#define XENHEAD "(XEN) "
> +#define PIDFILE "/var/run/xenlogd.pid"
> +#define STARTMSG "logging has been started."
> +
> 	And STARTMSG isn't used. :-)
> 
> This xenlog tools clear a reading part of console buffer.
> I think xenlogd which don't clear the buffer is more cool.
> But this fix might modify xen's code.
> So this fix is trade off.
> Hmm.... 
> I hope other comments from community. :-)
> 
> Best Regards,
> 
> Akio Takebe
> 
> >Hi all,
> >
> >I made the logging daemon of the Xen message.
> >This daemon outputs the messege to syslog if messages were found
> >in the console ring by the polling every one second.
> >
> >Following is the usage, but usually no option is necessary.
> >It will automatically be daemon.
> >
> >usage: xenlogd [-h] [-k] [-r]
> >       -h: print help
> >       -k: kill alive logging daemon
> >       -r: restart logging daemon after kill alive one
> >
> >Signed-off-by: Katsuhito Minai <minai@jp.fujitsu.com>
> >
> >Best regards,
> >Katsuhito Minai
> >
> >
> >
> >On Tue, 23 May 2006 16:31:44 +0100
> >Mark Williamson <mark.williamson@cl.cam.ac.uk> wrote:
> >> > short questin: I want to direct the oputput of xm dmesg in (near) 
> >> > realtime
> >> > to syslog.
> >> >
> >> > Is this possible?
> >> 
> >> dom0 Linux doesn't (currently) get any notifications that the xm dmesg 
> >> output 
> >> has changed.
> >> 
> >> You could write a daemon to poll xm dmesg and send changes to syslog...  
> >> You 
> >> might like to consider having it clear the dmesg also, since it uses a 
> >> fixed-size buffer and won't work once that is full.
> >> 
> >> Cheers,
> >> Mark
> >> 
> >> -- 
> >> Dave: Just a question. What use is a unicyle with no seat?  And no pedals!
> >> Mark: To answer a question with a question: What use is a skateboard?
> >> Dave: Skateboards have wheels.
> >> Mark: My wheel has a wheel!
> >> 
> >> _______________________________________________
> >> Xen-users mailing list
> >> Xen-users@lists.xensource.com
> >> http://lists.xensource.com/xen-users
> >
> >
> >-------------------------------text/plain-------------------------------
> >_______________________________________________
> >Xen-devel mailing list
> >Xen-devel@lists.xensource.com
> >http://lists.xensource.com/xen-devel
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xensource.com
> http://lists.xensource.com/xen-devel

      reply	other threads:[~2006-07-07  8:15 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <200605231459.33644.mog.johnny@gmx.net>
     [not found] ` <200605231631.44919.mark.williamson@cl.cam.ac.uk>
2006-06-09  7:32   ` directing xm dmesg to syslog MINAI Katsuhito
2006-06-15 13:32     ` [Xen-devel] " Akio Takebe
2006-07-07  8:15       ` MINAI Katsuhito [this message]

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=20060707164914.E3C6.MINAI@jp.fujitsu.com \
    --to=minai@jp.fujitsu.com \
    --cc=mark.williamson@cl.cam.ac.uk \
    --cc=mog.johnny@gmx.net \
    --cc=takebe_akio@jp.fujitsu.com \
    --cc=xen-devel@lists.xensource.com \
    /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 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.