On Sat, Mar 22, 2014 at 10:55:46AM -0400, Youzhong Yang wrote: > We had zones zpool data corruption when 'dr_sas' driver is loaded for > Supermicro > SMC2108 (LSI MegaRAID) controller.
What makes you believe there is a causal relationship between these two things? > After I rebuilt smartos image with the change in /etc/driver_aliases, it > works fine. What I did is to roll back the following commit: Were you planning to give us even the slightest hint of what the "corruption" looked like, its history, your efforts to debug it, how you came to isolate its cause to this change, etc.? Or were you just trying to enrage and infuriate us by stirring up vague, impotent fear? If the latter, you're a winner. > https://github.com/joyent/smartos-live/commit/c0409b90008a6dd76afdf5d9aad0b5be8c0d6bec > > My questions are: > - what is this OS-1529 all about? does it fix any known issue by using > dr_sas driver? has this driver been fully tested? dr_sas was forked off mr_sas at the point in time when mr_sas took a huge amount of change to support various new devices. That change consisted of black box code from LSI that we had limited ability to test or otherwise verify. Therefore, to reduce risk, we duplicated the driver and assigned the existing PCI IDs to dr_sas and the newly-supported ones only to mr_sas. Since we don't have nor intend to have any of the devices for which the change introduced support, this was a zero-risk way to add HW support for the benefit of third parties who want it. We have been running this particular driver (previously named mr_sas, now named dr_sas) in production for several years. I am not aware of a single incident in which the driver has been responsible for data corruption in that time. If anything here is poorly tested, it's mr_sas, not dr_sas. ------------------------------------------- smartos-discuss Archives: https://www.listbox.com/member/archive/184463/=now RSS Feed: https://www.listbox.com/member/archive/rss/184463/25769125-55cfbc00 Modify Your Subscription: https://www.listbox.com/member/?member_id=25769125&id_secret=25769125-7688e9fb Powered by Listbox: http://www.listbox.com
