--- deliantra/Deliantra-Client/DC/DB.pm	2007/08/28 20:07:18	1.20
+++ deliantra/Deliantra-Client/DC/DB.pm	2008/03/30 06:05:52	1.35
@@ -1,10 +1,10 @@
 =head1 NAME
 
-CFPlus::DB - async. database and filesystem access for cfplus
+DC::DB - async. database and filesystem access for deliantra
 
 =head1 SYNOPSIS
 
- use CFPlus::DB;
+ use DC::DB;
 
 =head1 DESCRIPTION
 
@@ -12,137 +12,214 @@
 
 =cut
 
-package CFPlus::DB;
+package DC::DB;
 
 use strict;
 use utf8;
 
+use File::Path ();
 use Carp ();
 use Storable ();
 use Config;
-use Event ();
+use BDB;
 
-use CFPlus;
+use DC;
 
-our $DB_HOME = "$Crossfire::VARDIR/cfplus-$BerkeleyDB::db_version-$Config{archname}";
+our $ODBDIR  = "cfplus-" . BDB::VERSION . "-$Config{archname}";
+our $DBDIR   = "client-" . BDB::VERSION . "-$Config{archname}";
+our $DB_HOME = "$Deliantra::VARDIR/$DBDIR";
+
+unless (-d $DB_HOME) {
+   if (-d "$Deliantra::VARDIR/$ODBDIR") {
+      rename "$Deliantra::VARDIR/$ODBDIR", $DB_HOME;
+      print STDERR "INFO: moved old database from $Deliantra::VARDIR/$ODBDIR to $DB_HOME\n";
+   } elsif (-d "$Deliantra::OLDDIR/$ODBDIR") {
+      rename "$Deliantra::OLDDIR/$DBDIR", $DB_HOME;
+      print STDERR "INFO: moved old database from $Deliantra::OLDDIR/$ODBDIR to $DB_HOME\n";
+   } else {
+      File::Path::mkpath [$DB_HOME]
+         or die "unable to create database directory $DB_HOME: $!";
+   }
+}
 
