答复: [PATCH] pvscan --cache /dev/xx needs to scan all disks and aliases when enable device filter

[wangzhiqiang (Q) <wangzhiqiang95@huawei.com>]

Thanks. the _filter_uses_symlinks function handles this case well.

wangzhiqiang

-----????-----
???: David Teigland <teigland@redhat.com> 
????: 2022?12?7? 23:30
???: wangzhiqiang (Q) <wangzhiqiang95@huawei.com>
??: lvm-devel at redhat.com; linfeilong <linfeilong@huawei.com>; liuzhiqiang (I) <liuzhiqiang26@huawei.com>
??: Re: [PATCH] pvscan --cache /dev/xx needs to scan all disks and aliases when enable device filter

On Wed, Dec 07, 2022 at 09:34:13AM +0000, wangzhiqiang (Q) wrote:
> When the system restarts and the disk goes online, the udev rule will 
> be triggered to scan the metadata on the disk to automatically 
> activate the logical volume, If we specify the scanned disk alias in lvm.conf, reject other unspecified disks, then lv cannot activate since the aliases mismatch with disk name.
> 
> If device filtering is configured in lvm.conf, then pvscan --cache 
> /dev/xx needs to scan all disks and aliases

Hi, if I understand correctly, pvscan is already attempting to handle this case.  See the use of "_filter_uses_symlinks" in pvscan.c.  Perhaps it needs check for other symlink substrings?  The problem with your patch is that it's a significant performance penalty for a common configuration.

Dave