Message ID | ce2da767-8193-bb80-ba33-4aba0ec6cc6d@redhat.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
On 05/24/2016 10:50 AM, Paolo Bonzini wrote: > On 24/05/2016 10:17, Janosch Frank wrote: >> The script already outputs a help text, which could be extended to the >> man's text. I.e. I left it out on purpose. >> >> Anyway, creating the manpage from asciidoc sources is a matter of having >> the right packages and simply calling: >> a2x --doctype manpage --format manpage file.txt >> >> Works flawlessly with your example and is much more readable than the >> texi source. I would give it a try and add it to the first patch if you >> do not have any concerns? > > I will include the following as a separate patch: Great, thanks! Don't forget the manpage changes for the pid monitoring, i.e. the -p/--pid argument and the removal of the note. > > ---------------- 8< --------------- > From ccceb628a51e52a4b6384e6ef1cc9d88daf00a62 Mon Sep 17 00:00:00 2001 > From: Paolo Bonzini <pbonzini@redhat.com> > Date: Tue, 24 May 2016 10:41:15 +0200 > Subject: [PATCH] tools: Add kvm_stat man page > > Converted from the Texinfo source in QEMU to asciidoc. The a2x > incantation was provided by Frank Janosch. s/Frank Janosch/Janosch Frank/ Janosch is my first name. > > Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> > --- > tools/kvm/kvm_stat/Makefile | 40 ++++++++++++++++++++++++-- > tools/kvm/kvm_stat/kvm_stat.txt | 62 +++++++++++++++++++++++++++++++++++++++++ > 2 files changed, 100 insertions(+), 2 deletions(-) > create mode 100644 tools/kvm/kvm_stat/kvm_stat.txt > > diff --git a/tools/kvm/kvm_stat/Makefile b/tools/kvm/kvm_stat/Makefile > index c639b8d30688..5b1cba57e3b3 100644 > --- a/tools/kvm/kvm_stat/Makefile > +++ b/tools/kvm/kvm_stat/Makefile > @@ -1,5 +1,41 @@ > +include ../../scripts/Makefile.include > +include ../../scripts/utilities.mak > BINDIR=usr/bin > +MANDIR=usr/share/man > +MAN1DIR=$(MANDIR)/man1 > > -install: > - mkdir -p $(INSTALL_ROOT)/$(BINDIR) > +MAN1=kvm_stat.1 > + > +A2X=a2x > +a2x_path := $(call get-executable,$(A2X)) > + > +all: man > + > +ifneq ($(findstring $(MAKEFLAGS),s),s) > + ifneq ($(V),1) > + QUIET_A2X = @echo ' A2X '$@; > + endif > +endif > + > +%.1: %.txt > +ifeq ($(a2x_path),) > + $(error "You need to install asciidoc for man pages") > +else > + $(QUIET_A2X)$(A2X) --doctype manpage --format manpage $< > +endif > + > +clean: > + rm -f $(MAN1) > + > +man: $(MAN1) > + > +install-man: man > + install -d -m 755 $(INSTALL_ROOT)/$(MAN1DIR) > + install -m 644 kvm_stat.1 $(INSTALL_ROOT)/$(MAN1DIR) > + > +install-tools: > + install -d -m 755 $(INSTALL_ROOT)/$(BINDIR) > install -m 755 -p "kvm_stat" "$(INSTALL_ROOT)/$(BINDIR)/$(TARGET)" > + > +install: install-tools install-man > +.PHONY: all clean man install-tools install-man install > diff --git a/tools/kvm/kvm_stat/kvm_stat.txt b/tools/kvm/kvm_stat/kvm_stat.txt > new file mode 100644 > index 000000000000..039dee80ddcb > --- /dev/null > +++ b/tools/kvm/kvm_stat/kvm_stat.txt > @@ -0,0 +1,62 @@ > +kvm_stat(1) > +=========== > + > +NAME > +---- > +kvm_stat - Report KVM kernel module event counters > + > +SYNOPSIS > +-------- > +[verse] > +'kvm_stat' [OPTION]... > + > +DESCRIPTION > +----------- > +kvm_stat prints counts of KVM kernel module trace events. These events signify > +state transitions such as guest mode entry and exit. > + > +This tool is useful for observing guest behavior from the host perspective. > +Often conclusions about performance or buggy behavior can be drawn from the > +output. > + > +The set of KVM kernel module trace events may be specific to the kernel version > +or architecture. It is best to check the KVM kernel module source code for the > +meaning of events. > + > +Note that trace events are counted globally across all running guests. > + > +OPTIONS > +------- > +-1:: > +--once:: > +--batch:: > + run in batch mode for one second > + > +-l:: > +--log:: > + run in logging mode (like vmstat) > + > +-t:: > +--tracepoints:: > + retrieve statistics from tracepoints > + > +-d:: > +--debugfs:: > + retrieve statistics from debugfs > + > +-f<fields>:: > +--fields=<fields>:: > + fields to display (regex) > + > +-h:: > +--help:: > + > + show help message > + > +SEE ALSO > +-------- > +'perf'(1), 'trace-cmd'(1) > + > +AUTHOR > +------ > +Stefan Hajnoczi <stefanha@redhat.com> > -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 24/05/2016 10:57, Janosch Frank wrote: >> Converted from the Texinfo source in QEMU to asciidoc. The a2x >> incantation was provided by Frank Janosch. > > s/Frank Janosch/Janosch Frank/ > Janosch is my first name. Oops, sorry. I must not be the first to make this mistake. :) Paolo >> >> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> >> --- >> tools/kvm/kvm_stat/Makefile | 40 ++++++++++++++++++++++++-- >> tools/kvm/kvm_stat/kvm_stat.txt | 62 +++++++++++++++++++++++++++++++++++++++++ >> 2 files changed, 100 insertions(+), 2 deletions(-) >> create mode 100644 tools/kvm/kvm_stat/kvm_stat.txt >> >> diff --git a/tools/kvm/kvm_stat/Makefile b/tools/kvm/kvm_stat/Makefile >> index c639b8d30688..5b1cba57e3b3 100644 >> --- a/tools/kvm/kvm_stat/Makefile >> +++ b/tools/kvm/kvm_stat/Makefile >> @@ -1,5 +1,41 @@ >> +include ../../scripts/Makefile.include >> +include ../../scripts/utilities.mak >> BINDIR=usr/bin >> +MANDIR=usr/share/man >> +MAN1DIR=$(MANDIR)/man1 >> >> -install: >> - mkdir -p $(INSTALL_ROOT)/$(BINDIR) >> +MAN1=kvm_stat.1 >> + >> +A2X=a2x >> +a2x_path := $(call get-executable,$(A2X)) >> + >> +all: man >> + >> +ifneq ($(findstring $(MAKEFLAGS),s),s) >> + ifneq ($(V),1) >> + QUIET_A2X = @echo ' A2X '$@; >> + endif >> +endif >> + >> +%.1: %.txt >> +ifeq ($(a2x_path),) >> + $(error "You need to install asciidoc for man pages") >> +else >> + $(QUIET_A2X)$(A2X) --doctype manpage --format manpage $< >> +endif >> + >> +clean: >> + rm -f $(MAN1) >> + >> +man: $(MAN1) >> + >> +install-man: man >> + install -d -m 755 $(INSTALL_ROOT)/$(MAN1DIR) >> + install -m 644 kvm_stat.1 $(INSTALL_ROOT)/$(MAN1DIR) >> + >> +install-tools: >> + install -d -m 755 $(INSTALL_ROOT)/$(BINDIR) >> install -m 755 -p "kvm_stat" "$(INSTALL_ROOT)/$(BINDIR)/$(TARGET)" >> + >> +install: install-tools install-man >> +.PHONY: all clean man install-tools install-man install >> diff --git a/tools/kvm/kvm_stat/kvm_stat.txt b/tools/kvm/kvm_stat/kvm_stat.txt >> new file mode 100644 >> index 000000000000..039dee80ddcb >> --- /dev/null >> +++ b/tools/kvm/kvm_stat/kvm_stat.txt >> @@ -0,0 +1,62 @@ >> +kvm_stat(1) >> +=========== >> + >> +NAME >> +---- >> +kvm_stat - Report KVM kernel module event counters >> + >> +SYNOPSIS >> +-------- >> +[verse] >> +'kvm_stat' [OPTION]... >> + >> +DESCRIPTION >> +----------- >> +kvm_stat prints counts of KVM kernel module trace events. These events signify >> +state transitions such as guest mode entry and exit. >> + >> +This tool is useful for observing guest behavior from the host perspective. >> +Often conclusions about performance or buggy behavior can be drawn from the >> +output. >> + >> +The set of KVM kernel module trace events may be specific to the kernel version >> +or architecture. It is best to check the KVM kernel module source code for the >> +meaning of events. >> + >> +Note that trace events are counted globally across all running guests. >> + >> +OPTIONS >> +------- >> +-1:: >> +--once:: >> +--batch:: >> + run in batch mode for one second >> + >> +-l:: >> +--log:: >> + run in logging mode (like vmstat) >> + >> +-t:: >> +--tracepoints:: >> + retrieve statistics from tracepoints >> + >> +-d:: >> +--debugfs:: >> + retrieve statistics from debugfs >> + >> +-f<fields>:: >> +--fields=<fields>:: >> + fields to display (regex) >> + >> +-h:: >> +--help:: >> + >> + show help message >> + >> +SEE ALSO >> +-------- >> +'perf'(1), 'trace-cmd'(1) >> + >> +AUTHOR >> +------ >> +Stefan Hajnoczi <stefanha@redhat.com> >> > -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/tools/kvm/kvm_stat/Makefile b/tools/kvm/kvm_stat/Makefile index c639b8d30688..5b1cba57e3b3 100644 --- a/tools/kvm/kvm_stat/Makefile +++ b/tools/kvm/kvm_stat/Makefile @@ -1,5 +1,41 @@ +include ../../scripts/Makefile.include +include ../../scripts/utilities.mak BINDIR=usr/bin +MANDIR=usr/share/man +MAN1DIR=$(MANDIR)/man1 -install: - mkdir -p $(INSTALL_ROOT)/$(BINDIR) +MAN1=kvm_stat.1 + +A2X=a2x +a2x_path := $(call get-executable,$(A2X)) + +all: man + +ifneq ($(findstring $(MAKEFLAGS),s),s) + ifneq ($(V),1) + QUIET_A2X = @echo ' A2X '$@; + endif +endif + +%.1: %.txt +ifeq ($(a2x_path),) + $(error "You need to install asciidoc for man pages") +else + $(QUIET_A2X)$(A2X) --doctype manpage --format manpage $< +endif + +clean: + rm -f $(MAN1) + +man: $(MAN1) + +install-man: man + install -d -m 755 $(INSTALL_ROOT)/$(MAN1DIR) + install -m 644 kvm_stat.1 $(INSTALL_ROOT)/$(MAN1DIR) + +install-tools: + install -d -m 755 $(INSTALL_ROOT)/$(BINDIR) install -m 755 -p "kvm_stat" "$(INSTALL_ROOT)/$(BINDIR)/$(TARGET)" + +install: install-tools install-man +.PHONY: all clean man install-tools install-man install diff --git a/tools/kvm/kvm_stat/kvm_stat.txt b/tools/kvm/kvm_stat/kvm_stat.txt new file mode 100644 index 000000000000..039dee80ddcb --- /dev/null +++ b/tools/kvm/kvm_stat/kvm_stat.txt @@ -0,0 +1,62 @@ +kvm_stat(1) +=========== + +NAME +---- +kvm_stat - Report KVM kernel module event counters + +SYNOPSIS +-------- +[verse] +'kvm_stat' [OPTION]... + +DESCRIPTION +----------- +kvm_stat prints counts of KVM kernel module trace events. These events signify +state transitions such as guest mode entry and exit. + +This tool is useful for observing guest behavior from the host perspective. +Often conclusions about performance or buggy behavior can be drawn from the +output. + +The set of KVM kernel module trace events may be specific to the kernel version +or architecture. It is best to check the KVM kernel module source code for the +meaning of events. + +Note that trace events are counted globally across all running guests. + +OPTIONS +------- +-1:: +--once:: +--batch:: + run in batch mode for one second + +-l:: +--log:: + run in logging mode (like vmstat) + +-t:: +--tracepoints:: + retrieve statistics from tracepoints + +-d:: +--debugfs:: + retrieve statistics from debugfs + +-f<fields>:: +--fields=<fields>:: + fields to display (regex) + +-h:: +--help:: + + show help message + +SEE ALSO +-------- +'perf'(1), 'trace-cmd'(1) + +AUTHOR +------ +Stefan Hajnoczi <stefanha@redhat.com>