-sub path_of_res($) {
-   utf8::downgrade $_[0]; # bug in unpack "H*"
-   "$DB_HOME/res-data-" . unpack "H*", $_[0]
+BDB::max_poll_time 0.03;
+BDB::max_parallel 1;
+
+our $DB_ENV;
+our $DB_STATE;
+our %DB_TABLE;
+our $TILE_SEQ;
+
+sub try_open_db {
+   File::Path::mkpath [$DB_HOME];
+
+   $DB_ENV = db_env_create;
+
+   $DB_ENV->set_errfile (\*STDERR);
+   $DB_ENV->set_msgfile (\*STDERR);
+   $DB_ENV->set_verbose (-1, 1);
+
+   $DB_ENV->set_flags (BDB::AUTO_COMMIT | BDB::LOG_AUTOREMOVE | BDB::TXN_WRITE_NOSYNC);
+   $DB_ENV->set_cachesize (0, 2048 * 1024, 0);
+
+   db_env_open $DB_ENV, $DB_HOME,
+               BDB::CREATE | BDB::REGISTER | BDB::RECOVER | BDB::INIT_MPOOL | BDB::INIT_LOCK | BDB::INIT_TXN,
+               0666;
+
+   $! and die "cannot open database environment $DB_HOME: " . BDB::strerror;
+
+   1
 }
 
-sub sync {
-   # for debugging
-   #CFPlus::DB::Server::req (sync => sub { });
-   CFPlus::DB::Server::sync ();
+sub table($) {
+   $DB_TABLE{$_[0]} ||= do {
+      my ($table) = @_;
+
+      $table =~ s/([^a-zA-Z0-9_\-])/sprintf "=%x=", ord $1/ge;
+
+      my $db = db_create $DB_ENV;
+      $db->set_flags (BDB::CHKSUM);
+
+      db_open $db, undef, $table, undef, BDB::BTREE,
+              BDB::AUTO_COMMIT | BDB::CREATE | BDB::READ_UNCOMMITTED, 0666;
+
+      $! and "unable to open/create database table $_[0]: ". BDB::strerror;
+
+      $db
+   }
 }
 
+#############################################################################
+
+our $WATCHER;
+our $SYNC;
+our $facemap;
+
 sub exists($$$) {
-   CFPlus::DB::Server::req (exists => @_);
+   my ($db, $key, $cb) = @_;
+
+   my $data;
+   db_get table $db, undef, $key, $data, 0, sub {
+      $cb->($! ? () : length $data);
+   };
 }
 
 sub get($$$) {
-   CFPlus::DB::Server::req (get => @_);
+   my ($db, $key, $cb) = @_;
+
+   my $data;
+   db_get table $db, undef, $key, $data, 0, sub {
+      $cb->($! ? () : $data);
+   };
 }
 
 sub put($$$$) {
-   CFPlus::DB::Server::req (put => @_);
-}
+   my ($db, $key, $data, $cb) = @_;
 
-sub unlink($$) {
-   CFPlus::DB::Server::req (unlink => @_);
+   db_put table $db, undef, $key, $data, 0, sub {
+      $cb->($!);
+      $SYNC->again unless $SYNC->is_active;
+   };
 }
 
-sub read_file($$) {
-   CFPlus::DB::Server::req (read_file => @_);
-}
+sub do_table {
+   my ($db, $cb) = @_;
 
-sub write_file($$$) {
-   CFPlus::DB::Server::req (write_file => @_);
-}
+   $db = table $db;
 
-sub prefetch_file($$$) {
-   CFPlus::DB::Server::req (prefetch_file => @_);
-}
+   my $cursor = $db->cursor;
+   my %kv;
 
-sub logprint($$$) {
-   CFPlus::DB::Server::req (logprint => @_);
+   for (;;) {
+      db_c_get $cursor, my $k, my $v, BDB::NEXT;
+      last if $!;
+      $kv{$k} = $v;
+   }
+
+   $cb->(\%kv);
 }
 
-our $tilemap;
+sub do_get_tile_id {
+   my ($name, $cb) = @_;
 
-sub get_tile_id_sync($) {
-   my ($name) = @_;
+   my $table = table "facemap";
+   my $id;
 
-   # fetch the full face table first
-   unless ($tilemap) {
-      CFPlus::DB::Server::req (table => facemap => sub {
-         $tilemap = $_[0];
-         delete $tilemap->{id};
-         my %maptile = reverse %$tilemap;#d#
-         if ((scalar keys %$tilemap) != (scalar keys %maptile)) {#d#
-            $tilemap = { };#d#
-            CFPlus::error "FATAL: facemap is not a 1:1 mapping, please report this and delete your $DB_HOME directory!\n";#d#
-         }#d#
-      });
-      sync;
+   db_get $table, undef, $name => $id, 0;
+   $! or return $cb->($id);
+
+   unless ($TILE_SEQ) {
+      $TILE_SEQ = $table->sequence;
+      $TILE_SEQ->initial_value (64);
+      $TILE_SEQ->set_cachesize (0);
+      db_sequence_open $TILE_SEQ, undef, "id", BDB::CREATE;
    }
 
-   $tilemap->{$name} ||= do {
+   db_sequence_get $TILE_SEQ, undef, 1, my $id;
+
+   die "unable to allocate tile id: $!"
+      if $!;
+   
+   db_put $table, undef, $name => $id, 0;
+   $cb->($id);
+
+}
+
+sub get_tile_id_sync($) {
+   my ($name) = @_;
+
+   $facemap->{$name} ||= do {
       my $id;
-      CFPlus::DB::Server::req (get_tile_id => $name, sub { $id = $_[0] });
-      sync;
+      do_get_tile_id $name, sub {
+         $id = $_[0];
+      };
+      BDB::flush;
       $id
    }
 }
 
-package CFPlus::DB::Server;
-
-use strict;
+#############################################################################
 
-use Fcntl;
-use BerkeleyDB;
+sub path_of_res($) {
+   utf8::downgrade $_[0]; # bug in unpack "H*"
+   "$DB_HOME/res-data-" . unpack "H*", $_[0]
+}
 
-our $DB_ENV;
-our $DB_STATE;
-our %DB_TABLE;
+sub sync {
+   # for debugging
+   #DC::DB::Server::req (sync => sub { });
+   DC::DB::Server::sync ();
+}
 
-sub open_db {
-   mkdir $DB_HOME, 0777;
-   my $recover = $BerkeleyDB::db_version >= 4.4 
-                 ? eval "DB_REGISTER | DB_RECOVER"
-                 : 0;
-
-   $DB_ENV = new BerkeleyDB::Env
-                    -Home => $DB_HOME,
-                    -Cachesize => 8_000_000,
-                    -ErrFile => "$DB_HOME/errorlog.txt",
-#                 -ErrPrefix => "DATABASE",
-                    -Verbose => 1,
-                    -Flags => DB_CREATE | DB_RECOVER | DB_INIT_MPOOL | DB_INIT_LOCK | DB_INIT_TXN | $recover,
-                    -SetFlags => DB_AUTO_COMMIT | DB_LOG_AUTOREMOVE | DB_TXN_WRITE_NOSYNC,
-                       or die "unable to create/open database home $DB_HOME: $BerkeleyDB::Error";
+sub unlink($$) {
+   DC::DB::Server::req (unlink => @_);
+}
 
-   1
+sub read_file($$) {
+   DC::DB::Server::req (read_file => @_);
 }
 
-sub table($) {
-   $DB_TABLE{$_[0]} ||= do {
-      my ($table) = @_;
+sub write_file($$$) {
+   DC::DB::Server::req (write_file => @_);
+}
 
-      $table =~ s/([^a-zA-Z0-9_\-])/sprintf "=%x=", ord $1/ge;
+sub prefetch_file($$$) {
+   DC::DB::Server::req (prefetch_file => @_);
+}
 
-      new BerkeleyDB::Btree
-         -Env      => $DB_ENV,
-         -Filename => $table,
-#      -Filename => "database",
-#      -Subname  => $table,
-         -Property => DB_CHKSUM,
-         -Flags    => DB_AUTO_COMMIT | DB_CREATE | DB_UPGRADE,
-            or die "unable to create/open database table $_[0]: $BerkeleyDB::Error"
-   }
+sub logprint($$$) {
+   DC::DB::Server::req (logprint => @_);
 }
 
+#############################################################################
+
+package DC::DB::Server;
+
+use strict;
+
+use EV ();
+use Fcntl;
+
 our %CB;
 our $FH;
 our $ID = "aaa0";
@@ -151,11 +228,6 @@
 our $write_buf;
 our $read_buf;
 
-our $SYNC = Event->idle (min => 120, max => 180, parked => 1, cb => sub {
-   CFPlus::DB::Server::req (sync => sub { });
-   $_[0]->w->stop;
-});
-
 sub fh_write {
    my $len = syswrite $FH, $write_buf;
 
@@ -208,81 +280,6 @@
    $CB{$id} = $cb;
 
    $fh_w_watcher->start;
-   $SYNC->start;
-}
-
-sub do_sync {
-   $DB_ENV->txn_checkpoint (0, 0, 0);
-   ()
-}
-
-sub do_exists {
-   my ($db, $key) = @_;
-
-   utf8::downgrade $key;
-   my $data;
-   (table $db)->db_get ($key, $data) == 0
-      ? length $data
-      : ()
-}
-
-sub do_get {
-   my ($db, $key) = @_;
-
-   utf8::downgrade $key;
-   my $data;
-   (table $db)->db_get ($key, $data) == 0
-      ? $data
-      : ()
-}
-
-sub do_put {
-   my ($db, $key, $data) = @_;
-
-   utf8::downgrade $key;
-   utf8::downgrade $data;
-   (table $db)->db_put ($key => $data)
-}
-
-sub do_table {
-   my ($db) = @_;
-
-   $db = table $db;
-
-   my $cursor = $db->db_cursor;
-   my %kv;
-   my ($k, $v);
-   $kv{$k} = $v while $cursor->c_get ($k, $v, BerkeleyDB::DB_NEXT) == 0;
-
-   \%kv
-}
-
-sub do_get_tile_id {
-   my ($name) = @_;
-
-   my $id;
-   my $table = table "facemap";
-
-   return $id
-      if $table->db_get ($name, $id) == 0;
-
-   for (1..100) {
-      my $txn = $DB_ENV->txn_begin;
-      my $status = $table->db_get (id => $id);
-      if ($status == 0 || $status == BerkeleyDB::DB_NOTFOUND) {
-         $id = ($id || 64) + 1;
-         if ($table->db_put (id => $id) == 0
-             && $table->db_put ($name => $id) == 0) {
-            $txn->txn_commit;
-
-            return $id;
-         }
-      }
-      $txn->txn_abort;
-      select undef, undef, undef, 0.01 * rand;
-   }
-
-   die "maximum number of transaction retries reached - database problems?";
 }
 
 sub do_unlink {
@@ -347,7 +344,7 @@
 }
 
 sub run {
-   ($FH, my $fh) = CFPlus::socketpipe;
+   ($FH, my $fh) = DC::socketpipe;
 
    my $oldfh = select $FH; $| = 1; select $oldfh;
    my $oldfh = select $fh; $| = 1; select $oldfh;
@@ -355,17 +352,12 @@
    my $pid = fork;
    
    if (defined $pid && !$pid) {
-      local $SIG{QUIT};
+      local $SIG{QUIT} = "IGNORE";
       local $SIG{__DIE__};
       local $SIG{__WARN__};
       eval {
          close $FH;
 
-         unless (eval { open_db }) {
-            eval { File::Path::rmtree $DB_HOME };
-            open_db;
-         }
-
          while () {
             4 == read $fh, my $len, 4
                or last;
@@ -376,40 +368,79 @@
             $req = Storable::thaw $req;
 
             my ($id, $type, @args) = @$req;
-            my $cb = CFPlus::DB::Server->can ("do_$type")
+            my $cb = DC::DB::Server->can ("do_$type")
                or die "$type: unknown database request type\n";
             my $res = pack "N/a*", Storable::freeze [$id, $cb->(@args)];
             (syswrite $fh, $res) == length $res
-               or die;
+               or die "DB::write: $!";
          }
       };
 
       my $error = $@;
 
       eval {
-         undef %DB_TABLE;
-         undef $DB_ENV;
-
          Storable::store_fd [die => $error], $fh;
       };
 
-      CFPlus::_exit 0;
+      warn $error
+         if $error;
+
+      DC::_exit 0;
    }
 
    close $fh;
-   CFPlus::fh_nonblocking $FH, 1;
+   DC::fh_nonblocking $FH, 1;
 
    $CB{die} = sub { die shift };
 
-   $fh_r_watcher = Event->io (fd => $FH, poll => 'r', nice => 1, cb => \&fh_read);
-   $fh_w_watcher = Event->io (fd => $FH, poll => 'w', nice => -1, parked => 1, cb => \&fh_write);
-   $SYNC->start;
+   $fh_r_watcher = EV::io $FH, EV::READ , \&fh_read;
+   $fh_w_watcher = EV::io $FH, EV::WRITE, \&fh_write;
 }
 
 sub stop {
    close $FH;
 }
 
+package DC::DB;
+
+sub nuke_db {
+   File::Path::mkpath [$DB_HOME];
+   eval { File::Path::rmtree $DB_HOME };
+}
+
+sub open_db {
+   unless (eval { try_open_db }) {
+      warn "$@";#d#
+      eval { nuke_db };
+      try_open_db;
+   }
+
+   # fetch the full face table first
+   unless ($facemap) {
+      do_table facemap => sub {
+         $facemap = $_[0];
+         delete $facemap->{id};
+         my %maptile = reverse %$facemap;#d#
+         if ((scalar keys %$facemap) != (scalar keys %maptile)) {#d#
+            $facemap = { };#d#
+            DC::error "FATAL: facemap is not a 1:1 mapping, please report this and delete your $DB_HOME directory!\n";#d#
+         }#d#
+      };
+   }
+
+   $WATCHER = EV::io BDB::poll_fileno, EV::READ, \&BDB::poll_cb;
+   $SYNC = EV::timer_ns 0, 60, sub {
+      $_[0]->stop;
+      db_env_txn_checkpoint $DB_ENV, 0, 0, 0, sub { };
+   };
+}
+
+END {
+   undef $TILE_SEQ;
+   %DB_TABLE = ();
+   undef $DB_ENV;
+}
+
 1;
 
 =back