技術

[xv6 #69] Chapter 5 – File system – Exercises

テキストの77ページ

本文

1. どういう場合にballocでpanicが起きるか? それを回復することはできるか?

2. どういう場合にiallocでpanicが起きるか? それを回復することはできるか?

3. inode世代番号。

4. fileがない場合になぜfileallocはpanicを起こさないのか? なぜこれがより一般的でそれゆえ良いハンドリングの方法なのか?

5. sys_linkがiunlock(ip)とdirlinkを呼び出す間に、そのipが、他のプロセスによってアンリンクされたipと一致すると仮定する。
リンクは正しく作成されるだろうか?
なぜ正しく作成されるのか?もしくはなぜ正しく作成されないのか?

6. createは正常に完了するために4つの関数呼び出し(一つはialloc、他の三つはdirlink)を必要とする。
そうでない場合、createはpanicを呼ぶ。
なぜこれは許容できるのか?
なぜ、それらの4つの関数呼び出しは失敗し得ないのだろうか?

7. sys_chdirはiput(cp->cwd)の前にiunlock(ip)を呼び、iput(cp->cwd)はcp->cwdをロックしようとする。
iputの後のiunlock(ip)はデッドロックを引き起こさないが、それは何故か?

作業

1. について

見ての通り、ballocは空いてるブロックを探して返すので、空きブロックがなければpanicまで到達します。
panicじゃなくて呼び出し元に例外を通知するような仕組みにするとpanicは回避出来るかもしれませんが、どちらにしろ無いものを魔法の力で捻出することは出来ないので回復は無理だと思います。

fs.cのballoc関数

// Allocate a zeroed disk block.<br />
static uint<br />
balloc(uint dev)<br />
{<br />
  int b, bi, m;<br />
  struct buf *bp;<br />
  struct superblock sb;</p>
<p>  bp = 0;<br />
  readsb(dev, &amp;sb);<br />
  for(b = 0; b &lt; sb.size; b += BPB){<br />
    bp = bread(dev, BBLOCK(b, sb.ninodes));<br />
    for(bi = 0; bi &lt; BPB &amp;&amp; bi &lt; (sb.size - b); bi++){<br />
      m = 1 &lt;&lt; (bi % 8);<br />
      if((bp-&gt;data[bi/8] &amp; m) == 0){  // Is block free?<br />
        bp-&gt;data[bi/8] |= m;  // Mark block in use on disk.<br />
        log_write(bp);<br />
        brelse(bp);<br />
        bzero(dev, b + bi);<br />
        return b + bi;<br />
      }<br />
    }<br />
    brelse(bp);<br />
  }<br />
  panic(&quot;balloc: out of blocks&quot;);<br />
}

2. について

これも1と同じだと思います。

fs.cのialloc関数

// Allocate a new inode with the given type on device dev.<br />
struct inode*<br />
ialloc(uint dev, short type)<br />
{<br />
  int inum;<br />
  struct buf *bp;<br />
  struct dinode *dip;<br />
  struct superblock sb;</p>
<p>  readsb(dev, &amp;sb);<br />
  for(inum = 1; inum &lt; sb.ninodes; inum++){  // loop over inode blocks<br />
    bp = bread(dev, IBLOCK(inum));<br />
    dip = (struct dinode*)bp-&gt;data + inum%IPB;<br />
    if(dip-&gt;type == 0){  // a free inode<br />
      memset(dip, 0, sizeof(*dip));<br />
      dip-&gt;type = type;<br />
      log_write(bp);   // mark it allocated on the disk<br />
      brelse(bp);<br />
      return iget(dev, inum);<br />
    }<br />
    brelse(bp);<br />
  }<br />
  panic(&quot;ialloc: no inodes&quot;);<br />
}

3. について

元の文もinode generation numbers.だけなのでちょっと課題の意図が分かりませんが、inode世代番号を実装するにはどうしたら良いか?という事だと解釈して考えてみます。
まずinode世代番号というのは、あるinodeが指し示すデータが変更されたタイミングでインクリメントされる値です。(多分…)
これはinodeの変更状況を追跡するのに使えて、その値を元に割り当てのアルゴリズムなんかも改良出来たりするんじゃないかななんて思います。(想像)
inode世代番号なんて言葉正直今日知りました。
で、inode世代番号の実装方法ですが、まずinode構造体とdinode構造体に世代番号のフィールドを追加する必要があるでしょう。
dinodeのほうには追加せずに、例えばスーパーブロックとかにまとめて記録するのもありかもしれません。
そしたら、fs.cのwritei関数でその世代番号をインクリメントしてあげればいいと思いますが、ジャーナリングのために実際はwriteiではなくそこから呼ばれるlog_writeで書き込みが行われるので、そちらで実装する必要があると思います。
実際の実装レベルであまり細かいところまで考えるとロックとかでドツボにハマリそうなのでこの程度で勘弁していただきたく…

4. について

ballocやiallocは、ブロックやinode単位の割り当てであり、そのレベルで失敗してもユーザプログラムは正直何も出来ないはず(何か出来るということはカーネルがやってることより難しいことをしなきゃならないわけで…)なので、panicせずユーザプログラムに事後処理の責任を負わせるのは現実的じゃないと思います。
一方、fileallocが失敗するのは、システム全体で同時に開けるファイルの最大数(xv6の場合はたったの100)に達したときであって、前者の失敗の深刻度と比べたら、かなりヌルい失敗と言えます。
ユーザプログラムも、fileallocが失敗したら、必要なファイルが開けなかったとして単に終了するとか、開けるようになるまで、それまで開いてて必要のないファイルを閉じるとかの事後処理が行え、その実装はプログラムの内容にもよりますが基本的にはそう難しくないはずです。
なので、このレベルの失敗でいちいちpanicするよりは、ユーザプログラムにその後の処理を任せた方が現実的かなと思います。

file.cのfilealloc関数

// Allocate a file structure.<br />
struct file*<br />
filealloc(void)<br />
{<br />
  struct file *f;</p>
<p>  acquire(&amp;ftable.lock);<br />
  for(f = ftable.file; f &lt; ftable.file + NFILE; f++){<br />
    if(f-&gt;ref == 0){<br />
      f-&gt;ref = 1;<br />
      release(&amp;ftable.lock);<br />
      return f;<br />
    }<br />
  }<br />
  release(&amp;ftable.lock);<br />
  return 0;<br />
}

5. について

質問の答えとしては、「正しく作成されない。なぜならdirlinkが失敗するので、badラベル以下の行が実行され巻き戻されるので。」ということになるかと思います。
トランザクションを使ってるので、中途半端な状態で終わることはないので問題ないはずです。

sysfile.cのsys_link関数

// Create the path new as a link to the same inode as old.<br />
int<br />
sys_link(void)<br />
{<br />
  char name[DIRSIZ], *new, *old;<br />
  struct inode *dp, *ip;</p>
<p>  if(argstr(0, &amp;old) &lt; 0 || argstr(1, &amp;new) &lt; 0)<br />
    return -1;<br />
  if((ip = namei(old)) == 0)<br />
    return -1;</p>
<p>  begin_trans();</p>
<p>  ilock(ip);<br />
  if(ip-&gt;type == T_DIR){<br />
    iunlockput(ip);<br />
    commit_trans();<br />
    return -1;<br />
  }</p>
<p>  ip-&gt;nlink++;<br />
  iupdate(ip);<br />
  iunlock(ip);</p>
<p>  if((dp = nameiparent(new, name)) == 0)<br />
    goto bad;<br />
  ilock(dp);<br />
  if(dp-&gt;dev != ip-&gt;dev || dirlink(dp, name, ip-&gt;inum) &lt; 0){<br />
    iunlockput(dp);<br />
    goto bad;<br />
  }<br />
  iunlockput(dp);<br />
  iput(ip);</p>
<p>  commit_trans();</p>
<p>  return 0;</p>
<p>bad:<br />
  ilock(ip);<br />
  ip-&gt;nlink--;<br />
  iupdate(ip);<br />
  iunlockput(ip);<br />
  commit_trans();<br />
  return -1;<br />
}

6. について

まず、dirlinkが失敗(panicではなく)するパターンというのは、対象のディレクトリにすでに同じ名前が登録されているときだけです。
3つのdirlinkのうち2つは、新しく作成されたディレクトリに対して”.”と”..”を作成する処理であり、上記の理由から失敗しようがないはずです。
また3つめのdirlinkは、新しく作成したディレクトリ、もしくはファイルを親ディレクトリに登録するためのものですが、すでに親ディレクトリに同じ名前が登録されている場合は、前半のif((ip = dirlookup(dp, name, &off)) != 0)の部分によって除外されるので、3つめのdirlinkは失敗しないことになります。
当然、一連の処理中はdp(親ディレクトリのinode)がロックされているので、途中で他のプロセスで変更されることを考慮する必要もありません。

sysfile.cのcreate関数

static struct inode*<br />
create(char *path, short type, short major, short minor)<br />
{<br />
  uint off;<br />
  struct inode *ip, *dp;<br />
  char name[DIRSIZ];</p>
<p>  if((dp = nameiparent(path, name)) == 0)<br />
    return 0;<br />
  ilock(dp);</p>
<p>  if((ip = dirlookup(dp, name, &amp;off)) != 0){<br />
    iunlockput(dp);<br />
    ilock(ip);<br />
    if(type == T_FILE &amp;&amp; ip-&gt;type == T_FILE)<br />
      return ip;<br />
    iunlockput(ip);<br />
    return 0;<br />
  }</p>
<p>  if((ip = ialloc(dp-&gt;dev, type)) == 0)<br />
    panic(&quot;create: ialloc&quot;);</p>
<p>  ilock(ip);<br />
  ip-&gt;major = major;<br />
  ip-&gt;minor = minor;<br />
  ip-&gt;nlink = 1;<br />
  iupdate(ip);</p>
<p>  if(type == T_DIR){  // Create . and .. entries.<br />
    dp-&gt;nlink++;  // for &quot;..&quot;<br />
    iupdate(dp);<br />
    // No ip-&gt;nlink++ for &quot;.&quot;: avoid cyclic ref count.<br />
    if(dirlink(ip, &quot;.&quot;, ip-&gt;inum) &lt; 0 || dirlink(ip, &quot;..&quot;, dp-&gt;inum) &lt; 0)<br />
      panic(&quot;create dots&quot;);<br />
  }</p>
<p>  if(dirlink(dp, name, ip-&gt;inum) &lt; 0)<br />
    panic(&quot;create: dirlink&quot;);</p>
<p>  iunlockput(dp);</p>
<p>  return ip;<br />
}

7. について

本文でははiput(cp->cwd)となってますが、ソースではiput(proc->cwd)となっています。
また、ソースではiputでiunlockは呼ばれません。
ソースのログを辿ったところ、この課題は今のソースではなく古いソースに対するもののようです。
古いソースでは、iputはロック中のinodeを要求するようになってました。(現在は未ロックのinodeを要求する。)
iputだけでなく関連の関数の実装も今と大きく違うみたいなので、この課題はパスします。

sysfile.cのsys_chdir関数

int<br />
sys_chdir(void)<br />
{<br />
  char *path;<br />
  struct inode *ip;</p>
<p>  if(argstr(0, &amp;path) &lt; 0 || (ip = namei(path)) == 0)<br />
    return -1;<br />
  ilock(ip);<br />
  if(ip-&gt;type != T_DIR){<br />
    iunlockput(ip);<br />
    return -1;<br />
  }<br />
  iunlock(ip);<br />
  iput(proc-&gt;cwd);<br />
  proc-&gt;cwd = ip;<br />
  return 0;<br />
}

感想

今までのExercisesの回の中では一番まともに対応できたんじゃないかと思います。
まぁ思い込みで嘘書いてる可能性も大ですが…

一応今回で本文は終わりです。
長いようで短かったです。
興味深く読めたので、ここまで続けれたんだと思います。

あとは、付録A, Bが残ってますのでぼちぼちそちらも読んでいきたいと思います。

コメント

こんにちは。
オフライン端末や印刷したものを読みたいので、これらのページをPDFで配布して欲しいです。

コメントありがとうございます。

PDF化しました。
こちらからzipで一括ダウンロードできます。
フォーマット等気に入らなければ勝手にPDF化していただいて構いません。
その他の注意点はこちら

いきなり不躾な要望ですみませんでした。
それにも関わらず、心良く応じてくださりありがとうございます。
「英語のドキュメントばかりだと、時間がある時に本腰を入れて
読まないといけないな…(→結局読まない)」という方は私だけではなく、
たくさんいらっしゃると思います。
xv6の学習の間口を広げていただき、ありがとうございます。
早速読ませていただきます!

コメントを残す

メールアドレスが公開されることはありません。



※画像をクリックして別の画像を表示

このサイトはスパムを低減するために Akismet を使っています。コメントデータの処理方法の詳細はこちらをご覧ください