Erik Nordmark wrote: > Glenn Faden wrote: >> Erik Nordmark wrote: >>> >>> This part of the system is extremely complex and brittle - it isn't >>> built to be able to do per-zones routes. Hence being concerned about >>> customers creating invalid configurations is a concern to be taken >>> very seriously. >>> >>> Perhaps we should go back to the drawing port and design per zones >>> routes i.e., the ability for route(1m) to explicitly specify a zone >>> name and have that cause ire_zoneid to be set in the kernel. >> Perhaps, but the proposed solution meets the requirements of >> customers that have asked for this feature, and I'd like to get it >> integrated and backported in time for S10U6. > > I'd encourage you at least reflect upon Ed's concerns before charging > full steam ahead - it is good to know whether there is a cliff in > front of you before you decide to dismiss the advise of others.
So far, I don't see a cliff yet. Ed's concern is mostly about not cleaning up default routes after zones are halted. While I think Ed has a point, I don't see zones not cleaning up their default routes upon halting as a disaster. We already have situations that default/static routes don't get cleaned up after they are no longer needed. Your earlier suggestion about adding zoneid to route(1M) is interesting. When ire_zoneid is set explicitly by "zoneid", it's clear who set the ire entry, who can use the entry, and who should clean up the entry. It also implies that the routing table could have many almost identical entries that only differ in ire_zoneid. This may have performance implications when finding a matching entry. Anyway, we haven't thought about the design enough. My suggestion, which an architecture purest may not like :-), is to let Glenn go forward as he planned. We have a separate effort to enhance zone routing in shared IP stack. The interface proposed in this case looks forward compatible with your "route add default <network> <zoneid>" suggestion. We do have some paying customers needing immediate relief badly. Jarrett > > Erik
