On Sun, 2020-07-05 at 20:30 +0200, Amit Shah wrote: > On (Wed) 01 Jul 2020 [22:09:50], Rikard Falkeborn wrote: > > The id_table and feature_table pointers in struct virtio_driver are > > pointers to const. Mark the corresponding static variables const to > > allow the compiler to put them in read-only memory. > > > > Before: > > text data bss dec hex filename > > 25447 713 76 26236 667c drivers/char/virtio_console.o > > > > After: > > text data bss dec hex filename > > 25488 673 76 26237 667d drivers/char/virtio_console.o > > > > Signed-off-by: Rikard Falkeborn <rikard.falkeborn@xxxxxxxxx> > > Reviewed-by: Amit Shah <amit@xxxxxxxxxx> > > Please CC me on the entire series instead of individual patches in the > future. CC'ing individual maintainers on entire patch sets that span multiple subsystems doesn't scale. Given that lore.kernel.org now stores all emails sent to lkml, it's a reasonable thing to add a reference to the cover letter below the --- line to make it easier for individual recipients of patches in a series to find the entire patch set. i.e. Send emails with something like: Subject: [PATCH N/M] foo: bar commit message Signed-off-by: J. Random Developer <address@xxxxxxxxxx> --- Link: https://lore.kernel.org/r/<message_id_of_cover_letter> [patch] A trivial script to insert these links if all patches in a series are stored in a separate directory: $ bash insert_cover_letter_link.bash <patch_directory> --- $ cat insert_cover_letter_link.bash #!/bin/bash find $@ -name "*.patch" | sort | \ while read file ; do echo "File: <$file>" if [[ $(basename $file) =~ ^0000- ]] ; then message_id=$(grep '^Message-Id: <' $file) if [ $? -ne 0 ] ; then echo "Message_Id not found" exit 1 fi message_id=$(echo $message_id | \ sed -r -e 's/^Message-Id:\s*<//' -e 's/>\s*$//') separator="-- " echo "Message_Id: <$message_id>" else separator="---" fi if [[ "$message_id" == "" ]] ; then echo "Patch series cover letter Message_Id not found" exit 1 fi perl -n -i -e "if (\$_ =~ /^$separator$/) { print; print \"\\n\"; print \"Link: https://lore.kernel.org/r/\"; . '$message_id' . \"\\n\"; print \"\\n\"; } else { print; }" $file done exit 0 _______________________________________________ Virtualization mailing list Virtualization@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linuxfoundation.org/mailman/listinfo/virtualization
