public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Greg KH <gregkh@linuxfoundation.org>
To: Youquan Song <youquan.song@intel.com>
Cc: dan.j.williams@intel.com, vinod.koul@intel.com,
	andriy.shevchenko@intel.com, mika.westerberg@intel.com,
	linux-kernel@vger.kernel.org,
	Youquan Song <youquan.song@linux.intel.com>
Subject: Re: [PATCH 2/2] dma: calculate the data tranferred by 8250
Date: Fri, 11 Oct 2013 06:32:00 -0700	[thread overview]
Message-ID: <20131011133200.GA32689@kroah.com> (raw)
In-Reply-To: <1381527738-9339-3-git-send-email-youquan.song@intel.com>

On Fri, Oct 11, 2013 at 05:42:18PM -0400, Youquan Song wrote:
> When using UART transfers data by DMA mode, but it always shows 0 at 
> /sys/class/dma/dma0chan*/bytes_transferred.
> 
> Call the new function to calculate how many the data has been transferred
>  after doing it by DMA mode. 

How nice, you just leaked to userspace the size of passwords and other
sensitive things typed into a serial console :(

No, we can't do this, we fixed up this problem in other places we leaked
the transmition of serial data through proc and sysfs files, let's not
add new ones and go backwards in security.

Sorry, we can't do this, and I really doubt that it matters.

greg k-h

      reply	other threads:[~2013-10-11 13:33 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-10-11 21:42 DMA: Calculate how many data transferred by DMA Youquan Song
2013-10-11 21:42 ` [PATCH 1/2] dma: Add interface to calculate data transferred Youquan Song
2013-10-11 10:22   ` Shevchenko, Andriy
2013-10-11 13:33   ` Greg KH
2013-10-13 15:26     ` Vinod Koul
2013-10-15 18:31       ` Youquan Song
2013-10-15 15:30         ` Greg KH
2013-10-15 15:55         ` Dan Williams
2013-10-15 16:17           ` Greg KH
2013-10-16  5:38           ` Vinod Koul
2013-10-16  8:36             ` Shevchenko, Andriy
2013-10-16  7:57               ` Vinod Koul
2013-10-16  9:13                 ` Shevchenko, Andriy
2013-10-16 14:12                   ` Greg KH
2013-10-16 15:07                     ` Vinod Koul
2013-10-16 18:17                       ` Dan Williams
2013-10-11 21:42   ` [PATCH 2/2] dma: calculate the data tranferred by 8250 Youquan Song
2013-10-11 13:32     ` Greg KH [this message]

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=20131011133200.GA32689@kroah.com \
    --to=gregkh@linuxfoundation.org \
    --cc=andriy.shevchenko@intel.com \
    --cc=dan.j.williams@intel.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mika.westerberg@intel.com \
    --cc=vinod.koul@intel.com \
    --cc=youquan.song@intel.com \
    --cc=youquan.song@linux.intel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox