All of lore.kernel.org
 help / color / mirror / Atom feed
From: Roger Gammans <roger@computer-surgery.co.uk>
To: Sam Ravnborg <sam@ravnborg.org>
Cc: linux-kernel@vger.kernel.org, akpm@zip.com.au
Subject: Re: JBD Documentation added in BK-current
Date: Mon, 14 Oct 2002 23:03:15 +0100	[thread overview]
Message-ID: <20021014230314.A19801@computer-surgery.co.uk> (raw)
In-Reply-To: <20021008212310.A13265@mars.ravnborg.org>; from sam@ravnborg.org on Tue, Oct 08, 2002 at 09:23:10PM +0200

[-- Attachment #1: Type: text/plain, Size: 2896 bytes --]

On Tue, Oct 08, 2002 at 09:23:10PM +0200, Sam Ravnborg wrote:
> The JBD documentation have been added in BK-current.
> But 'compiling' the documentation result in a lot of SGML related
> errors.
> 
> Someone knowing SGML that care to take a look?

Well. I'm not surprised there are a few I couldn't find an SGML reference
for the linuxdoc dtd , or a tutorial when I wrote it, so I guessed by looking
at other documents and source code. 

I thought I'd checked the warnings

> [I'm cleaning up the mess in the Makefile after JBD was added right now].

Did I really make it that messy?
I'm a little behind with bk pulling and it takes 2-3hours to do each pull
here, so I can't be sure until tomorrow .

but if the line number haven't changed...

>   DB2PS   Documentation/DocBook/journal-api.ps
> Using catalogs: /etc/sgml/sgml-docbook-3.1.cat
> Using stylesheet: /usr/share/sgml/docbook/utils-0.6.9/docbook-utils.dsl#print
> Working on: /home/sam/src/linux/kernel/bk/clean3/Documentation/DocBook/journal-api.sgml
> jade:/.../journal-api.sgml:211:5:E: document type does not allow element "PARA" here; missing one of "FOOTNOTE", "MSGTEXT", "CAUTION", "IMPORTANT", "NOTE", "TIP", "WARNING", "BLOCKQUOTE", "INFORMALEXAMPLE" start-tag
> jade:/.../journal-api.sgml:212:7:E: end tag for "PARA" omitted, but OMITTAG NO was specified
> jade:/.../journal-api.sgml:212:7:E: end tag for "PARA" omitted, but OMITTAG NO was specified

Oops, there are errant <para> tag journal-api.tmpl line 211. Delete the line.

> jade:/.../journal-api.sgml:248:9:E: end tag for "PARA" omitted, but OMITTAG NO was specified
> jade:/.../journal-api.sgml:248:9:E: end tag for "SECT1" omitted, but OMITTAG NO was specified

Hmm, can't see what these asre unless they are run-on errors form above.

Hang on. yup they seem to be.... , the trivial patch follows.

You also mentioned in private mail to me that you think there is
a requirement for BKL to be held on some journal calls. Could
someone else preferably part of the ext3 maintance team give me
cahpter and verse on this situationin 2.4 and 2.5 so I can
ensure the docs are correct, assuming they aren't already which is quite
likely since the docs were written by inspection of the ext3/jbd code 
and reading of various other docs avaliable (as mention in the sgml).


diff -Nru a/Documentation/DocBook/journal-api.tmpl b/Documentation/DocBook/journal-api.tmpl
--- a/Documentation/DocBook/journal-api.tmpl	Mon Oct 14 22:46:55 2002
+++ b/Documentation/DocBook/journal-api.tmpl	Mon Oct 14 22:46:55 2002
@@ -208,7 +208,6 @@
 if you allow unprivileged userspace to trigger codepaths containing these
 calls.
 
-<para>
 </sect1>
 <sect1>
 <title>Summary</title>


TTFN
-- 
Roger.
Master of Peng Shui.  (Ancient oriental art of Penguin Arranging)
GPG Key FPR: CFF1 F383 F854 4E6A 918D  5CFF A90D E73B 88DE 0B3E

[-- Attachment #2: Type: application/pgp-signature, Size: 232 bytes --]

  reply	other threads:[~2002-10-14 21:57 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-10-08 19:23 JBD Documentation added in BK-current Sam Ravnborg
2002-10-14 22:03 ` Roger Gammans [this message]
2002-10-15 18:16   ` Sam Ravnborg
2002-10-15 21:38     ` Roger Gammans

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=20021014230314.A19801@computer-surgery.co.uk \
    --to=roger@computer-surgery.co.uk \
    --cc=akpm@zip.com.au \
    --cc=linux-kernel@vger.kernel.org \
    --cc=sam@ravnborg.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 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.