Messages in this thread | ![/](/images/icornerl.gif) | | Date | Sat, 11 Mar 2023 00:23:04 +0100 | Subject | Re: [PATCH v1 1/2] Add release hook to LSM | From | Mirsad Goran Todorovac <> |
| |
On 10. 03. 2023. 21:12, Andy Shevchenko wrote: > On Fri, Mar 10, 2023 at 08:52:32PM +0100, Mirsad Goran Todorovac wrote: >> On 10. 03. 2023. 20:47, Andy Shevchenko wrote: >>> On Fri, Mar 10, 2023 at 08:42:00PM +0100, Mirsad Goran Todorovac wrote: >>>> >>>> Add release() hook to the definition of the LSM modules, to enable calling >>>> destructors and deallocating allocated resources cleanly. >>>> >>>> Signed-off-by: Mirsad Goran Todorovac <mirsad.todorovac@alu.unizg.hr> >>> >>>> Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> >>> >>> I haven't given you this tag. Sorry, you must not add something >>> which was not explicitly given. >> >> Mea culpa. >> >> I misunderstood that you have approved the change when saying that I should >> file a formal patch. It worked with Florian Westphal in my first patch, so I >> thought it would be inappropriate to leave out who endorsed the patch. > > We have Suggested-by tag for that.
Done exactly that, and now I have to reward myself for the first successful multi-part patch submission.
>> Thousand apologies. This is my second patch, so I am asking for some extra >> grace ... > >> What should I do now? > > Wait at least a couple of days (even more, since it's weekend) for other > comments and then send a v2 of the series with the comments addressed.
Actually, Mr. Paul Moore requested the rest of the patch, so I did a complete resend immediately.
> Btw, you may take my "smart" script [1] for sending patch series. It > makes much less headache on how to do that properly. > > [1]: https://github.com/andy-shev/home-bin-tools/blob/master/ge2maintainer.sh
Thanks for the tip.
It's pleasure to be working with you guys, it's a great brainstorming session and I really enjoyed it. It's a privilege to assist your excellent team.
Best regards, Mirsad
-- Mirsad Goran Todorovac Sistem inženjer Grafički fakultet | Akademija likovnih umjetnosti Sveučilište u Zagrebu System engineer Faculty of Graphic Arts | Academy of Fine Arts University of Zagreb, Republic of Croatia The European Union
| ![\](/images/icornerr.gif) |