Messages in this thread | | | Subject | Re: [PATCH] kernel/signal: Signal-based pre-coredump notification | From | Enke Chen <> | Date | Mon, 15 Oct 2018 11:49:11 -0700 |
| |
Hi, Greg:
On 10/15/18 11:43 AM, Greg Kroah-Hartman wrote: > On Mon, Oct 15, 2018 at 11:16:36AM -0700, Enke Chen wrote: >> Hi, Greg: >> >>> Shouldn't there also be a manpage update, and a kselftest added for this >>> new user/kernel api that is being created? >>> >> >> I will submit a patch for manpage update once the code is accepted. > > Writing a manpage update is key to see if what you are describing > actually matches the code you have submitted. You should do both at the > same time so that they can be reviewed together.
Ok, will do at the same time. But should I submit it as a separate patch?
> >> Regarding the kselftest, I am not sure. Once the prctl() is limited to >> self (which I will do), the logic would be pretty straightforward. Not >> sure if the selftest would add much value. > > If you do not have a test for this feature, how do you know it even > works at all? How will you know if it breaks in a future kernel > release? Have you tested this? If so, how?
I have the test code. I am just not sure whether I should submit and check it in to the kselftest?
Thanks. -- Enke
| |