On Sun, Dec 03, 2017 at 07:59:07PM +0300, ak wrote:
> Thank you for your extensive comments.
> 
> I've also thought about adding more protection into unregister_shrinker(),
> but not sure how to properly organize the patch set, because there will be
> three patches:
>     * change in mm/vmscan that adds protection and sanitizer.
>     * fixed change for Lustre driver
>     * there also two explicit usages of shrinker->nr_deferred in drivers -
>        good idea to fix too.
> All patches have different lists of maintainers, and second and third depend
> on first one. And I don't like to send them separately.
> So, I'm going to at least prepend this patch with mm/vmscan one.

Fix the style for the Lustre patch and resend.  Then patch
unregister_shrinker().  Then remove the checks.

The unregister_shrinker() changes seem like a good idea, but I haven't
really looked at it.  It might be more involved than it seems.

regards,
dan carpenter
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to