From: "Li, Yi" <yi1.li@linux.intel.com>
To: "Luis R. Rodriguez" <mcgrof@kernel.org>
Cc: atull@kernel.org, Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Daniel Wagner <wagi@monom.org>,
David Woodhouse <dwmw2@infradead.org>,
rafal@milecki.pl, Arend Van Spriel <arend.vanspriel@broadcom.com>,
"Rafael J. Wysocki" <rjw@rjwysocki.net>,
Moritz Fischer <moritz.fischer@ettus.com>,
Petr Mladek <pmladek@suse.com>,
Johannes Berg <johannes.berg@intel.com>,
Emmanuel Grumbach <emmanuel.grumbach@intel.com>,
Luciano Coelho <luciano.coelho@intel.com>,
Kalle Valo <kvalo@codeaurora.org>,
Andy Lutomirski <luto@kernel.org>,
"AKASHI, Takahiro" <takahiro.akashi@linaro.org>,
David Howells <dhowells@redhat.com>,
Peter Jones <pjones@redhat.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
linux-fpga@vger.kernel.org
Subject: Re: [PATCHv2 0/3] Enable no_cache flag to driver_data
Date: Fri, 26 May 2017 16:05:43 -0500 [thread overview]
Message-ID: <d92603b1-02bc-9aa6-7cb0-c96514a7c6d7@linux.intel.com> (raw)
In-Reply-To: <CAB=NE6WW4V=iYvaGFzhdUsQz9CsFad90yB-71HeP3gqYm08FVg@mail.gmail.com>
hi Luis
On 5/25/2017 5:43 PM, Luis R. Rodriguez wrote:
> On Thu, May 25, 2017 at 3:30 PM, Li, Yi <yi1.li@linux.intel.com> wrote:
>> This patch is for "disabling the cache" for streaming and iwlwifi case,
>> adding the test to verify the cache function should be a separate patch,
>> right? I can look more into the cache part.
> How can we know cache was disabled without first testing what having
> cache enabled could be like ?
Understand the point, adding the test for cache enabled still cannot
cover the disable cache part though:-). Please give me couple days to
think about it. Maybe will first have a new patch to test the cache
part, likely through the debugfs as you suggested; then amend this
series to complete the cache disabling test part. Thanks for the insight.
Yi
>
> Luis
> --
> To unsubscribe from this list: send the line "unsubscribe linux-fpga" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2017-05-27 1:16 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-05-20 6:46 [PATCHv2 0/3] Enable no_cache flag to driver_data yi1.li
2017-05-20 6:46 ` [PATCHv2 1/3] firmware_class: move NO_CACHE from private to driver_data_req_params yi1.li
2017-05-20 6:46 ` [PATCHv2 2/3] iwlwifi: use DRIVER_DATA_REQ_NO_CACHE for driver_data yi1.li
2017-05-20 6:46 ` [PATCHv2 3/3] test: add no_cache to driver_data load tester yi1.li
2017-05-24 19:03 ` [PATCHv2 0/3] Enable no_cache flag to driver_data Luis R. Rodriguez
2017-05-24 20:32 ` Luis R. Rodriguez
2017-05-25 22:30 ` Li, Yi
2017-05-25 22:43 ` Luis R. Rodriguez
2017-05-26 21:05 ` Li, Yi [this message]
2017-05-26 21:13 ` Luis R. Rodriguez
2017-06-06 19:31 ` Li, Yi
2017-06-07 17:59 ` Luis R. Rodriguez
2017-06-07 21:00 ` Li, Yi
2017-06-07 23:02 ` Luis R. Rodriguez
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=d92603b1-02bc-9aa6-7cb0-c96514a7c6d7@linux.intel.com \
--to=yi1.li@linux.intel.com \
--cc=arend.vanspriel@broadcom.com \
--cc=atull@kernel.org \
--cc=dhowells@redhat.com \
--cc=dwmw2@infradead.org \
--cc=emmanuel.grumbach@intel.com \
--cc=gregkh@linuxfoundation.org \
--cc=johannes.berg@intel.com \
--cc=kvalo@codeaurora.org \
--cc=linux-fpga@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=luciano.coelho@intel.com \
--cc=luto@kernel.org \
--cc=mcgrof@kernel.org \
--cc=moritz.fischer@ettus.com \
--cc=pjones@redhat.com \
--cc=pmladek@suse.com \
--cc=rafal@milecki.pl \
--cc=rjw@rjwysocki.net \
--cc=takahiro.akashi@linaro.org \
--cc=wagi@monom.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.