From: Schrempf Frieder <frieder.schrempf@kontron.de>
To: Boris Brezillon <boris.brezillon@bootlin.com>
Cc: Miquel Raynal <miquel.raynal@bootlin.com>,
Richard Weinberger <richard@nod.at>,
"stable@vger.kernel.org" <stable@vger.kernel.org>,
"David Woodhouse" <dwmw2@infradead.org>,
Brian Norris <computersforpeace@gmail.com>,
Marek Vasut <marek.vasut@gmail.com>,
"linux-mtd@lists.infradead.org" <linux-mtd@lists.infradead.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] mtd: nand: Fix memory allocation in nanddev_bbt_init()
Date: Thu, 29 Nov 2018 18:11:11 +0000 [thread overview]
Message-ID: <fa05e203-430c-6ec4-55e9-a7f9f548e481@kontron.de> (raw)
In-Reply-To: <20181128162810.597a7fd3@bbrezillon>
On 28.11.18 16:28, Boris Brezillon wrote:
> On Wed, 28 Nov 2018 15:19:37 +0000
> Schrempf Frieder <frieder.schrempf@kontron.de> wrote:
>
>> On 28.11.18 16:02, Boris Brezillon wrote:
>>> On Wed, 28 Nov 2018 14:55:45 +0000
>>> Schrempf Frieder <frieder.schrempf@kontron.de> wrote:
>>>
>>>> Hi Boris,
>>>>
>>>> On 28.11.18 15:41, Boris Brezillon wrote:
>>>>> On Tue, 27 Nov 2018 07:44:52 +0000
>>>>> Schrempf Frieder <frieder.schrempf@kontron.de> wrote:
>>>>>
>>>>>> Fix the size of the buffer allocated to store the in-memory BBT.
>>>>>> This bug was previously hidden by a different bug, that was fixed in
>>>>>> d098093ba06e.
>>>>>>
>>>>>> Fixes: 9c3736a3de21 ("mtd: nand: Add core infrastructure to deal with NAND devices")
>>>>>> Cc: <stable@vger.kernel.org>
>>>>>> Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
>>>>>
>>>>> Looks like your From header does not match the SoB tag
>>>>> ('Frieder Schrempf' vs 'Schrempf Frieder') and checkpatch does not like
>>>>> that. I'll fix it when applying, but maybe you should fix
>>>>> your .gitconfig to make them match.
>>>>
>>>> Actually the From header in my local patch is correct (<first name>
>>>> <last name>, Frieder Schrempf) as it comes from my git config. But since
>>>> our company was renamed and our mail servers were transferred, our
>>>> e-mails are sent with From=<last name> <first name>.
>>>>
>>>> It seems like git send-email or patchwork or whatever uses the
>>>> information from the e-mail header instead of what is in the patch.
>>>>
>>>> I will try to raise this issue with our IT department as this would be
>>>> best fixed on their side.
>>>
>>> There's another solution: make git send-email add a From header in the
>>> message body.
>>>
>>> git config --global sendemail.from "Schrempf Frieder <frieder.schrempf@kontron.de>"
>>
>> I don't get it. How would that change things? My From still wouldn't
>> match my SoB tags.
>
> When there's a From line in the message Body, git am will use this one
> instead of the one in the header section.
Ok, now I got it. I didn't know that git send-email adds the From tag in
the body if sendemail.from differs from the author, but that makes sense
indeed.
I just checked and with my old mail setup, the SMTP server was
respecting my sender address from the client (send-email, thunderbird).
The new setup uses DavMail Gateway to send via the companies Exchange
server and it just ignores my sender from the client and sends with the
reversed name as stored in the Exchange server.
I don't like sending mail with reversed names (people start calling me
by my last name), but using a different SMTP server would likely cause
trouble with some recipients using SPF for spam detection.
So I guess I have to convince our IT to change the names, if even
possible...
Oh, why do such small things always cause so much trouble?
next prev parent reply other threads:[~2018-11-29 18:11 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-11-27 7:44 [PATCH] mtd: nand: Fix memory allocation in nanddev_bbt_init() Schrempf Frieder
2018-11-27 8:22 ` Boris Brezillon
2018-11-27 8:30 ` Schrempf Frieder
2018-11-27 9:27 ` Miquel Raynal
2018-11-28 14:41 ` Boris Brezillon
2018-11-28 14:55 ` Schrempf Frieder
2018-11-28 15:02 ` Boris Brezillon
2018-11-28 15:19 ` Schrempf Frieder
2018-11-28 15:28 ` Boris Brezillon
2018-11-29 18:11 ` Schrempf Frieder [this message]
2018-11-30 9:37 ` Boris Brezillon
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=fa05e203-430c-6ec4-55e9-a7f9f548e481@kontron.de \
--to=frieder.schrempf@kontron.de \
--cc=boris.brezillon@bootlin.com \
--cc=computersforpeace@gmail.com \
--cc=dwmw2@infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mtd@lists.infradead.org \
--cc=marek.vasut@gmail.com \
--cc=miquel.raynal@bootlin.com \
--cc=richard@nod.at \
--cc=stable@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.