The last time Hackerfall tried to access this page, it returned a not found error. A cached version of the page is below, or clickhereto continue anyway

65535 interfaces ought to be enough for anybody

65535 interfaces ought to be enough for anybody

Date: February 14, 2017. Category: .NET Tags: .NET Rider Bugs Mono

It was a bright, sunny morning. There were no signs of trouble. I came to work, opened Slack, and received many messages from my coworkers about failed tests.

After a few hours of investigation, the situation became clear:

So, what was special about this "bad commit"? Spoiler: after these changes, we sometimes have more than 65535 interface implementations at runtime.

Going back to 2005

Do you remember these days? It was time of the .NET Framework 1.1, C# 1.2, and Mono 1.1.3. Yes, Mono already existed, but no one ran huge applications on it. So, it seemed ok to use 16-bit integer type guint16 for interface_id. Indeed, who needs more than 65535 interfaces?

Present day

Rider uses the ReSharper codebase which is really big. Of course, we have many generic classes. A short fact about .NET: if you have an interface IFoo<T>, the runtime generates a separate method table per each IFoo<int>, IFoo<double>, IFoo<bool>, and so on. A short fact about the unit tests subsystem in ReSharper: it uses a lot of generic classes (especially in the tests for unit tests).

On Windows, everything worked fine because Rider uses the full .NET Framework which doesn't have such limitation on the number of interfaces. On Linux and MacOS, we use Mono as a runtime. And tests were failing because we have too many interfaces! It took a week of debugging to find the problem, but we finally did it.

We found a report in the Mono bug tracking system: bugzilla.xamarin#10784 (2013-02-28). We also found a pull request (2016-01-08) which should solve this problem. Unfortunately, it was unmerged without any progress. A friendly reminder (2016-10-20) helped, and it was eventually merged into master (2016-11-07).

This fix is not a part of the latest stable mono yet. Fortunately, Rider uses its own Mono fork. So, we just cherry-picked this commit, and now all of our tests are green again.

Links

Share:

Please enable JavaScript to view the comments powered by Disqus.

Continue reading on aakinshin.net