Hi!
Short:
1)
greg k-h> don't add marker lines like this that provide no relevancy to anyone
else.
"no relevancy to anyone else" -- strange: it "has relevance for Hyper-V
sysadmins"
For search by keywords in web archives of this mail list
1b)
greg k-h> If you want to refer to a public bug
* Sricharan R [140626 00:29]:
> From: R Sricharan
>
> There is a IRQ crossbar device in the soc, which
> maps the irq requests from the peripherals to the
> mpu interrupt controller's inputs. The Peripheral irq
> requests are connected to only one crossbar
> input and the output of the crossbar
* Sricharan R [140626 00:14]:
> This series does some cleanups, fixes for handling two interrupts
> getting mapped twice to same crossbar and provides support for
> hardwired IRQ and crossbar definitions.
>
> On certain platforms such as DRA7, SPIs 0, 1, 2, 3, 5, 6, 10,
> 131, 132, 133 are
On Wed, Jun 25, 2014 at 08:56:35PM +0200, Matias Bj??rling wrote:
> Only the review. Do you want me to change it to a Reviewed-by?
I've not really don't a formal review of the latest patch yet. I plan
to do it, and will give me Reviewed-by-tag then. For now I don't think
I'm actually
> Von: Bryan Wu [mailto:coolo...@gmail.com]
> Gesendet: Donnerstag, 26. Juni 2014 01:11
> An: Werner, Andreas
> Cc: lkml; Samuel Ortiz; Lee Jones; w...@iguana.be; linux-
> watch...@vger.kernel.org; rpur...@rpsys.net; Linux LED Subsystem;
> Thumshirn, Johannes Tobias
> Betreff: Re: [PATCH v2 3/3]
On Wed, Jun 25, 2014 at 06:17:57PM -0300, Arnaldo Carvalho de Melo wrote:
SNIP
> >
> > Merge remote-tracking branch 'jolsa/perf/urgent' into ptool-v0.5
> >
> > commit a93f0e551af9e194db38bfe16001e17a3a1d189a
> > Author: Simon Que
> > Date: Mon Jun 16 11:32:09 2014 -0700
> >
> >
On 25 June 2014 20:23, Mike Turquette wrote:
>
> Peter,
>
> Just FYI, I'm trying to reverse the trend of prepending double
> underscores for functions that are used by clock providers. That stuff
> started out small and sort of grew out of control ;-)
I'm looking at rebasing Rabin's patches for
Hi Ben,
On Wed, 2014-06-25 at 11:24 -0400, Benjamin LaHaise wrote:
> I finally have some time to look at this patch in detail. I'd rather do the
> below variant that does what Kent suggested. Mike, can you confirm that
> this fixes the issue you reported? It's on top of my current aio-next
On Wed, Jun 25, 2014 at 10:11:47PM +0200, Borislav Petkov wrote:
> On Wed, Jun 25, 2014 at 04:42:54PM -0300, Arnaldo Carvalho de Melo wrote:
> > Probably will come in a separate patchkit that is dependent on this,
> > but not strictly related?
>
> Yeah, so roughly speaking, we want to carve out
On 26 June 2014 00:32, Stephen Boyd wrote:
>> + cpu_reg = regulator_get_optional(cpu_dev, "cpu0");
> I don't think this driver should be using regulator_get_optional() (Mark
> B. please correct me if I'm wrong). I doubt a supply is actually
> optional for CPUs, just some DTs aren't
* Nishanth Menon [140625 15:29]:
> On 06/25/2014 07:56 AM, Roger Quadros wrote:
> > The SATA and USB PHYs need the 1.8V and 3.3V supplies.
> > The PHY drivers/framework don't yet support regulator
> > supply so we have to keep these regulators always-on till
> > then.
> >
> > Signed-off-by:
From: R Sricharan
There is a IRQ crossbar device in the soc, which maps the
irq requests from the peripherals to the mpu interrupt
controller's inputs. The gic provides the support for such
IPs in the form of routable-irqs. So adding the property
here to gic node.
Signed-off-by: Sricharan R
From: R Sricharan
There is a IRQ crossbar device in the soc, which
maps the irq requests from the peripherals to the
mpu interrupt controller's inputs. The Peripheral irq
requests are connected to only one crossbar
input and the output of the crossbar is connected to only one
controller's input
This series introduces DT support for crossbar device and
changes dra7 peripherals to use crossbar number instead of irq.
This depends on below driver fixes and cleanup series.
http://marc.info/?l=linux-omap=140376708127157=2
[V2] Rebased on 3.15 mainline.
[V3] Added ti,irqs-skip property and
From: Nishanth Menon
When, in the system due to varied reasons, interrupts might be unusable
due to hardware behavior, but register maps do exist, then those interrupts
should be skipped while mapping irq to crossbars.
Signed-off-by: Nishanth Menon
Signed-off-by: Sricharan R
Acked-by: Santosh
From: Nishanth Menon
If irq_of_parse_and_map is executed twice, the same crossbar is mapped to two
different GIC interrupts. This is completely undesirable. Instead, check
if the requested crossbar event is pre-allocated and provide that GIC
mapping back to caller if already allocated.
This series does some cleanups, fixes for handling two interrupts
getting mapped twice to same crossbar and provides support for
hardwired IRQ and crossbar definitions.
On certain platforms such as DRA7, SPIs 0, 1, 2, 3, 5, 6, 10,
131, 132, 133 are direct wired to hardware blocks bypassing
From: Nishanth Menon
crossbar_of_init always returns -ENOMEM in case of errors.
There can be other causes of failure like invalid data from
DT. So return a appropriate error value for that case.
Signed-off-by: Nishanth Menon
Signed-off-by: Sricharan R
Acked-by: Santosh Shilimkar
---
From: Nishanth Menon
Using err1,2,3,4 etc makes it hard to ensure a new exit path in the
middle will not result in spurious changes, so rename the error paths
as per the function it does.
Signed-off-by: Nishanth Menon
Signed-off-by: Sricharan R
Acked-by: Santosh Shilimkar
---
From: Nishanth Menon
There is absolutely no need for crossbar driver to expose functions and
variables into global namespace. So make them all static
Also fix a couple of checkpatch warnings.
Fixes sparse warnings:
drivers/irqchip/irq-crossbar.c:129:29: warning: symbol
From: Nishanth Menon
IS_ERR_VALUE makes sense only *if* there could be valid values in
negative error range. But in the cases that we do use it, there is no
such case. Just remove the same.
Signed-off-by: Nishanth Menon
Signed-off-by: Sricharan R
Acked-by: Santosh Shilimkar
---
If crossbar_of_init returns with a error, then set the cb pointer
to null.
Signed-off-by: Sricharan R
Acked-by: Santosh Shilimkar
---
drivers/irqchip/irq-crossbar.c |2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/irqchip/irq-crossbar.c b/drivers/irqchip/irq-crossbar.c
index
From: Nishanth Menon
The current crossbar description does not include the description
required for the consumer of the crossbar, a.k.a devices whoes events
pass through the crossbar into the GIC interrupt controller.
So, provide documentation for the same.
Signed-off-by: Nishanth Menon
From: Nishanth Menon
On certain platforms such as DRA7, SPIs 0, 1, 2, 3, 5, 6, 10, 131,
132, 133 are direct wired to hardware blocks bypassing crossbar.
This quirky implementation is *NOT* supposed to be the expectation
of crossbar hardware usage. However, these are already marked in our
Adding kerneldoc for unmap callback function.
Signed-off-by: Sricharan R
Acked-by: Santosh Shilimkar
---
drivers/irqchip/irq-crossbar.c | 11 +++
1 file changed, 11 insertions(+)
diff --git a/drivers/irqchip/irq-crossbar.c b/drivers/irqchip/irq-crossbar.c
index a8c6156..518d712
From: Nishanth Menon
Currently we attempt to map any crossbar value to an IRQ, however,
this is not correct from hardware perspective. There is a max crossbar
event number upto which hardware supports. So describe the same in
device tree using 'ti,max-crossbar-sources' property and use it to
From: Nishanth Menon
This is a basic check to ensure that crossbar register needs to be
written. This ensures that we have a common check which is used in
both map and unmap logic.
Signed-off-by: Nishanth Menon
Signed-off-by: Sricharan R
Acked-by: Santosh Shilimkar
---
From: Nishanth Menon
Adding missing properties for kerneldoc (@write) and cleanup
of harmless warnings while we are here.
kerneldoc warnings:
Warning(drivers/irqchip/irq-crossbar.c:27): missing initial short description
on line:
* struct crossbar_device: crossbar device description
From: Nishanth Menon
Since crossbar is s/w configurable, the initial settings of the
crossbar cannot be assumed to be sane. This implies that:
a) On initialization all un-reserved crossbars must be initialized to
a known 'safe' value.
b) When unmapping the interrupt, the safe value must be
From: Nishanth Menon
Today '0' is actually reserved, but may not be the same in the future.
So, use a flag to mark the GIC interrupts that are reserved.
Signed-off-by: Nishanth Menon
Signed-off-by: Sricharan R
Acked-by: Santosh Shilimkar
---
drivers/irqchip/irq-crossbar.c |5 +++--
1
From: Nishanth Menon
Reverse the search algorithm to ensure that address mapping and IRQ
allocation logics are proper. This makes the below bugs visible sooner.
class 1. address space errors -> example:
reg =
ti,max-irqs = is a wrong parameter
class 2: irq-reserved list - which decides which
On Wed, 25 Jun 2014 12:20:16 -0700, Dan Malek wrote:
>
> On Jun 25, 2014, at 7:48 AM, Pantelis Antoniou
> wrote:
>
> >>> +int of_transaction_commit(struct of_transaction *oft);
> >>
> >> How about of_transaction_apply()?
> >>
> >
> > Sure.
>
> LOL! Panto, you give in too easily :)
>
>
On Wed, Jun 25, 2014 at 09:43:27PM -0700, Josh Triplett wrote:
> How would you feel about a patch that flagged long lines with a warning
> in patch mode, but not in file mode?
Just tell people to not send patches to just cleanup checkpath.pl
warnings in code they don't actually maintain or make
On Wed, Jun 25, 2014 at 05:08:28PM +0800, Herbert Xu wrote:
> On Mon, Jun 23, 2014 at 09:11:29AM +0200, Stephan Mueller wrote:
> > As reported by a static code analyzer, the code for the ordering of
> > the linked list can be simplified.
> >
> > Reported-by: kbuild test robot
> > Signed-off-by:
Hi all,
The powerpc allyesconfig is again broken more than usual.
Changes since 20140625:
New tree: usb-serial
The staging tree still had its build failure for which I disabled a driver.
The akpm-current tree gained a build failure for which I reverted a commit.
Non-merge commits (relative
Hi Kishon,
On Thu, Jun 26, 2014 at 02:10:02PM +0800, Kishon Vijay Abraham I wrote:
> Hi Pratyush,
>
> On Thursday 26 June 2014 11:07 AM, Pratyush Anand wrote:
> > Hi Kishon,
> >
> > Few things, if you can help me to understand:
> >
> > On Wed, Jun 25, 2014 at 11:26 PM, Kishon Vijay Abraham I
Hi Jon,
On Thu, Jun 26, 2014 at 4:46 AM, Jon Ringle wrote:
> The only thing I needed to do was to remove the
> BROKEN dependency on USB_MUSB_DA8XX. Is there anything else I would
> have to do besides just removing BROKEN to get this accepted back into
> the kernel source tree?
I tried to remove
On Mon, 23 Jun 2014, Joe Perches wrote:
Remove the now unnecessary memset too.
Signed-off-by: Joe Perches
Looks good, thanks
Acked-by: Govindarajulu Varadarajan <_gov...@gmx.com>
---
drivers/net/ethernet/cisco/enic/vnic_dev.c | 8 +++-
1 file changed, 3 insertions(+), 5 deletions(-)
Hi Andrew,
After merging the akpm tree, today's linux-next build (powerpc
ppc44x_defconfig) failed like this:
kernel/printk/printk.c: In function 'log_buf_add_cpu':
kernel/printk/printk.c:269:37: error: 'CONFIG_LOG_CPU_MAX_BUF_SHIFT' undeclared
(first use in this function)
#define
On Wed, Jun 25, 2014 at 05:54:50PM +0200, Jiri Kosina wrote:
> > - wait_event_freezable(khubd_wait,
> > + wait_event_freezable(khubd_wait,
> > ({ kgr_task_safe(current);
> >
> > The changes are somewhat ugly with all the kgraft crap leaking into plces
> > like jbd and
Msi or Msi-x interrupts assign vectors from a CPU while the number of
CUPs is more than 8. __assign_irq_vector() is responsible for assigning vectors
from low-numbered CPUs by default.
If there are a lot of Msi or Msi-x interrupts, it will cause the
low-numbered CPUs out of
On Thu, 2014-06-26 at 11:06 +1000, Dave Chinner wrote:
> Your particular use case can be handled by directing your benchmark
> at a filesystem mount point and unmounting the filesystem in between
> benchmark runs. There is no ned to adding kernel functionality for
> somethign that can be so
Hi Pratyush,
On Thursday 26 June 2014 11:07 AM, Pratyush Anand wrote:
> Hi Kishon,
>
> Few things, if you can help me to understand:
>
> On Wed, Jun 25, 2014 at 11:26 PM, Kishon Vijay Abraham I
> wrote:
>> In DRA7, the cpu sees 32bit address, but the pcie controller can see only
>> 28bit
>>
Currently, LONG_LINE just informs the user about the line length,
leaving them to shorten the line. Too many users run checkpatch and
blindly follow its recommendation by splitting long lines, which almost
invariably results in worse code. On rare occasions, the line-width
limit encourages
Currently, LONG_LINE just informs the user about the line length,
leaving them to shorten the line. Too many users run checkpatch and
blindly follow its recommendation by splitting long lines, which almost
invariably results in worse code. On rare occasions, the line-width
limit encourages
Hi Pratyush,
On Thursday 26 June 2014 11:07 AM, Pratyush Anand wrote:
Hi Kishon,
Few things, if you can help me to understand:
On Wed, Jun 25, 2014 at 11:26 PM, Kishon Vijay Abraham I kis...@ti.com
wrote:
In DRA7, the cpu sees 32bit address, but the pcie controller can see only
28bit
On Thu, 2014-06-26 at 11:06 +1000, Dave Chinner wrote:
Your particular use case can be handled by directing your benchmark
at a filesystem mount point and unmounting the filesystem in between
benchmark runs. There is no ned to adding kernel functionality for
somethign that can be so easily
Msi or Msi-x interrupts assign vectors from a CPU while the number of
CUPs is more than 8. __assign_irq_vector() is responsible for assigning vectors
from low-numbered CPUs by default.
If there are a lot of Msi or Msi-x interrupts, it will cause the
low-numbered CPUs out of
On Wed, Jun 25, 2014 at 05:54:50PM +0200, Jiri Kosina wrote:
- wait_event_freezable(khubd_wait,
+ wait_event_freezable(khubd_wait,
({ kgr_task_safe(current);
The changes are somewhat ugly with all the kgraft crap leaking into plces
like jbd and freezer and usb.
Hi Andrew,
After merging the akpm tree, today's linux-next build (powerpc
ppc44x_defconfig) failed like this:
kernel/printk/printk.c: In function 'log_buf_add_cpu':
kernel/printk/printk.c:269:37: error: 'CONFIG_LOG_CPU_MAX_BUF_SHIFT' undeclared
(first use in this function)
#define
On Mon, 23 Jun 2014, Joe Perches wrote:
Remove the now unnecessary memset too.
Signed-off-by: Joe Perches j...@perches.com
Looks good, thanks
Acked-by: Govindarajulu Varadarajan _gov...@gmx.com
---
drivers/net/ethernet/cisco/enic/vnic_dev.c | 8 +++-
1 file changed, 3 insertions(+), 5
Hi Jon,
On Thu, Jun 26, 2014 at 4:46 AM, Jon Ringle j...@ringle.org wrote:
The only thing I needed to do was to remove the
BROKEN dependency on USB_MUSB_DA8XX. Is there anything else I would
have to do besides just removing BROKEN to get this accepted back into
the kernel source tree?
I
Hi Kishon,
On Thu, Jun 26, 2014 at 02:10:02PM +0800, Kishon Vijay Abraham I wrote:
Hi Pratyush,
On Thursday 26 June 2014 11:07 AM, Pratyush Anand wrote:
Hi Kishon,
Few things, if you can help me to understand:
On Wed, Jun 25, 2014 at 11:26 PM, Kishon Vijay Abraham I kis...@ti.com
Hi all,
The powerpc allyesconfig is again broken more than usual.
Changes since 20140625:
New tree: usb-serial
The staging tree still had its build failure for which I disabled a driver.
The akpm-current tree gained a build failure for which I reverted a commit.
Non-merge commits (relative
On Wed, Jun 25, 2014 at 05:08:28PM +0800, Herbert Xu wrote:
On Mon, Jun 23, 2014 at 09:11:29AM +0200, Stephan Mueller wrote:
As reported by a static code analyzer, the code for the ordering of
the linked list can be simplified.
Reported-by: kbuild test robot fengguang...@intel.com
On Wed, Jun 25, 2014 at 09:43:27PM -0700, Josh Triplett wrote:
How would you feel about a patch that flagged long lines with a warning
in patch mode, but not in file mode?
Just tell people to not send patches to just cleanup checkpath.pl
warnings in code they don't actually maintain or make
On Wed, 25 Jun 2014 12:20:16 -0700, Dan Malek dan.ma...@konsulko.com wrote:
On Jun 25, 2014, at 7:48 AM, Pantelis Antoniou
pantelis.anton...@konsulko.com wrote:
+int of_transaction_commit(struct of_transaction *oft);
How about of_transaction_apply()?
Sure.
LOL! Panto, you
From: Nishanth Menon n...@ti.com
Reverse the search algorithm to ensure that address mapping and IRQ
allocation logics are proper. This makes the below bugs visible sooner.
class 1. address space errors - example:
reg = a size_b
ti,max-irqs = is a wrong parameter
class 2: irq-reserved list -
From: Nishanth Menon n...@ti.com
Adding missing properties for kerneldoc (@write) and cleanup
of harmless warnings while we are here.
kerneldoc warnings:
Warning(drivers/irqchip/irq-crossbar.c:27): missing initial short description
on line:
* struct crossbar_device: crossbar device description
From: Nishanth Menon n...@ti.com
Since crossbar is s/w configurable, the initial settings of the
crossbar cannot be assumed to be sane. This implies that:
a) On initialization all un-reserved crossbars must be initialized to
a known 'safe' value.
b) When unmapping the interrupt, the safe value
From: Nishanth Menon n...@ti.com
Today '0' is actually reserved, but may not be the same in the future.
So, use a flag to mark the GIC interrupts that are reserved.
Signed-off-by: Nishanth Menon n...@ti.com
Signed-off-by: Sricharan R r.sricha...@ti.com
Acked-by: Santosh Shilimkar
Adding kerneldoc for unmap callback function.
Signed-off-by: Sricharan R r.sricha...@ti.com
Acked-by: Santosh Shilimkar santosh.shilim...@ti.com
---
drivers/irqchip/irq-crossbar.c | 11 +++
1 file changed, 11 insertions(+)
diff --git a/drivers/irqchip/irq-crossbar.c
From: Nishanth Menon n...@ti.com
Currently we attempt to map any crossbar value to an IRQ, however,
this is not correct from hardware perspective. There is a max crossbar
event number upto which hardware supports. So describe the same in
device tree using 'ti,max-crossbar-sources' property and
From: Nishanth Menon n...@ti.com
The current crossbar description does not include the description
required for the consumer of the crossbar, a.k.a devices whoes events
pass through the crossbar into the GIC interrupt controller.
So, provide documentation for the same.
Signed-off-by: Nishanth
From: Nishanth Menon n...@ti.com
This is a basic check to ensure that crossbar register needs to be
written. This ensures that we have a common check which is used in
both map and unmap logic.
Signed-off-by: Nishanth Menon n...@ti.com
Signed-off-by: Sricharan R r.sricha...@ti.com
Acked-by:
From: Nishanth Menon n...@ti.com
On certain platforms such as DRA7, SPIs 0, 1, 2, 3, 5, 6, 10, 131,
132, 133 are direct wired to hardware blocks bypassing crossbar.
This quirky implementation is *NOT* supposed to be the expectation
of crossbar hardware usage. However, these are already marked in
From: Nishanth Menon n...@ti.com
There is absolutely no need for crossbar driver to expose functions and
variables into global namespace. So make them all static
Also fix a couple of checkpatch warnings.
Fixes sparse warnings:
drivers/irqchip/irq-crossbar.c:129:29: warning: symbol
If crossbar_of_init returns with a error, then set the cb pointer
to null.
Signed-off-by: Sricharan R r.sricha...@ti.com
Acked-by: Santosh Shilimkar santosh.shilim...@ti.com
---
drivers/irqchip/irq-crossbar.c |2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/irqchip/irq-crossbar.c
From: Nishanth Menon n...@ti.com
IS_ERR_VALUE makes sense only *if* there could be valid values in
negative error range. But in the cases that we do use it, there is no
such case. Just remove the same.
Signed-off-by: Nishanth Menon n...@ti.com
Signed-off-by: Sricharan R r.sricha...@ti.com
From: Nishanth Menon n...@ti.com
crossbar_of_init always returns -ENOMEM in case of errors.
There can be other causes of failure like invalid data from
DT. So return a appropriate error value for that case.
Signed-off-by: Nishanth Menon n...@ti.com
Signed-off-by: Sricharan R r.sricha...@ti.com
From: Nishanth Menon n...@ti.com
Using err1,2,3,4 etc makes it hard to ensure a new exit path in the
middle will not result in spurious changes, so rename the error paths
as per the function it does.
Signed-off-by: Nishanth Menon n...@ti.com
Signed-off-by: Sricharan R r.sricha...@ti.com
From: Nishanth Menon n...@ti.com
If irq_of_parse_and_map is executed twice, the same crossbar is mapped to two
different GIC interrupts. This is completely undesirable. Instead, check
if the requested crossbar event is pre-allocated and provide that GIC
mapping back to caller if already
This series does some cleanups, fixes for handling two interrupts
getting mapped twice to same crossbar and provides support for
hardwired IRQ and crossbar definitions.
On certain platforms such as DRA7, SPIs 0, 1, 2, 3, 5, 6, 10,
131, 132, 133 are direct wired to hardware blocks bypassing
From: Nishanth Menon n...@ti.com
When, in the system due to varied reasons, interrupts might be unusable
due to hardware behavior, but register maps do exist, then those interrupts
should be skipped while mapping irq to crossbars.
Signed-off-by: Nishanth Menon n...@ti.com
Signed-off-by:
This series introduces DT support for crossbar device and
changes dra7 peripherals to use crossbar number instead of irq.
This depends on below driver fixes and cleanup series.
http://marc.info/?l=linux-omapm=140376708127157w=2
[V2] Rebased on 3.15 mainline.
[V3] Added ti,irqs-skip property and
From: R Sricharan r.sricha...@ti.com
There is a IRQ crossbar device in the soc, which
maps the irq requests from the peripherals to the
mpu interrupt controller's inputs. The Peripheral irq
requests are connected to only one crossbar
input and the output of the crossbar is connected to only one
From: R Sricharan r.sricha...@ti.com
There is a IRQ crossbar device in the soc, which maps the
irq requests from the peripherals to the mpu interrupt
controller's inputs. The gic provides the support for such
IPs in the form of routable-irqs. So adding the property
here to gic node.
* Nishanth Menon n...@ti.com [140625 15:29]:
On 06/25/2014 07:56 AM, Roger Quadros wrote:
The SATA and USB PHYs need the 1.8V and 3.3V supplies.
The PHY drivers/framework don't yet support regulator
supply so we have to keep these regulators always-on till
then.
Signed-off-by: Roger
On 26 June 2014 00:32, Stephen Boyd sb...@codeaurora.org wrote:
+ cpu_reg = regulator_get_optional(cpu_dev, cpu0);
I don't think this driver should be using regulator_get_optional() (Mark
B. please correct me if I'm wrong). I doubt a supply is actually
optional for CPUs, just some DTs
On Wed, Jun 25, 2014 at 10:11:47PM +0200, Borislav Petkov wrote:
On Wed, Jun 25, 2014 at 04:42:54PM -0300, Arnaldo Carvalho de Melo wrote:
Probably will come in a separate patchkit that is dependent on this,
but not strictly related?
Yeah, so roughly speaking, we want to carve out the code
Hi Ben,
On Wed, 2014-06-25 at 11:24 -0400, Benjamin LaHaise wrote:
I finally have some time to look at this patch in detail. I'd rather do the
below variant that does what Kent suggested. Mike, can you confirm that
this fixes the issue you reported? It's on top of my current aio-next
On 25 June 2014 20:23, Mike Turquette mturque...@linaro.org wrote:
Peter,
Just FYI, I'm trying to reverse the trend of prepending double
underscores for functions that are used by clock providers. That stuff
started out small and sort of grew out of control ;-)
I'm looking at rebasing
On Wed, Jun 25, 2014 at 06:17:57PM -0300, Arnaldo Carvalho de Melo wrote:
SNIP
Merge remote-tracking branch 'jolsa/perf/urgent' into ptool-v0.5
commit a93f0e551af9e194db38bfe16001e17a3a1d189a
Author: Simon Que s...@chromium.org
Date: Mon Jun 16 11:32:09 2014 -0700
Von: Bryan Wu [mailto:coolo...@gmail.com]
Gesendet: Donnerstag, 26. Juni 2014 01:11
An: Werner, Andreas
Cc: lkml; Samuel Ortiz; Lee Jones; w...@iguana.be; linux-
watch...@vger.kernel.org; rpur...@rpsys.net; Linux LED Subsystem;
Thumshirn, Johannes Tobias
Betreff: Re: [PATCH v2 3/3]
On Wed, Jun 25, 2014 at 08:56:35PM +0200, Matias Bj??rling wrote:
Only the review. Do you want me to change it to a Reviewed-by?
I've not really don't a formal review of the latest patch yet. I plan
to do it, and will give me Reviewed-by-tag then. For now I don't think
I'm actually
* Sricharan R r.sricha...@ti.com [140626 00:14]:
This series does some cleanups, fixes for handling two interrupts
getting mapped twice to same crossbar and provides support for
hardwired IRQ and crossbar definitions.
On certain platforms such as DRA7, SPIs 0, 1, 2, 3, 5, 6, 10,
131, 132,
* Sricharan R r.sricha...@ti.com [140626 00:29]:
From: R Sricharan r.sricha...@ti.com
There is a IRQ crossbar device in the soc, which
maps the irq requests from the peripherals to the
mpu interrupt controller's inputs. The Peripheral irq
requests are connected to only one crossbar
input
Hi!
Short:
1)
greg k-h don't add marker lines like this that provide no relevancy to anyone
else.
no relevancy to anyone else -- strange: it has relevance for Hyper-V
sysadmins
For search by keywords in web archives of this mail list
1b)
greg k-h If you want to refer to a public bug
On Thu, Jun 26, 2014 at 10:08:46AM +0800, Ming Lei wrote:
Firstly this patch supports more than one virtual queues for virtio-blk
device.
Secondly this patch maps the virtual queue to blk-mq's hardware queue.
With this approach, both scalability and performance can be improved.
On Wed, Jun 25, 2014 at 11:05:56PM -0600, Jens Axboe wrote:
On 2014-06-25 20:08, Ming Lei wrote:
Hi,
These patches try to support multi virtual queues(multi-vq) in one
virtio-blk device, and maps each virtual queue(vq) to blk-mq's
hardware queue.
With this approach, both scalability and
On Mon, Jun 02, 2014 at 03:09:16PM +0100, Jan Beulich wrote:
In particular seeing zero in eft-month is problematic, as it results
in -1 (converted to unsigned int, i.e. yielding 0x) getting
passed to rtc_year_days(), where the value gets used as an array index
(normally resulting in a
In some conditions (echo or particular sequence of special
characters), on buffer push, the tty layer calls the write operation
while we are holding the spinlock. This means deadlock within the same
process on kernels version 3.12. It seems not a problem on recent
kernel, but the patch still
Looks good, thanks Rickard.
Acked-by: Jack Wang xjtu...@gmail.com
On 06/01/2014 03:13 PM, Rickard Strandqvist wrote:
There is a risk that the variable will be used without being initialized.
This was largely found by using a static code analysis program called
cppcheck.
Signed-off-by:
On Thu, 2014-06-26 at 09:46 +0200, Federico Vaga wrote:
In some conditions (echo or particular sequence of special
characters), on buffer push, the tty layer calls the write operation
while we are holding the spinlock. This means deadlock within the same
process on kernels version 3.12. It
Hi,
On 06/26/2014 02:28 AM, Rickard Strandqvist wrote:
If it's not obvious, I do all my fixes without changing the previous
intent. But obviously it is not always right, rather it is one of main
reasons to fix this type of error :)
Yes it is obvious, your patch was correct (it didn't modify
Thanks Rickard,
From my point of view, looks good, but I'd like to get review from Anand
(cc-ed).
Anand, could you share your opinion?
Regards,
Jack
On 06/25/2014 04:01 PM, Rickard Strandqvist wrote:
A struct member variable is set to different values without having used in
between.
This
Hi!
2014-03-09
https://git.kernel.org/cgit/linux/kernel/git/stable/linux-stable.git/commit/drivers/net/hyperv?id=99d3016de4f2a29635f5382b0e9bd0e5f2151487
hyperv: Change the receive buffer size for legacy hosts
+
Haiyang Zhang write on May 27, 2014 6:22 PM:
I ( Haiyang Zhang) will ask the
On 06/26/2014 10:09 AM, Jack Wang wrote:
Thanks Rickard,
From my point of view, looks good, but I'd like to get review from Anand
(cc-ed).
I would like to add that I noticed that this fields is only set and appears to
be never used,
maybe it could be completely removed.
Regards,
Maurizio
On Thu, Jun 26, 2014 at 3:45 PM, Michael S. Tsirkin m...@redhat.com wrote:
On Thu, Jun 26, 2014 at 10:08:46AM +0800, Ming Lei wrote:
Firstly this patch supports more than one virtual queues for virtio-blk
device.
Secondly this patch maps the virtual queue to blk-mq's hardware queue.
With
On 06/25/2014 02:42 PM, One Thousand Gnomes wrote:
On Wed, 25 Jun 2014 13:06:59 +0200
Jiri Slaby jsl...@suse.cz wrote:
This is based on Udo's text which was augmented in this patch.
Signed-off-by: Jiri Slaby jsl...@suse.cz
Cc: Udo Seidel udosei...@gmx.de
Cc: Vojtech Pavlik vojt...@suse.cz
601 - 700 of 1288 matches
Mail list logo
