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

2014-12-09 Thread Youzhong Yang via smartos-discuss
By the way, we will evaluate this inotify and see if it can help us get rid of gam_server (change notification done through portfs APIs for Samba). If it can and has the same or even better performance, that would be great. On Tue, Dec 9, 2014 at 11:28 AM, Youzhong Yang wrote: > We have a stable

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

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 ar

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 wrote: > On 12/9/14 8:02 , Youzhong Yang via smartos-discuss wrote: > > Hi All, > > > > I cherry-picked this cool inotify implementation ( > > > https://github.com/joyent/illumos-joyent/commit/a9a246c0c49e

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 lo

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

2014-12-09 Thread Robert Mustacchi via smartos-discuss
On 12/9/14 8:02 , Youzhong Yang via smartos-discuss wrote: > Hi All, > > 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 loaded so > I h

[smartos-discuss] inotify driver not loaded automatically

2014-12-09 Thread Youzhong Yang via smartos-discuss
Hi All, 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 loaded so I have to manually load it through add_drv command. Is there a way to hav