On Mon, Aug 07, 2017 at 09:12:28PM +0000, Warner Losh wrote:
> +     LIST_FOREACH(gap, &pp->geom->aliases, ga_next) {
> +             error = make_dev_alias_p(MAKEDEV_CHECKNAME | MAKEDEV_WAITOK, 
> &adev, dev,
> +                 "%s", gap->ga_alias);
> +             if (error) {
> +                     printf("%s: make_dev_alias_p() failed (name=%s, 
> error=%d)\n",
> +                         __func__, gap->ga_alias, error);
> +                     continue;
> +             }
> +             adev->si_flags |= SI_UNMAPPED;
Why do you set the flag unconditionally ?

> +             adev->si_iosize_max = dev->si_iosize_max;
> +             adev->si_drv2 = dev->si_drv2;
And what are you trying to do by these initializations, including the
si_flags adjustment ?

Aliases cause creation of symlinks in the devfs populate loop, which
makes it impossible to access the alias cdevs.

_______________________________________________
[email protected] mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "[email protected]"

Reply via email to