]> granicus.if.org Git - zfs/commitdiff
Merge commit 'refs/top-bases/gcc-unused' into gcc-unused
authorBrian Behlendorf <behlendorf1@llnl.gov>
Fri, 28 May 2010 22:07:36 +0000 (15:07 -0700)
committerBrian Behlendorf <behlendorf1@llnl.gov>
Fri, 28 May 2010 22:07:36 +0000 (15:07 -0700)
Conflicts:
module/zfs/dbuf.c
module/zfs/dmu.c
module/zfs/dmu_objset.c
module/zfs/dnode_sync.c
module/zfs/dsl_dataset.c
module/zfs/dsl_prop.c
module/zfs/metaslab.c
module/zfs/spa.c
module/zfs/vdev.c
module/zfs/zap_micro.c

13 files changed:
1  2 
cmd/ztest/ztest.c
module/zfs/arc.c
module/zfs/dbuf.c
module/zfs/dmu_objset.c
module/zfs/dnode.c
module/zfs/dnode_sync.c
module/zfs/dsl_dataset.c
module/zfs/dsl_dir.c
module/zfs/spa.c
module/zfs/vdev.c
module/zfs/vdev_cache.c
module/zfs/zap_micro.c
module/zfs/zio.c

Simple merge
Simple merge
index 86a6287f35542248d49e1601c12e9930095a407c,42ae439972e4ce25ebd2ef38692af468bca7e0ac..273bae9c632e60fb335fd0fa10c12b577e6c72b7
@@@ -333,12 -341,12 +341,13 @@@ dbuf_verify(dmu_buf_impl_t *db
                                ASSERT(db->db_parent == NULL);
                        else
                                ASSERT(db->db_parent != NULL);
-                       ASSERT3P(db->db_blkptr, ==,
-                           &dn->dn_phys->dn_blkptr[db->db_blkid]);
+                       if (db->db_blkid != DMU_SPILL_BLKID)
+                               ASSERT3P(db->db_blkptr, ==,
+                                   &dn->dn_phys->dn_blkptr[db->db_blkid]);
                } else {
                        /* db is pointed to by an indirect block */
 -                      int epb = db->db_parent->db.db_size >> SPA_BLKPTRSHIFT;
 +                      ASSERTV(int epb = db->db_parent->db.db_size >>
 +                              SPA_BLKPTRSHIFT);
                        ASSERT3U(db->db_parent->db_level, ==, db->db_level+1);
                        ASSERT3U(db->db_parent->db.db_object, ==,
                            db->db.db_object);
index 8aad15bd7ece3103a5ca7535c772b94daa579e8e,690e6ecdee6ab84e5611ee49ca8ca295fb25c97a..3aea0f321ec67abee05c21818301aac800834db3
@@@ -888,12 -949,11 +949,12 @@@ dmu_objset_sync_dnodes(list_t *list, li
  
  /* ARGSUSED */
  static void
- ready(zio_t *zio, arc_buf_t *abuf, void *arg)
dmu_objset_write_ready(zio_t *zio, arc_buf_t *abuf, void *arg)
  {
        blkptr_t *bp = zio->io_bp;
-       objset_impl_t *os = arg;
+       objset_t *os = arg;
        dnode_phys_t *dnp = &os->os_phys->os_meta_dnode;
 +      ASSERTV(blkptr_t *bp_orig = &zio->io_bp_orig);
  
        ASSERT(bp == os->os_rootbp);
        ASSERT(BP_GET_TYPE(bp) == DMU_OT_OBJSET);
Simple merge
index 383eac854397290d9c6aba9fcefc2e097489181c,f9ec9f6023d2abc3c4feb63c2111756cd2ce58b9..afad0f576fc3e7af4e136966456c8ca19c786fc3
@@@ -512,7 -511,8 +511,8 @@@ dnode_sync(dnode_t *dn, dmu_tx_t *tx
        dnode_phys_t *dnp = dn->dn_phys;
        int txgoff = tx->tx_txg & TXG_MASK;
        list_t *list = &dn->dn_dirty_records[txgoff];
 -      static const dnode_phys_t zerodn = { 0 };
+       boolean_t kill_spill = B_FALSE;
 +      ASSERTV(static const dnode_phys_t zerodn = { 0 });
  
        ASSERT(dmu_tx_is_syncing(tx));
        ASSERT(dnp->dn_type != DMU_OT_NONE || dn->dn_allocated_txg);
index 45ddfd8f3040df2783779769111a637f6c99e629,ddd83576c65e8fbc69cb6b6eba180bc89cc0f66f..53413ef194fe7aab17fabd67f6dfff4884cce795
@@@ -1097,9 -1094,9 +1094,8 @@@ dsl_dataset_destroy(dsl_dataset_t *ds, 
         * context, the user space accounting should be zero.
         */
        if (ds->ds_phys->ds_bp.blk_fill == 0 &&
-           dmu_objset_userused_enabled(os->os)) {
+           dmu_objset_userused_enabled(os)) {
 -              uint64_t count;
 -
 +              ASSERTV(uint64_t count);
                ASSERT(zap_count(os, DMU_USERUSED_OBJECT, &count) != 0 ||
                    count == 0);
                ASSERT(zap_count(os, DMU_GROUPUSED_OBJECT, &count) != 0 ||
@@@ -1959,11 -1883,11 +1882,11 @@@ dsl_dataset_destroy_sync(void *arg1, vo
                dsl_dataset_rele(ds_prev, FTAG);
  
        spa_prop_clear_bootfs(dp->dp_spa, ds->ds_object, tx);
-       spa_history_internal_log(LOG_DS_DESTROY, dp->dp_spa, tx,
-           cr, "dataset = %llu", ds->ds_object);
+       spa_history_log_internal(LOG_DS_DESTROY, dp->dp_spa, tx,
+           "dataset = %llu", ds->ds_object);
  
        if (ds->ds_phys->ds_next_clones_obj != 0) {
 -              uint64_t count;
 +              ASSERTV(uint64_t count);
                ASSERT(0 == zap_count(mos,
                    ds->ds_phys->ds_next_clones_obj, &count) && count == 0);
                VERIFY(0 == dmu_object_free(mos,
Simple merge
index d45e070eafa4bf3291367bf08e0d08e0d7f2bbd2,d7c5de0d357a2fab7e3a281ed52196aab97519c8..4243c4842f6bee281c73b8597038c7e68c30b42e
@@@ -2997,8 -3645,8 +3645,8 @@@ spa_vdev_add(spa_t *spa, nvlist_t *nvro
  int
  spa_vdev_attach(spa_t *spa, uint64_t guid, nvlist_t *nvroot, int replacing)
  {
-       uint64_t txg, open_txg;
+       uint64_t txg, dtl_max_txg;
 -      vdev_t *rvd = spa->spa_root_vdev;
 +      ASSERTV(vdev_t *rvd = spa->spa_root_vdev;)
        vdev_t *oldvd, *newvd, *newrootvd, *pvd, *tvd;
        vdev_ops_t *pvops;
        char *oldvdpath, *newvdpath;
index 2189297c63bf4b0f8735352e939d65dc6e2b25ba,a61f29b8e78a6deb58690a813aeed9ccee715ad1..ad88f8371942edd2688a67b828f77642e93c0cec
@@@ -1274,7 -1401,8 +1401,8 @@@ vdev_validate(vdev_t *vd
  void
  vdev_close(vdev_t *vd)
  {
 -      spa_t *spa = vd->vdev_spa;
 +      ASSERTV(spa_t *spa = vd->vdev_spa);
+       vdev_t *pvd = vd->vdev_parent;
  
        ASSERT(spa_config_held(spa, SCL_STATE_ALL, RW_WRITER) == SCL_STATE_ALL);
  
Simple merge
index 7a6b1e883b8bd6f6541a61e2a2da642b83ad87c8,2d89c20c47d7ba8b14aefdc6387f4459a529f7b4..52a7e4f57c1fb0e7389d626b225df8a6d2227bf2
@@@ -747,8 -937,8 +937,8 @@@ mzap_addent(zap_name_t *zn, uint64_t va
  
  #ifdef ZFS_DEBUG
        for (i = 0; i < zap->zap_m.zap_num_chunks; i++) {
 -              mzap_ent_phys_t *mze = &zap->zap_m.zap_phys->mz_chunk[i];
 +              ASSERTV(mzap_ent_phys_t *mze=&zap->zap_m.zap_phys->mz_chunk[i]);
-               ASSERT(strcmp(zn->zn_name_orij, mze->mze_name) != 0);
+               ASSERT(strcmp(zn->zn_key_orig, mze->mze_name) != 0);
        }
  #endif
  
Simple merge