RE: [PATCH v2 3/3] pstore: add pstore unregister

From: Luck, Tony
Date: Mon Oct 19 2015 - 18:57:00 EST


> pstore doesn't support unregistering yet. It was marked as TODO.

Thanks for looking to close out this TODO item.

The thing that scared me about unloading pstore was what happens to
a process that is in the middle of reading some /sys/fs/pstore/file-name-here

Do we have all the right reference counts to make sure that process doesn't do
weird things if you rmmod pstore in the middle of a read? Or for a subsequent
read from the still-open file descriptor?

-Tony


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/