panic after r244584
Alexander Motin
mav at FreeBSD.org
Fri Jan 18 15:11:03 UTC 2013
On 18.01.2013 15:49, Vitalij Satanivskij wrote:
> May be just do sanitizing for elmpriv->descr?
>
> something like change whitespace to "_" or just delete it?
Yes, that is not difficult. The only question is how to stay consistent,
compatible, user-readable.
> Vitalij Satanivskij wrote:
> VS> Jaakko Heinonen wrote:
> VS> JH> On 2013-01-18, Alexander Motin wrote:
> VS> JH> > > AM> > V> panic: make_dev_alias_v: bad si_name (error=22 si_name=enc at n5003048000bab37d/tpe0/slot at 1/elmdesc at Slot 01/pass7)
> VS> JH>
> VS> JH> > > AM> The panic is triggered by the check added by the recent r244584 change.
> VS> JH> > > AM> The space in device name came from the enclosure device, and I guess it
> VS> JH> > > AM> may be quite often situation. Using human readable name supposed to help
> VS> JH> > > AM> system administrators, but with spaces banned that may be a problem.
> VS> JH> > >
> VS> JH> > > That's was not created by human, it was generated (I think so) by system.
> VS> JH> >
> VS> JH> > These strings are flashed into enclosure firmware by manufacturer.
> VS> JH>
> VS> JH> You can't rely on that any string can be safely used as a device name
> VS> JH> even if spaces were allowed. Consider for example duplicate names and
> VS> JH> "../".
> VS> JH>
> VS> JH> Where these names are generated? The original report didn't contain a
> VS> JH> backtrace.
> VS>
> VS> Yes. No backtrace, because of switching off all debuging in kernel.
> VS>
> VS> For now I can't use that's server for testing, but there are another servers waiting for upgrade.
> VS>
> VS> I will try to reproduce problem with kernel debuger enabled.
> VS>
> VS>
> VS> _______________________________________________
> VS> freebsd-current at freebsd.org mailing list
> VS> http://lists.freebsd.org/mailman/listinfo/freebsd-current
> VS> To unsubscribe, send any mail to "freebsd-current-unsubscribe at freebsd.org"
>
--
Alexander Motin
More information about the freebsd-current
mailing list