Re: [smartos-discuss] inotify driver not loaded automatically

2014-12-09 Thread Keith Wesolowski via smartos-discuss
On Tue, Dec 09, 2014 at 11:02:03AM -0500, Youzhong Yang via smartos-discuss wrote: I cherry-picked this cool inotify implementation ( https://github.com/joyent/illumos-joyent/commit/a9a246c0c49e192616e7499eaa2362b21fde8f5e) and built my own image. After the machine reboot, inotify is not

Re: [smartos-discuss] inotify driver not loaded automatically

2014-12-09 Thread Youzhong Yang via smartos-discuss
Yeah, that's what I need. Thanks Robert. On Tue, Dec 9, 2014 at 11:04 AM, Robert Mustacchi r...@joyent.com wrote: On 12/9/14 8:02 , Youzhong Yang via smartos-discuss wrote: Hi All, I cherry-picked this cool inotify implementation (

Re: [smartos-discuss] inotify driver not loaded automatically

2014-12-09 Thread Robert Mustacchi via smartos-discuss
On 12/9/14 8:24 , Youzhong Yang via smartos-discuss wrote: Yeah, that's what I need. Thanks Robert. Yes, however, all of Keith's comments on this matter still apply. There are often follow up commits to these things and if you're cherry picking, you'll need to go through and verify that there

Re: [smartos-discuss] inotify driver not loaded automatically

2014-12-09 Thread Youzhong Yang via smartos-discuss
We have a stable base platform and don't want to introduce too many changes which could potentially break something. I've gone through the history of all files involved in this inotify implementation and also picked up a few other bug fixes. We will run thorough qualification to make sure the new