From: Rich Johnston <rjohnston@sgi.com>
To: Ryan Wang <openspace.wang@gmail.com>
Cc: "Eric Sandeen" <sandeen@redhat.com>,
xfs@oss.sgi.com, "Theodore Ts'o" <tytso@mit.edu>,
"Lukáš Czerner" <lczerner@redhat.com>
Subject: Re: How to run xfstests against multipath disks on SuSE?
Date: Wed, 28 Nov 2012 09:17:11 -0600 [thread overview]
Message-ID: <50B62AF7.6030907@sgi.com> (raw)
In-Reply-To: <CAPxxNQnGE_L2xdOimA-czk1RhnhvLdcY=swRYYucZM+bzNBn9w@mail.gmail.com>
On 11/28/2012 08:10 AM, Ryan Wang wrote:
> 2012/11/28 Rich Johnston <rjohnston@sgi.com>:
>> On 11/27/2012 02:37 PM, Eric Sandeen wrote:
>>>
>>> On 11/27/12 2:35 PM, Dave Chinner wrote:
>>>>
>>>> On Tue, Nov 27, 2012 at 11:08:30AM -0600, Eric Sandeen wrote:
>>>>>
>>>>> On 11/26/12 9:14 PM, Dave Chinner wrote:
>>>>>>
>>>>>> On Tue, Nov 27, 2012 at 09:20:47AM +0800, Ryan Wang wrote:
>>>>>>>
>>>>>>> 2012/11/26 Theodore Ts'o <tytso@mit.edu>:
>>>>>>>>
>>>>>>>> On Mon, Nov 26, 2012 at 02:43:40PM +0100, Lukáš Czerner wrote:
>>>>>>>>>>
>>>>>>>>>> The I checked the script common.rc, and found that it uses
>>>>>>>>>> df to determine the filesystem type and it uses the realpath
>>>>>>>>>> "/dev/dm-0" instead of "/dev/mapper/mpatha", while df always
>>>>>>>>>> shows "devtmpfs" instead of "ext4" for /dev/dm-0.
>>>>>>>>>>
>>>>>>>>>> I run "mount" and found that devtmpfs is mount on /dev/ on opensuse
>>>>>>>>>> ***by default***. Even though I run "mount -t btrfs /dev/dm-0
>>>>>>>>>> /somedir",
>>>>>>>>>> df still showed devtmpfs.
>>>>>>>>>>
>>>>>>>>>> Any ideas to kick off the xfstests?
>>>>>>>>
>>>>>>>>
>>>>>>>> At least with the df which ships with Debian Testing, df will display
>>>>>>>> the file system in use if it matches the device name which is
>>>>>>>> /etc/mtab. It looks the issue is that mount is canonicalizing the
>>>>>>>> device name to be the human friendly version:
>>>>>>>>
>>>>>>>> # ls -l /dev/mapper/closure-bigscratch
>>>>>>>> 0 lrwxrwxrwx 1 root root 7 Nov 16 15:15
>>>>>>>> /dev/mapper/closure-bigscratch -> ../dm-7
>>>>>>>> <tytso.root@closure> {/home/tytso/bin}, level 2
>>>>>>>> # mount /dev/dm-7 /u2
>>>>>>>> # df /u2
>>>>>>>> Filesystem 1K-blocks Used Available Use%
>>>>>>>> Mounted on
>>>>>>>> /dev/mapper/closure-bigscratch 52403200 224772 52178428 1% /u2
>>>>>>>> # grep /u2 /etc/mtab
>>>>>>>> /dev/mapper/closure-bigscratch /u2 xfs
>>>>>>>> rw,relatime,attr2,inode64,noquota 0 0
>>>>>>>>
>>>>>>>> So the simplest solution is to just use /dev/mapper/mpatha instead of
>>>>>>>> insisting on using /dev/dm-0. That's what I do....
>>>>>>>
>>>>>>>
>>>>>>> In xfstests/common.rc, it will use realpath to obtain /dev/dm-0, even
>>>>>>> if you
>>>>>>> specified /dev/mapper/mpatha.
>>>>>>> My problem is in SuSE, the devtmpfs is mounted on /dev by default,
>>>>>>> seems
>>>>>>> used by udev. And df reports devtmpfs on /dev/dm-0, instead of ext4.
>>>>>>
>>>>>>
>>>>>> What version of xfstests are you using? Do you have a
>>>>>> "README.device-mapper" file in the base directory? If not, upgrade
>>>>>> to the latest version and retry.
>>>>>>
>>>>>> $ gl -n 1 -p 0a81886
>>>>>> commit 0a818862bb74932e15971ec8bedb55765ec3a3f6
>>>>>> Author: Christoph Hellwig <hch@lst.de>
>>>>>> Date: Wed May 4 16:28:32 2011 +0200
>>>>>>
>>>>>> xfstests: support post-udev device mapper nodes
>>>>>>
>>>>>> Because of udevs complaining device mapper now creates /dev/dm-N
>>>>>> as the real
>>>>>> device nodes, and just symlinks the /dev/mapper/ names to it.
>>>>>> This would be
>>>>>> easy if everything used the /dev/mapper clear names, but most
>>>>>> system utilities
>>>>>> translate them back to the /dev/mapper/ names and thus confuse
>>>>>> various test
>>>>>> cases. Add support to _is_block_dev to read symlinks, and add
>>>>>> documentation
>>>>>> on how to run xfstests on device mapper volumes.
>>>>>>
>>>>>> Signed-off-by: Christoph Hellwig <hch@lst.de>
>>>>>> Reviewed-by: Alex Elder <aelder@sgi.com>
>>>>>>
>>>>>> Cheers,
>>>>>>
>>>>>> Dave.
>>>>>>
>>>>>
>>>>> Recent "realpath" changes also went in:
>>>>>
>>>>> b03c662399956609934ff063e9064e15dc7459ac xfstests: fix to build
>>>>> src/realpath and the correct the existence of target devices
>>>>> d5ea873fcbf00b841f0511a26fb341364a5c6425 xfstests: resolve symlinked
>>>>> devices to real paths
>>>>>
>>>>> so either of those could have broken and/or fixed it for you :)
>>>>
>>>>
>>>> Yup, those are almost certainly the patches that broke device mapper
>>>> support. There' aren't changes to common.rc, so I didn't notice them.
>>>> They basically override the fix that Christoph made to support
>>>> /dev/mapper links properly.
>>>>
>>>> I'd suggest a revert is in order, the two patches above were part
>>>> of Rich's hovering of old, unreviewed patches. This problem was
>>>> already fixed in a different way by Christoph, so adding these
>>>> patches was wrong.
>>>
>>>
>>> Fine with me ...
>>>
>>> Ryan, can you test w/o those, and if it works -
>>>
>>> Rich, can you revert those please?
>>
>>
>> Sure, I will wait to hear from Ryan first.
>>
>
> Sorry for my late response.
>
> I reverted the 2 patches and rebuilt xfstests on my box.
> It can work now on the multipath disks.
>
> Thanks for your support!
>
> Ryan
Patches c7459ac and 5c6425 have been reverted on
git://oss.sgi.com/xfs/cmds/xfstests master branch, commits 86cf77 and
82e889.
--Rich
>
>>>
>>> -Eric
>>>
>>>> Cheers,
>>>>
>>>> Dave.
>>>>
>>>
>>> _______________________________________________
>>> xfs mailing list
>>> xfs@oss.sgi.com
>>> http://oss.sgi.com/mailman/listinfo/xfs
>>>
>>
>>
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
prev parent reply other threads:[~2012-11-28 15:14 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-11-26 0:00 How to run xfstests against multipath disks on SuSE? Ryan Wang
2012-11-26 13:43 ` Lukáš Czerner
2012-11-26 13:43 ` Lukáš Czerner
2012-11-26 14:55 ` Theodore Ts'o
2012-11-26 14:55 ` Theodore Ts'o
2012-11-27 1:20 ` Ryan Wang
2012-11-27 1:20 ` Ryan Wang
2012-11-27 3:14 ` Dave Chinner
2012-11-27 3:14 ` Dave Chinner
2012-11-27 17:08 ` Eric Sandeen
2012-11-27 17:08 ` Eric Sandeen
2012-11-27 20:35 ` Dave Chinner
2012-11-27 20:35 ` Dave Chinner
2012-11-27 20:37 ` Eric Sandeen
2012-11-27 20:37 ` Eric Sandeen
2012-11-27 20:55 ` Rich Johnston
2012-11-27 20:55 ` Rich Johnston
2012-11-28 14:10 ` Ryan Wang
2012-11-28 15:17 ` Rich Johnston [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=50B62AF7.6030907@sgi.com \
--to=rjohnston@sgi.com \
--cc=lczerner@redhat.com \
--cc=openspace.wang@gmail.com \
--cc=sandeen@redhat.com \
--cc=tytso@mit.edu \
--cc=xfs@oss.sgi.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.