Meeting notes for 2013 - Revision history

Ben Myers at 14:40, 30 August 2013

2013-08-30T14:40:54Z

← Older revision		Revision as of 14:40, 30 August 2013
Line 1:		Line 1:
			== Meeting of Aug 29 ==

			Attending: Brian, Eric, Mark, Andrew, Troy, Ben, Rich, Phil, Geoffrey

			3.12 content:

			Remaining patches from last meeting (although we didn't discuss them in this meeting):
			* From Chandra Seetharaman - [3/3] gfs2: Add support for the Q_XGETQSTATV (Waiting from Ack from Steven Whitehouse)
			* From James Dingwall - Cleancache support in XFS (Although this is a one liner, this needs serious review)
			* From Jan Kara - two direct-io patches (need review)

			Patches we discussed during this meeting:
			* From David Chinner - three patches that fix regressions in commit 00574da1 (to be pulled in)
			* From Brian Foster - xfs: check correct status variable for xfs_inobt_get_rec() call (ready to be pulled in)
			* From David Chinner - xfs: inode log reservations are too small (Mark noted that this fixes some asserts, it's ready to be pulled in)
			* From David Chinner - xfs: finish removing IOP_* macros. (ready to be pulled in)
			* From David Chinner - xfs: prevent transient corrupt states during log recovery (Ben was reluctant to pull this in for 3.12, but Eric wanted it, and Mark said it's ready, so it will be pulled in for 3.12)
			* From David Chinner - XFS: Assertion failed: first <= last && last < BBTOB(bp->b_length), file: fs/xfs/xfs_trans_buf.c, line: 568
			This is a directory bug found by Brian. Mark thinks that there are actually two bugs here, this fixes one. Pull this in for 3.12.
			* From Eric Sandeen - Fix wrong flag ASSERT in xfs_attr_shortform_getvalue (Ben to pull this in)

			Other items:
			* Mark expressed some concerns about being able to turn crc calculation off in some situations, and there was some discussion of how it might be done.
			* Ben noted that Chandra asked about his quotactl patch, there may be further discussion on what Christoph wants done, and this might be work that needs to go in past -rc1.
			* Eric discussed the remaining issues related to the crc work. Mark noted that the kernel work is mostly pulled in. Ben noted that the biggest remaining piece is uerspace.

			Discussion of xfstests:
			* Eric and Rich discussed the merits of two methods of breaking up the xfstests development trees by filesystem:
			1) centralized tree with multiple committers
			2) distributed trees with pull requests
			* Rich to discuss it further with Troy.


	== Meeting of Aug 22 ==		== Meeting of Aug 22 ==

Ben Myers: /* Meeting of Aug 22 */

2013-08-22T20:45:55Z

Meeting of Aug 22

← Older revision		Revision as of 20:45, 22 August 2013
Line 16:		Line 16:
	Userspace:		Userspace:
	* BPM to update his patch for the di_pad regression.		* BPM to update his patch for the di_pad regression.
	* Rich mentioned that eric submitted a fix for a bug that was reported.		* Rich mentioned that eric submitted a fix for a bug that was reported, pulled these in:
			xfs_repair: zero out unused parts of superblocks
			xfs_repair: add prototype for alloc_ex_data()
	* Mark to repost his SEEK_DATA/SEEK_HOLE patch		* Mark to repost his SEEK_DATA/SEEK_HOLE patch
	* Rich pulled in a couple of one liner fixes for bugs that were detected by coverity.		* Rich pulled in a couple of one liner fixes for bugs that were detected by coverity.

Ben Myers at 20:39, 22 August 2013

2013-08-22T20:39:48Z

← Older revision		Revision as of 20:39, 22 August 2013
Line 1:		Line 1:
			== Meeting of Aug 22 ==

			Attending: Brian, Carlos, Troy, Rich, Geoffrey, Mark, Andrew, Ben

			3.12 content:

			Remaining patches:
			* From Jan Kara - two direct-io patches (need review)
			* From Jeff Liu - [v2] xfs: Simplify xfs_ail_min() with list_first_entry_or_null() (need review)
			* From Richard Weinberger - xfs: Register hotcpu notifier after initialization (reviewed, applied, not pushed yet)
			* From Dan Carpenter - xfs: check for underflow in xfs_iformat_fork() (BPM to ping Dan to see if he'd like to repost)
			* From zwu.kernel@gmail.com - [v4] xfs: introduce object readahead to log recovery (BPM to finish review)
			* From Chandra Seetharaman - [3/3] gfs2: Add support for the Q_XGETQSTATV (Waiting from Ack from Steven Whitehouse)
			* From James Dingwall - Cleancache support in XFS (Although this is a one liner, this needs serious review)

			Userspace:
			* BPM to update his patch for the di_pad regression.
			* Rich mentioned that eric submitted a fix for a bug that was reported.
			* Mark to repost his SEEK_DATA/SEEK_HOLE patch
			* Rich pulled in a couple of one liner fixes for bugs that were detected by coverity.
			* Dimitry Monakov has posted a series on 7/9 that needs review.

			Plumbers Conf:
			* Carlos is working on his talk on thin provisioning for the plumbers conference, also doing some benchmarking related to snapshots. May have interesting numbers for the next meeting.

			xfstests trees:
			* Rich to make a proposal regarding the xfstest trees on the list. (This was prompted by a discussion on irc.)

	== Meeting of Aug 15 ==		== Meeting of Aug 15 ==

Ben Myers: /* Meeting of Aug 15 */

2013-08-15T22:17:20Z

Meeting of Aug 15

← Older revision		Revision as of 22:17, 15 August 2013
Line 10:		Line 10:
	* Ben to pull in Zwu's log recovery readahead work.		* Ben to pull in Zwu's log recovery readahead work.
	* Ben to pull in Zwu's comment fixes.		* Ben to pull in Zwu's comment fixes.
	* There was a short discussion of secondary superblocks and crcs, this is Eric's patch 'xfs: don't verify checksum on non-V5 superblocks'.		* There was a short discussion of secondary superblocks and crcs, this is Eric's patch 'xfs: don't verify checksum on non-V5 superblocks'. Eric thinks this would wait for 3.13.
	* Dave will repost the libxfs sync with the 3.12 kernel code 'maybe today'.		* Dave will repost the libxfs sync with the 3.12 kernel code 'maybe today'.
	* Ben to commit 'xfs: use reference counts to free clean buffer items'.		* Ben to commit 'xfs: use reference counts to free clean buffer items'.

Ben Myers: /* Meeting of Aug 15 */

2013-08-15T22:16:05Z

Meeting of Aug 15

← Older revision		Revision as of 22:16, 15 August 2013
Line 14:		Line 14:
	* Ben to commit 'xfs: use reference counts to free clean buffer items'.		* Ben to commit 'xfs: use reference counts to free clean buffer items'.
	* We discussed the dirent filetype feature 'xfs: Add read-only support for dirent filetype field' and 'xfs: Add write support for dirent filetype field' at length and settled on several actions for getting this in:		* We discussed the dirent filetype feature 'xfs: Add read-only support for dirent filetype field' and 'xfs: Add write support for dirent filetype field' at length and settled on several actions for getting this in:
	1) Eric volunteered to do an xfstest		* Eric volunteered to do an xfstest
	2) xfs_repair, Dave said he was working on this		* xfs_repair, Dave said he was working on this
	3) v4 support in the kernel to be done by Mark		* v4 support in the kernel to be done by Mark
	4) v4 support for mkfs.xfs to be done by Mark		* v4 support for mkfs.xfs to be done by Mark
	5) Plan for 3.13		* Plan for 3.13

	== Meeting of Jul 19 ==		== Meeting of Jul 19 ==

Ben Myers at 22:15, 15 August 2013

2013-08-15T22:15:04Z

← Older revision		Revision as of 22:15, 15 August 2013
Line 1:		Line 1:
			== Meeting of Aug 15 ==

			Attending: Troy, Geoffrey, Mark, Ben, Andrew, Rich, Phil, Carlos, Brian, Eric, Dave

			3.12:
			* Ben will pull in Dwight's kernel work.
			* Rich will pull in Dwight's tests.
			* Ben to pull in Chandra's Kernel work, will apply fixes for Rich's comments at commit time, as per Chandra's request.
			* Rich pulled in Chandra's userspace work for quotas.
			* Ben to pull in Zwu's log recovery readahead work.
			* Ben to pull in Zwu's comment fixes.
			* There was a short discussion of secondary superblocks and crcs, this is Eric's patch 'xfs: don't verify checksum on non-V5 superblocks'.
			* Dave will repost the libxfs sync with the 3.12 kernel code 'maybe today'.
			* Ben to commit 'xfs: use reference counts to free clean buffer items'.
			* We discussed the dirent filetype feature 'xfs: Add read-only support for dirent filetype field' and 'xfs: Add write support for dirent filetype field' at length and settled on several actions for getting this in:
			1) Eric volunteered to do an xfstest
			2) xfs_repair, Dave said he was working on this
			3) v4 support in the kernel to be done by Mark
			4) v4 support for mkfs.xfs to be done by Mark
			5) Plan for 3.13

	== Meeting of Jul 19 ==		== Meeting of Jul 19 ==

Ben Myers at 23:00, 18 July 2013

2013-07-18T23:00:30Z

← Older revision		Revision as of 23:00, 18 July 2013
Line 1:		Line 1:
			== Meeting of Jul 19 ==

			Attending: Ric, Dave, Brian, Chandra, Troy, Ben, Mark, Andrew, Geoffrey, Eric

			3.11:

			* Ben notes the remaining content is limited to two patches from Chandra's quota work
			* Chandra, Dave, and Ben discussed the remaining issues with the patches, and Ben requested that when user/group/prj quotas are active the old interface continue to work for compatability
			* Chandra agreed to repost the patch without the -EINVAL return code
			* A bugfix from Chandra for a CIL deadlock (will be appropriate for 3.11) is posted to the list
			* The quota related hang at unmount in xfs/50 is reproduced by Dave, and Jeff. Chandra has not hit it
			* Dave notes a log space hang with generic/204 with a 512 bytes block size
			* Dave notes a log reservation overflow in generic/070

			3.12:

			* Dave notes that there are changes to shrinkers and LRU lists coming through the MM tree
			* Dave is going to post the rearrangement for libxfs later this morning
			* Ben notes there is work from Dwight, Jeff, and Chandra on the list for 3.12
			* Log reservation size verification from Jeff
			* Dave split his CIL optimisation patch into 5 separate patches
			* Dave has been working on dtype support to optimise lookups in directories to avoid having to stat each inode
			* Dave is working on swapextents with CRCS.
			* Dave requested an xfsprogs release to go with 3.12
			* Mark reported a bug with xfs/227, which Dave things is likely related to the log reservation overflow in generic/070


	== Meeting of Jul 12 ==		== Meeting of Jul 12 ==

Ben Myers: /* Meeting of Jul 12 */

2013-07-18T22:52:00Z

Meeting of Jul 12

← Older revision		Revision as of 22:52, 18 July 2013
Line 5:		Line 5:
	* Chandra posted his quotas series again yesterday, will separate out the QSTAT work, and post a new rev soon		* Chandra posted his quotas series again yesterday, will separate out the QSTAT work, and post a new rev soon
	* Dave suggested a complete separation of the new QSTAT interface from the old QSTAT interface		* Dave suggested a complete separation of the new QSTAT interface from the old QSTAT interface
	* Discussed the proper behavior for the old QSTAT interface when user/group/proj quotas are all active		* Discussed the proper behavior for the old QSTAT interface when user/group/proj quotas are all active, and came to the conclusion that it can return -EINVAL
	* Agreed we'd like to get the remaining patch into 3.11, but are aware it is very close to the merge window		* Agreed we'd like to get the remaining patch into 3.11, but are aware it is very close to the merge window
	* Dave mentioned that lately we've been rushing to get stuff in at the last minute, and it would be better to get things in earlier in the development window. Ben agrees.		* Dave mentioned that lately we've been rushing to get stuff in at the last minute, and it would be better to get things in earlier in the development window. Ben agrees.

Ben Myers at 22:51, 18 July 2013

2013-07-18T22:51:22Z

← Older revision		Revision as of 22:51, 18 July 2013
Line 1:		Line 1:
			== Meeting of Jul 12 ==

			Attending: Dave, Chandra, Andrew, Brian, Mark, Ben, Troy

			* Chandra posted his quotas series again yesterday, will separate out the QSTAT work, and post a new rev soon
			* Dave suggested a complete separation of the new QSTAT interface from the old QSTAT interface
			* Discussed the proper behavior for the old QSTAT interface when user/group/proj quotas are all active
			* Agreed we'd like to get the remaining patch into 3.11, but are aware it is very close to the merge window
			* Dave mentioned that lately we've been rushing to get stuff in at the last minute, and it would be better to get things in earlier in the development window. Ben agrees.
			* Dave is going to repost the rearrangement of code for libxfs syncs for 3.12
			* Dave mentioned that there are changes coming up which are going through the MM trees.

	== Meeting of Jun 6 ==		== Meeting of Jun 6 ==

Ben Myers at 22:44, 18 July 2013

2013-07-18T22:44:25Z

@@ Line 1: / Line 1: @@
 == Meeting of 30 May 2013 ==
@@ Line 49: / Line 69: @@
 >>> We're going to do a pull request with the stuff BPM outlined plus the attr stuff for 3.10-rc4.
 >>> The inode unlinked list work and quota stuff still needs review for a pull request in 3.10-rc5.