| To: | Barry Naujok <bnaujok@xxxxxxx> |
|---|---|
| Subject: | Re: REVIEW: Make sure xfs_repair uses sector aligned I/O |
| From: | David Chinner <dgc@xxxxxxx> |
| Date: | Fri, 13 Jul 2007 10:51:23 +1000 |
| Cc: | "xfs@xxxxxxxxxxx" <xfs@xxxxxxxxxxx>, xfs-dev <xfs-dev@xxxxxxx> |
| In-reply-to: | <op.tvdn7xhl3jf8g2@pc-bnaujok.melbourne.sgi.com> |
| References: | <op.tvdn7xhl3jf8g2@pc-bnaujok.melbourne.sgi.com> |
| Sender: | xfs-bounce@xxxxxxxxxxx |
| User-agent: | Mutt/1.4.2.1i |
On Fri, Jul 13, 2007 at 10:41:47AM +1000, Barry Naujok wrote: > Two places it didn't: > - reading the initial primary superblock > - checking the log > > The log checking fix was basically porting the kernel > xlog_bread/xlog_get_bp/xlog_align from the kernel and > setting up the log_t structure properly in Phase 2. > > The initial primary superblock read is now done > using the maximum supported sector size as once the > block is read and translated to another location, > it's discarded. Seems OK from a quick look of the code. Were you able to confirm that these are the only places we do non-sector sized I/O? Cheers, Dave. -- Dave Chinner Principal Engineer SGI Australian Software Group |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | REVIEW: Make sure xfs_repair uses sector aligned I/O, Barry Naujok |
|---|---|
| Next by Date: | Re: xfsqa 144 is failing now, David Chinner |
| Previous by Thread: | REVIEW: Make sure xfs_repair uses sector aligned I/O, Barry Naujok |
| Next by Thread: | [PATCH 0/6][TAKE7] fallocate system call, Amit K. Arora |
| Indexes: | [Date] [Thread] [Top] [All Lists] |