Re: [PATCH 01/23] staging/lustre/clio: don't ignore layout on writeback

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Andreas and Greg,

Sorry for top posting, but the text is mixed with Andreas' reply. And
I want to confirm with you what to keep/drop in the commit messages.

Currently, we have following non-sob/rb lines:
1. Intel-bug-id:
2. Lustre-commit:
3. Lustre-change:
4. one blank line seperating original commit message from the added
lines for kernel commit
5. one line of comment for upstream change explanation

I think we all agreed that we will at least keep 1 and 3, with 1
changed to full URL. So I'd like to confirm with you what to do with
the others. I think both of you have good reasoning about the format
of commit message. But we need to draw a conclusion to move forward,
right? :)

Greg, since Andreas' last reply was mixed with quotes, I
copied/reformated them bellow so that you can read easier.

> Lustre-commit: 3141db609d95d379761e3b54899618b4037d38f6
>
>
> Or this one?
>
>
[Andreas] This is the Lustre git commit hash, so we can track the
commits which have been merged into the kernel tree.

>
> Lustre-change: http://review.whamcloud.com/6154
>
>
> This one is at least informative, so it can stay, if you really want it
> there, but the others are not relevant to anyone outside of your
> internal development environment, so do not belong in a Linux kernel
> commit message, sorry.
>
>
[Andreas] Per Documentation/SubmittingPatches: "Some people also put
extra tags at the end.  They'll just be ignored for now, but you can
do this to mark internal company procedures or just point out some
special detail about the sign-off."

> [updated for upstream kernel submission]
>
>
> What's with the line break and this [] comment?  We don't care about
> that.
>
>
[Andreas] Also per SubmittingPatches:
[Andreas] "...it is recommended that you add a line between the last
Signed-off-by header and yours, indicating the nature of your changes.
While there is nothing mandatory about this, it seems like prepending
the description with your mail and/or name, all enclosed in square
brackets, is noticeable enough to make it obvious that you are
responsible for last-minute changes."

[Andreas] I guess we can remove the obvious ones (minor or no changes
from the original patch), and improve the ones with substantive
changes to be more descriptive.


Thanks and Best Regards,
Tao

PS, Andreas, if you happen to be forced to use Outlook like me, it is
better to set "use internet style quoting"[1] and "read all message as
plain text"[2], so that others can read properly your reply/quotes in
plain text.

[1] http://www.slipstick.com/outlook/email/to-use-internet-style-quoting/
[2] Tools->Trust Center->Email Security->Read all standard mail in plain text
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel




[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux