This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "XFS development tree". The branch, master has been updated e89318c xfs: eliminate some newly-reported gcc warnings a4874ad xfs: remove the ->kill_root btree operation fc4b731f xfs: stop using xfs_qm_dqtobp in xfs_qm_dqflush 75a93d4 xfs: simplify xfs_qm_dqusage_adjust 51749e4 xfs: log IO completion workqueue is a high priority queue a122eb2 xfs: prevent reading uninitialized stack memory from 0e251465b06b75dfed16b9373c25cce85eeda484 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- commit e89318c670af3959db3aa483da509565f5a2536c Author: Poyo VL <poyo_vl@xxxxxxxxx> Date: Thu Sep 2 07:41:55 2010 +0000 xfs: eliminate some newly-reported gcc warnings Ionut Gabriel Popescu <poyo_vl@xxxxxxxxx> submitted a simple change to eliminate some "may be used uninitialized" warnings when building XFS. The reported condition seems to be something that GCC did not used to recognize or report. The warnings were produced by: gcc version 4.5.0 20100604 [gcc-4_5-branch revision 160292] (SUSE Linux) Signed-off-by: Ionut Gabriel Popescu <poyo_vl@xxxxxxxxx> Signed-off-by: Alex Elder <aelder@xxxxxxx> commit a4874ada8f44cb7cca9fcad65a3c6a09fc9f1ccd Author: Christoph Hellwig <hch@xxxxxxxxxxxxx> Date: Tue Sep 7 23:34:07 2010 +0000 xfs: remove the ->kill_root btree operation The implementation os ->kill_root only differ by either simply zeroing out the now unused buffer in the btree cursor in the inode allocation btree or using xfs_btree_setbuf in the allocation btree. Initially both of them used xfs_btree_setbuf, but the use in the ialloc btree was removed early on because it interacted badly with xfs_trans_binval. In addition to zeroing out the buffer in the cursor xfs_btree_setbuf updates the bc_ra array in the btree cursor, and calls xfs_trans_brelse on the buffer previous occupying the slot. The bc_ra update should be done for the alloc btree updated too, although the lack of it does not cause serious problems. The xfs_trans_brelse call on the other hand is effectively a no-op in the end - it keeps decrementing the bli_recur refcount until it hits zero, and then just skips out because the buffer will always be dirty at this point. So removing it for the allocation btree is just fine. So unify the code and move it to xfs_btree.c. While we're at it also replace the call to xfs_btree_setbuf with a NULL bp argument in xfs_btree_del_cursor with a direct call to xfs_trans_brelse given that the cursor is beeing freed just after this and the state updates are superflous. After this xfs_btree_setbuf is only used with a non-NULL bp argument and can thus be simplified. Signed-off-by: Christoph Hellwig <hch@xxxxxx> Signed-off-by: Alex Elder <aelder@xxxxxxx> commit fc4b731f473ff5ec3a3b0d03bf7c60870a7e71f1 Author: Christoph Hellwig <hch@xxxxxxxxxxxxx> Date: Mon Sep 6 01:44:45 2010 +0000 xfs: stop using xfs_qm_dqtobp in xfs_qm_dqflush In xfs_qm_dqflush we know that q_blkno must be initialized already from a previous xfs_qm_dqread. So instead of calling xfs_qm_dqtobp we can simply read the quota buffer directly. This also saves us from a duplicate xfs_qm_dqcheck call check and allows xfs_qm_dqtobp to be simplified now that it is always called for a newly initialized inode. In addition to that properly unwind all locks in xfs_qm_dqflush when xfs_qm_dqcheck fails. This mirrors a similar cleanup in the inode lookup done earlier. Signed-off-by: Christoph Hellwig <hch@xxxxxx> Signed-off-by: Alex Elder <aelder@xxxxxxx> commit 75a93d4130e457fb67a348d5b96ef2cfe3192722 Author: Christoph Hellwig <hch@xxxxxx> Date: Mon Sep 6 01:44:22 2010 +0000 xfs: simplify xfs_qm_dqusage_adjust There is no need to have the users and group/project quota locked at the same time. Get rid of xfs_qm_dqget_noattach and just do a xfs_qm_dqget inside xfs_qm_quotacheck_dqadjust for the quota we are operating on right now. The new version of xfs_qm_quotacheck_dqadjust holds the inode lock over it's operations, which is not a problem as it simply increments counters and there is no concern about log contention during mount time. Signed-off-by: Christoph Hellwig <hch@xxxxxx> Signed-off-by: Alex Elder <aelder@xxxxxxx> ----------------------------------------------------------------------- Summary of changes: fs/xfs/quota/xfs_dquot.c | 164 +++++++++++++++++------------------- fs/xfs/quota/xfs_qm.c | 203 ++++++++++++++------------------------------- fs/xfs/xfs_alloc.c | 4 +- fs/xfs/xfs_alloc_btree.c | 33 ------- fs/xfs/xfs_btree.c | 52 ++++++++++-- fs/xfs/xfs_btree.h | 14 +--- fs/xfs/xfs_ialloc_btree.c | 33 ------- 7 files changed, 183 insertions(+), 320 deletions(-) hooks/post-receive -- XFS development tree _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs