From: Sudip Mukherjee <sudipm.mukherjee@gmail.com>
To: Dongsheng Yang <yangds.fnst@cn.fujitsu.com>
Cc: Brian Norris <computersforpeace@gmail.com>,
David Woodhouse <dwmw2@infradead.org>,
linux-kernel@vger.kernel.org, linux-mtd@lists.infradead.org
Subject: Re: [PATCH] mtd: mtdram: check offs and len in mtdram->erase
Date: Sat, 3 Oct 2015 11:12:09 +0530 [thread overview]
Message-ID: <20151003054209.GA4428@sudip-pc> (raw)
In-Reply-To: <560F4BF8.7070408@cn.fujitsu.com>
On Sat, Oct 03, 2015 at 11:31:04AM +0800, Dongsheng Yang wrote:
> On 10/03/2015 01:38 AM, Brian Norris wrote:
> >On Fri, Oct 02, 2015 at 03:31:33PM +0530, Sudip Mukherjee wrote:
> >>On Fri, Oct 02, 2015 at 05:39:02PM +0800, Dongsheng Yang wrote:
> >>>On 10/01/2015 12:41 AM, Sudip Mukherjee wrote:
> >>>>We should prevent user to erasing mtd device with an unaligned offset
> >>>>or length.
> >>>>
> >>>>Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org>
> >>>>---
> >>>>
> >>>>I am not sure if I should add the Signed-off-by of
> >>>>Dongsheng Yang <yangds.fnst@cn.fujitsu.com> . He is the original author
> >>>>and he should get the credit for that.
> >>>
> >>>But I had sent a a patch out to fix this problem before your v1.
> >>>
> >>>http://lists.infradead.org/pipermail/linux-mtd/2015-September/062234.html
> >>I didn't know that. I think your v1 was applied.
> >
> >Sorry if I left any confusion. Dongsheng's v1 was applied and reverted.
> >v2 is still under review (and was sent slightly before (?) your v1).
>
> Yea, sorry I should have mentioned it earlier. But I was and am still
> in a vacation, then I did not point it out in time.
>
> Sudip, any comment or test for my patch there is always welcome.
Sorry, I donot know anything about this driver to comment. My main patch
was to fix the build failure. And since by that time your patch was
reverted so I sent another path with my patch and your patch combined
together.
regards
sudip
next prev parent reply other threads:[~2015-10-03 5:42 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-09-30 16:41 [PATCH] mtd: mtdram: check offs and len in mtdram->erase Sudip Mukherjee
2015-10-02 9:39 ` Dongsheng Yang
2015-10-02 10:01 ` Sudip Mukherjee
2015-10-02 17:38 ` Brian Norris
2015-10-03 3:31 ` Dongsheng Yang
2015-10-03 5:42 ` Sudip Mukherjee [this message]
-- strict thread matches above, loose matches on Subject: below --
2015-07-21 8:30 Dongsheng Yang
2015-09-29 22:44 ` Brian Norris
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=20151003054209.GA4428@sudip-pc \
--to=sudipm.mukherjee@gmail.com \
--cc=computersforpeace@gmail.com \
--cc=dwmw2@infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mtd@lists.infradead.org \
--cc=yangds.fnst@cn.fujitsu.com \
/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.