Reviewed-by: Gilad Broner
> Sometimes queries from the device might return a failure so it is
> recommended to retry sending the query, before giving up.
> This change adds a wrapper to retry sending a query attribute,
> in cases where we need to wait longer, before we continue,
> or before
Reviewed-by: Gilad Broner
> Hibern8 exit can be called from 3 different context:
> - ufshcd_hibern8_exit_work
> - ufshcd_ungate_work
> - runtime/system resume
>
> If hibern8 exit fails for some reason then we try to bring the link to
> active state by link startup but this recovery
Reviewed-by: Gilad Broner
> DME commands such as Hibern8 enter/exit and gear switch generate 2
> completion interrupts, one for confirmation that command is received
> by local UniPro and 2nd one is the final confirmation after communication
> with remote UniPro. Currently both of these
The implementation for Open-Channel SSDs is divided into media
management and targets. This patch implements a generic media manager
for open-channel SSDs. After a media manager has been initialized,
single or multiple targets can be instantiated with the media managed as
the backend.
Reviewed-by: Gilad Broner
> The dme_peer get/set attribute commands are prone to errors, therefore
> we add three retries for the UIC command sending.
> Error code returned from ufshcd_send_uic_cmd() is checked, and unless
> it was successful or the retries have finished, another command will be
This target allows an Open-Channel SSD to be exposed asas a block
device.
It implements a round-robin approach for sector allocation,
together with a greedy cost-based garbage collector.
Signed-off-by: Matias Bjørling
---
drivers/lightnvm/Kconfig |7 +
drivers/lightnvm/Makefile |1 +
On Tuesday, October 27, 2015 09:52:50 AM Bjorn Helgaas wrote:
> Hi Suravee,
>
> On Wed, Oct 21, 2015 at 08:52:03AM -0700, Suravee Suthikulpanit wrote:
> > This patch series adds support to setup DMA coherency for PCI device using
> > the ACPI _CCA attribute. According to the ACPI spec, the _CCA
Reviewed-by: Gilad Broner
> If hibern8 enter command fails then UFS link state may be unknown which
> may result into timeout of all the commands issued after failure.
>
> This change does 2 things (for pre-defined number of retry counts) after
> hibern8 enter failure:
> 1. Recovers the UFS link
Reviewed-by: Gilad Broner
> If device raises the exception event in the response to the commands
> sent during the runtime/system PM callbacks, exception event handler
> might run in parallel with PM callbacks and may see unclocked register
> accesses. This change fixes this issue by not
On Tue, Oct 27, 2015 at 1:41 PM, Clemens Ladisch wrote:
> Muni Sekhar wrote:
>> Is it possible to print the timeout value in character driver poll() API?
>
> No. Your driver's poll callback never waits.
>
> Why do you think you need this value?
I need to find out when exactly driver's poll
Reviewed-by: Gilad Broner
> fDeviceInit query response time for some devices is too long that default
> query request timeout of 100ms may not be enough. Experiments show that
> fDeviceInit response sometimes takes 500ms so to be on safer side this
> change sets the timeout to 600ms. Without
The first generation of Open-Channel SSDs is based on NVMe. The NVMe
driver is extended with support for the LightNVM command set.
Detection is made through PCI IDs. Current supported devices are the
qemu nvme simulator and CNEX Labs Westlake SSD. The qemu nvme enables
support through vendor
This patchset implements support for Open-Channel SSDs.
Applies against Jens' for-4.4/drivers and is available through the lkml_v12
branch at:
https://github.com/OpenChannelSSD/linux
Changes since v11:
- Add support for group identifiers.
- Fixed kbuild error on nvme_nvm_unregister.
On Tue, Oct 27, 2015 at 3:27 PM, Tomeu Vizoso
wrote:
> PM domains shouldn't be modified after a device is probed, so set it
> before device registration to be sure of that.
>
> In the future the PM domain pointer will be set through a setter that
> will WARN if the device has been probed already.
Looks OK.
Reviewed-by: Gilad Broner
> Some of the data structures (like response UPIU) and/or its elements
> (unused fields) should be cleared before sending out the respective
> command to UFS device.
>
> This change clears the UPIU response data structure for query commands
> and NOP command
Reviewed-by: Gilad Broner
> When sending a query to the device returns with a timeout error,
> we clear the corresponding bit in the DOORBELL register but
> we don't clear the outstanding_request field as we should.
> This patch fixes this bug.
>
> Signed-off-by: Yaniv Gardi
>
> ---
>
Hello,
On Tue, Oct 27, 2015, at 15:32, Denys Vlasenko wrote:
> I have created a set of semi-automated scripts which look for
> large inlines in the kernel.
>
> Recently I taught it to even generate "git format-patch" patches
> (unfortunately, only for inlines in *.c files, not *.h),
> and here
Hi All,
As discussed in the recent "On-demand device probing" thread and in a Kernel
Summit session earlier today, there is a problem with handling cases where
functional dependencies between devices are involved.
What I mean by a "functional dependency" is when the driver of device B needs
both
> On Thursday 15 October 2015 02:11:57 Huan Wang wrote:
> > > On Wednesday 14 October 2015 10:18:47 Huan Wang wrote:
> > > > > On Thursday 24 September 2015 07:27:10 Huan Wang wrote:
> > > > > > > On Fri, Sep 18, 2015 at 4:38 AM, Huan Wang
> wrote:
> > > > > > Any suggestion? Thanks.
> > > > >
>
On 10/27/2015 10:39 PM, Mark Rutland wrote:
On Tue, Oct 27, 2015 at 01:09:15PM +, John Garry wrote:
On 26/10/2015 14:45, Mark Rutland wrote:
On Mon, Oct 26, 2015 at 10:14:33PM +0800, John Garry wrote:
Add devicetree bindings for HiSilicon SAS driver.
Signed-off-by: John Garry
---
Looks OK.
Reviewed-by: Gilad Broner
> Clear the UFS data structures before sending new request.
>
> The SCSI command is sent to the device within the UFS UPIU request.
> As part of the transfer UPIU preparation, the SCSI command is copied
> to the UPIU structure according to the SCSI command
Hi Suravee,
On Wed, Oct 21, 2015 at 08:52:03AM -0700, Suravee Suthikulpanit wrote:
> This patch series adds support to setup DMA coherency for PCI device using
> the ACPI _CCA attribute. According to the ACPI spec, the _CCA attribute
> is required for ARM64. Therefore, this patch is a pre-req for
Do not reschedule the delayed work used for polling when
the driver is removed, by testing the 'poll_link_needed'
flag.
Avoids this crash:
dsa dsa ethmv2 (unregistering): Link is Down
device eth1 left promiscuous mode
Unable to handle kernel paging request at virtual address bacc5cf6
...
This serie fixes further issues for DSA dynamic unbinding.
Frode Isaksen's patches make usage of delayed work and fixes kernel
crashes when dsa is unbind.
The other patches are simple fixes to permit cleanup and avoid netdev
related crashes.
Frode Isaksen (3):
net: dsa: Use delayed work instead
On 24 October 2015 at 15:57, Rafael J. Wysocki wrote:
> On Thursday, October 22, 2015 03:01:45 PM Tomeu Vizoso wrote:
>> On 22 October 2015 at 03:02, Rafael J. Wysocki wrote:
>> > On Monday, September 21, 2015 04:02:43 PM Tomeu Vizoso wrote:
>> >> When adding platform and AMBA devices, set the
Make sure that we unassign the master_netdev dsa_ptr to make the packet
processing go through the regular Ethernet receive path.
Suggested-by: Florian Fainelli
Signed-off-by: Neil Armstrong
---
net/dsa/dsa.c | 8
1 file changed, 8 insertions(+)
diff --git a/net/dsa/dsa.c
Avoids:
Unable to handle kernel NULL pointer dereference at virtual address 0064
Workqueue: events_power_efficient phy_state_machine
PC is at phy_state_machine+0x28/0x480
Signed-off-by: Frode Isaksen
Signed-off-by: Neil Armstrong
---
drivers/net/phy/phy_device.c | 6 ++
1 file
Simplifies the code and avoids a crash when removing the
module:
dsa dsa ethmv2 (unregistering): Link is Down
device eth1 left promiscuous mode
Unable to handle kernel paging request at virtual address bacc5cf6
...
(run_timer_softirq) from [] (__do_softirq+0xcc/0x320)
(__do_softirq) from []
Em Tue, Oct 27, 2015 at 02:24:03PM +, Liang, Kan escreveu:
>
>
> >
> > Em Sun, Oct 25, 2015 at 03:51:16PM +0100, Jiri Olsa escreveu:
> > > hi,
> > > sending another version of stat scripting.
> > >
> > > v5 changes:
> > > - several patches from v4 already taken
> > > - using u16 for cpu
Upon probe failure or unbinding, add missing dev_put() calls.
Signed-off-by: Neil Armstrong
---
net/dsa/dsa.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/net/dsa/dsa.c b/net/dsa/dsa.c
index acbf854..9240a46 100644
--- a/net/dsa/dsa.c
+++ b/net/dsa/dsa.c
@@ -931,8
Some switch drivers might request interrupts, remap register ranges,
allow such drivers to implement a "remove" callback doing just that.
Suggested-by: Florian Fainelli
Signed-off-by: Neil Armstrong
---
include/net/dsa.h | 1 +
net/dsa/dsa.c | 4
2 files changed, 5 insertions(+)
diff
On Wed, Oct 21, 2015 at 08:52:11AM -0700, Suravee Suthikulpanit wrote:
> This patch adds support for setting up PCI device DMA coherency from
> ACPI _CCA object that should normally be specified in the DSDT node
> of its PCI host bridge.
>
> Signed-off-by: Suravee Suthikulpanit
> CC: Bjorn
On Thu, Oct 22, 2015 at 06:41:30PM -0700, Leonid Yegoshin wrote:
> You can not use R4K CP0_count in SMP (multicore) without core-specific
> adjustment.
> After first power-saving with core clock off or core down the values in
> CP0_count
> in different cores are absolutely different.
>
> Until
From: Sudip Mukherjee
Trailing white space is not accepted in kernel coding style. Remove
them.
Signed-off-by: Sudip Mukherjee
---
drivers/parport/share.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/parport/share.c b/drivers/parport/share.c
index
On 26 October 2015 at 09:16, Geert Uytterhoeven wrote:
> On Mon, Sep 21, 2015 at 4:02 PM, Tomeu Vizoso
> wrote:
>> Walks the OF tree up and finds the closest ancestor that has a struct
>> device associated with it, probing it if isn't bound to a driver yet.
>>
>> The above should ensure that the
From: Sudip Mukherjee
The multi-line comments were not according to the kernel coding style.
Signed-off-by: Sudip Mukherjee
---
drivers/parport/share.c | 69 +++--
1 file changed, 44 insertions(+), 25 deletions(-)
diff --git
From: Geliang Tang
s/regsiter/register/
Signed-off-by: Geliang Tang
Signed-off-by: Sudip Mukherjee
---
drivers/parport/share.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/parport/share.c b/drivers/parport/share.c
index 5ce5ef2..8931696 100644
---
From: Sudip Mukherjee
All symbols were exported at the end of the file but they are supposed
to be exported just after the function. And checkpatch was complaining
about it.
Signed-off-by: Sudip Mukherjee
---
drivers/parport/share.c | 29 +
1 file changed, 13
From: Sudip Mukherjee
user visible strings should not be split as that affects the ability to
grep.
Signed-off-by: Sudip Mukherjee
---
drivers/parport/share.c | 18 --
1 file changed, 8 insertions(+), 10 deletions(-)
diff --git a/drivers/parport/share.c
From: Sudip Mukherjee
checkpatch complains that space is prohibited between function name and
open parenthesis '('.
Signed-off-by: Sudip Mukherjee
---
drivers/parport/share.c | 94 -
1 file changed, 47 insertions(+), 47 deletions(-)
diff --git
From: Sudip Mukherjee
checkpatch was complaining about NULL comparisons.
Signed-off-by: Sudip Mukherjee
---
drivers/parport/share.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/parport/share.c b/drivers/parport/share.c
index 441333b..ccd7df4 100644
---
On Thu, Oct 22, 2015 at 10:25:26AM +0800, Peter Hung wrote:
> Hi Johan,
>
> Johan Hovold 於 2015/9/14 下午 09:33 寫道:
> > On Tue, Jul 21, 2015 at 09:58:19AM +0800, Peter Hung wrote:
>
> >>4. RS422 Mode
> >> 1. The RTS mode is dont care.
> >> 2. Set M2/M1/M0 as 0/0/0
> >
>
From: Sudip Mukherjee
If kmalloc() or kzalloc() fails we will get sufficient messages in the logs,
no need to print these extra messages.
Signed-off-by: Sudip Mukherjee
---
drivers/parport/share.c | 13 +++--
1 file changed, 3 insertions(+), 10 deletions(-)
diff --git
On Tue, Oct 27, 2015 at 03:40:38PM +0100, Simon Guinot wrote:
> Should this driver not be moved under the "MFD GPIO expanders" menu ?
Maybe. It is still part of the PCH (Platform Controller Hub) as far as I
can tell so I would not call it a GPIO expander either ;-)
--
To unsubscribe from this
From: Sudip Mukherjee
checkpatch was complaining about braces for single statement block.
Signed-off-by: Sudip Mukherjee
---
drivers/parport/share.c | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/parport/share.c b/drivers/parport/share.c
index
From: Sudip Mukherjee
It is not an usual practise to assign some value to a variable in the if
test condition.
Signed-off-by: Sudip Mukherjee
---
drivers/parport/share.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/parport/share.c b/drivers/parport/share.c
From: Sudip Mukherjee
Code should be indented using tabs and not by space.
Signed-off-by: Sudip Mukherjee
---
drivers/parport/share.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/parport/share.c b/drivers/parport/share.c
index 840c730..388c138 100644
---
v7 LPAE multi-platform defconfig is based on v7 multi-platform
defconfig and adds LPAE support.
This defconfig is verified on LS1021A which enables GIANFAR,
I2C, WATCHDOG, AUDIO, EDMA and DSPI drivers, etc.
Signed-off-by: Alison Wang
---
arch/arm/configs/multi_v7_lpae_defconfig | 168
Hi Greg,
A small series of checkpatch cleanup on parport. This is the first set.
Its only cleanup and no other change is done. All the patches are also
sent along with this pull request for your convenience.
The following changes since commit 32b88194f71d6ae7768a29f87fbba454728273ee:
Linux
Em Sun, Oct 25, 2015 at 03:51:47PM +0100, Jiri Olsa escreveu:
> Synthesizing needed stat record data for report/script:
> - cpu/thread maps
> - stat config
After this it gets a bit better, but then I expected that to specify an
event I would be able to use:
[root@zoo linux]# perf stat record
Hi,
this is v11 of an attempt to make it easier for devices to remain in
runtime PM when the system goes to sleep, mainly to reduce the time
spent resuming devices.
For this, we interpret the absence of all PM callback implementations as
it being safe to do direct_complete, so their ancestors
Adds a function that tells whether a device is already bound to a
driver.
This is needed to warn when there is an attempt to change the PM domain
of a device that has finished probing already. The reason why we want to
enforce that is because in the general case that can cause problems and
also
>From "git log --oneline include/acpi/acpi_bus.h", I see the convention is
to capitalize "ACPI:" in the summary (also applies to other patches in this
series).
On Wed, Oct 21, 2015 at 08:52:04AM -0700, Suravee Suthikulpanit wrote:
> From: Jeremy Linton
>
> ACPI configurations can now mark
On 26 October 2015 at 13:56, Daniel Kurtz wrote:
> Hi Tomeu,
>
> On Wed, Oct 21, 2015 at 11:34 PM, Tomeu Vizoso
> wrote:
>> If a suitable prepare callback cannot be found for a given device and
>> its driver has no PM callbacks at all, assume that it can go direct to
>> complete when the system
Have dev_pm_ops.prepare return 1 for USB devices and ports so that USB
devices can remain runtime-suspended when the system goes to a sleep
state, if their wakeup state is correct and they have runtime PM enabled.
Signed-off-by: Tomeu Vizoso
Reviewed-by: Ulf Hansson
Acked-by: Alan Stern
---
Adds a function that sets the pointer to dev_pm_domain in struct device
and that warns if the device has already finished probing. The reason
why we want to enforce that is because in the general case that can
cause problems and also that we can simplify code quite a bit if we can
always assume
On Tue, Oct 27, 2015 at 04:23:43PM +0200, Mika Westerberg wrote:
> On Tue, Oct 27, 2015 at 11:06:58AM +0100, Linus Walleij wrote:
> > > +menu "ISA GPIO drivers"
> > > +
> > > +config GPIO_104_IDIO_16
> > > + tristate "ACCES 104-IDIO-16 GPIO support"
> > > + depends on X86
> > > +
Hi
26.10.2015, 08:53, "Jiri Kosina" :
> From: Jiri Kosina
>
> w1_process() calls try_to_freeze(), but the thread doesn't mark itself
> freezable through set_freezable(), so the try_to_freeze() call is useless.
I believe it is better to mark it freezable, what do you think?
Its task is useless
On Tue, Oct 27, 2015 at 01:09:15PM +, John Garry wrote:
> On 26/10/2015 14:45, Mark Rutland wrote:
> >On Mon, Oct 26, 2015 at 10:14:33PM +0800, John Garry wrote:
> >>Add devicetree bindings for HiSilicon SAS driver.
> >>
> >>Signed-off-by: John Garry
> >>---
> >>
If a suitable prepare callback cannot be found for a given device and
its driver has no PM callbacks at all, assume that it can go direct to
complete when the system goes to sleep.
The reason for this is that there's lots of devices in a system that do
no PM at all and there's no reason for them
Hi Suravee,
On Wed, Oct 21, 2015 at 08:52:10AM -0700, Suravee Suthikulpanit wrote:
> This patch move of_pci_dma_configure() to a more generic
> pci_dma_configure(), which can be extended by non-OF code (e.g. ACPI).
> -void of_pci_dma_configure(struct pci_dev *pci_dev)
> -{
> - struct device
On Tue, Oct 20, 2015 at 04:09:19PM +0200, Dirk Steinmetz wrote:
> Attempting to hardlink to an unsafe file (e.g. a setuid binary) from
> within an unprivileged user namespace fails, even if CAP_FOWNER is held
> within the namespace. This may cause various failures, such as a gentoo
> installation
On 20 October 2015 at 15:26, wrote:
> From: Axel Haslam
>
> prepare generic power domain init function parameters to accept a pointer
> to the states structure that describes the possible states that a power
> domain can enter.
>
> Also, as most platforms are not initializing states or
Hi,
I have created a set of semi-automated scripts which look for
large inlines in the kernel.
Recently I taught it to even generate "git format-patch" patches
(unfortunately, only for inlines in *.c files, not *.h),
and here are they for 4.3.0-rc1 - i.e. current Linus tree.
Submitting 300+
On 10/26/2015 07:16 PM, Mark Brown wrote:
On Mon, Oct 26, 2015 at 10:47:41AM -0500, Andrew F. Davis wrote:
On 10/25/2015 07:43 PM, Mark Brown wrote:
.of_compatible = "mediatek,mt6397-regulator",
This is in the MFD, this is not used in actual systems.
Not sure what you mean by "actual
PM domains shouldn't be modified after a device is probed, so set it
before device registration to be sure of that.
In the future the PM domain pointer will be set through a setter that
will WARN if the device has been probed already.
Signed-off-by: Tomeu Vizoso
Cc: Geert Uytterhoeven
---
Em Sun, Oct 25, 2015 at 03:51:45PM +0100, Jiri Olsa escreveu:
> Add 'perf stat record' command support. It creates simple
> (header only) perf.data file ATM.
Huh? Couldn't we have the tools providing a sensible message at this point?
[root@zoo linux]# rm -f perf.data
[root@zoo linux]# perf stat
Hi Ley,
On Fri, Oct 23, 2015 at 06:27:09PM +0800, Ley Foon Tan wrote:
> This is the 12th version of patch set to add support for Altera PCIe host
> controller with MSI feature on Altera FPGA device families. This patchset
> mainly resovle the comments from Bjorn.
>
>
> This patchset is based on
>
> Em Sun, Oct 25, 2015 at 03:51:16PM +0100, Jiri Olsa escreveu:
> > hi,
> > sending another version of stat scripting.
> >
> > v5 changes:
> > - several patches from v4 already taken
> > - using u16 for cpu number in cpu_map_event
> > - renamed PERF_RECORD_HEADER_ATTR_UPDATE to
>
On Tue, Oct 27, 2015 at 11:06:58AM +0100, Linus Walleij wrote:
> > +menu "ISA GPIO drivers"
> > +
> > +config GPIO_104_IDIO_16
> > + tristate "ACCES 104-IDIO-16 GPIO support"
> > + depends on X86
> > + help
> > + Enables GPIO support for the ACCES 104-IDIO-16 family.
> >
On 10/23/2015 09:31 AM, Masahiro Yamada wrote:
> This UART driver should not depend on the console. They should be
> orthogonal.
>
> Surround the earlycon code with CONFIG_SERIAL_EARLYCON conditional
> and rip off "depends on SERIAL_8250_CONSOLE".
Reviewed-by: Peter Hurley
PS - While
On Tue, 2015-10-27 at 07:31 -0500, Josh Cartwright wrote:
> Okay, yes, I like the first suggestion better as well, I've included a
> patch below that does just that. I hope you don't mind me turning it
> into a Suggested-by :).
>
> Thanks for taking a look!
> Josh
> @@ -6969,7 +6969,7 @@
On 10/23/2015 09:31 AM, Masahiro Yamada wrote:
> SERIAL_8250_INGENIC depends on SERIAL_8250_CONSOLE, which already
> selects SERIAL_EARLYCON.
>
> This line is redundant.
Reviewed-by: Peter Hurley
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a
Hi Balbi,
On 26/10/15 16:55, Felipe F. Tonello wrote:
> Patch 7 has changes on how to transmit IN USB requests. It implements a FIFO
> of pre-allocated usb requests and uses then as needed, instead of allocating
> then on demand. This is my initial implementation and is open for
> suggestions and
Em Tue, Oct 27, 2015 at 02:51:54PM +0100, Jiri Olsa escreveu:
> On Tue, Oct 27, 2015 at 10:44:40AM -0300, Arnaldo Carvalho de Melo wrote:
> > > I think we could easily add record specific event for this
> > > once it's needed.. there's plenty of free numbers in user
> > > events area ;-)
> >
On October 27, 2015 4:40:42 PM GMT+05:30, Dan Carpenter
wrote:
>On Tue, Oct 27, 2015 at 11:45:18AM +0200, Leon Romanovsky wrote:
>> On Tue, Oct 27, 2015 at 11:19 AM, Dan Carpenter
>> wrote:
>> > On Sun, Oct 25, 2015 at 01:21:11PM +0200, Leon Romanovsky wrote:
>> >> On Sun, Oct 25, 2015 at
On Tue, Oct 27, 2015 at 09:53:30AM +0900, Jerome Glisse wrote:
> On Tue, Oct 27, 2015 at 09:47:48AM +0900, Jerome Glisse wrote:
> > On Mon, Oct 26, 2015 at 12:07:17PM -0400, Konrad Rzeszutek Wilk wrote:
> > > On Mon, Aug 31, 2015 at 06:13:03PM -0400, j.gli...@gmail.com wrote:
> > > > From: Jérôme
Hi Prashanth,
On 27/10/15 00:21, Prakash, Prashanth wrote:
Hi Sudeep,
While testing with these patches everything looked fine except that we
are flattening all the LPI
states even if it is disabled. I added a simple check to fix it. Please
let me know if you have any
feedback on the same.
On Tue, Oct 27, 2015 at 05:02:16PM +1100, Alexey Kardashevskiy wrote:
> >+unsigned long dma_get_page_shift(struct device *dev)
> >+{
> >+struct iommu_table *tbl = get_iommu_table_base(dev);
> >+if (tbl)
> >+return tbl->it_page_shift;
>
>
> All PCI devices have this
string_get_size() can't really handle huge block sizes, especially
blk_size > U32_MAX but string_get_size() interface states the opposite.
Change blk_size from u64 to u32 to reflect the reality.
Signed-off-by: Vitaly Kuznetsov
---
include/linux/string_helpers.h | 2 +-
lib/string_helpers.c
string_get_size() loses precision when there is a remainder for
blk_size / divisor[units] and size is big enough. E.g
string_get_size(8192, 4096, STRING_UNITS_10, ...) returns "32.7 MB"
while it is supposed to return "33.5 MB". For some artificial inputs
the result can be ridiculously wrong, e.g.
Add a couple of simple tests for string_get_size().
Signed-off-by: Vitaly Kuznetsov
---
Changes since v1:
- Make test_string_get_size_one() check both STRING_UNITS_2 and
STRING_UNITS_10 in one shot [Rasmus Villemoes]
- Check U64_MAX instead of some-very-big-number.
---
Linux always lies about your storage size when it has 4k sectors and its
size is big enough. E.g. a device with 8192 4k sectors will be reported as
"32.7 MB/32 MiB" while "33.5 MB/32 MiB" is expected. This series is
supposed to fix the issue by fixing calculation precision in
string_get_size() for
27.10.2015 03:52, Andy Lutomirski пишет:
> On Mon, Oct 26, 2015 at 4:45 AM, Stas Sergeev wrote:
>> 26.10.2015 04:25, Andy Lutomirski пишет:
>>> This is take 2 at fixing x86 64-bit signals wrt SS. After a lot of
>>> thought, this is not controlled by any flags -- I would much prefer
>>> to avoid
On Tue, Oct 27, 2015 at 09:21:13PM +0800, Yang Yingliang wrote:
> In multi-core system, if the clock is not sync perfectly, it
> will make cycle_last that recorded by CPU-A is a little more
> than cycle_now that read by CPU-B.
If that is happening, that sounds like a hardware and/or firmware bug.
On 23/10/15 23:57, Mark Brown wrote:
> On Thu, Oct 22, 2015 at 08:09:38PM +0100, Damien Horsley wrote:
>> On 19/10/15 18:47, Mark Brown wrote:
>>> On Mon, Oct 12, 2015 at 01:40:29PM +0100, Damien Horsley wrote:
>
>>> The APIs here all seem a bit odd - for example the enable API taking a
>>>
Hi Masahiro,
On 10/24/2015 12:17 AM, Masahiro Yamada wrote:
> The IER has already been masked in early_serial8250_setup(), there is
> no reason to save and restore it every time early_serial8250_write()
> is called.
>
> Signed-off-by: Masahiro Yamada
> ---
>
>
On Tue, 2015-10-27 at 12:42 +0200, Jarkko Sakkinen wrote:
> n Mon, Oct 26, 2015 at 07:44:39AM +0200, Jarkko Sakkinen wrote:
> > On Sun, Oct 25, 2015 at 03:21:31PM -0400, Mimi Zohar wrote:
> > > On Sat, 2015-10-24 at 15:42 +0300, Jarkko Sakkinen wrote:
> > > > Added 'hashalg=' option for selecting
Hi Masahiro,
On 10/26/2015 12:11 AM, Masahiro Yamada wrote:
> Currently, 8-bit (MMIO) and 32-bit (MMIO32) register interfaces are
> supported for the 8250 console, but the 16-bit (MMIO16) is not.
> The 8250 UART device on my board is connected to a 16-bit bus
> and my main motivation is to use
On Tue, Oct 27, 2015 at 10:44:40AM -0300, Arnaldo Carvalho de Melo wrote:
> Em Tue, Oct 27, 2015 at 02:30:43PM +0100, Jiri Olsa escreveu:
> > On Tue, Oct 27, 2015 at 10:16:05AM -0300, Arnaldo Carvalho de Melo wrote:
> > > Em Sun, Oct 25, 2015 at 03:51:27PM +0100, Jiri Olsa escreveu:
> > > > Adding
Em Tue, Oct 27, 2015 at 02:30:43PM +0100, Jiri Olsa escreveu:
> On Tue, Oct 27, 2015 at 10:16:05AM -0300, Arnaldo Carvalho de Melo wrote:
> > Em Sun, Oct 25, 2015 at 03:51:27PM +0100, Jiri Olsa escreveu:
> > > Adding stat config event to pass/store stat config data,
> > > so report tools
On 10/27/2015 03:11 PM, Grygorii Strashko wrote:
On 10/27/2015 02:22 PM, Franklin S Cooper Jr. wrote:
On 10/16/2015 10:55 AM, Andy Shevchenko wrote:
On Fri, Oct 16, 2015 at 6:47 PM, Franklin S Cooper Jr.
wrote:
On 10/16/2015 10:45 AM, Andy Shevchenko wrote:
On Fri, Oct 16, 2015 at 6:29
From: Michal Hocko
Date: Tue, 27 Oct 2015 13:26:47 +0100
> On Mon 26-10-15 12:56:19, Johannes Weiner wrote:
> [...]
>> Or any other combination of pick-and-choose consumers. But
>> honestly, nowadays all our paths are lockless, and the counting is an
>> atomic-add-return with a per-cpu batch
On Tue, Oct 27, 2015 at 10:16:05AM -0300, Arnaldo Carvalho de Melo wrote:
> Em Sun, Oct 25, 2015 at 03:51:27PM +0100, Jiri Olsa escreveu:
> > Adding stat config event to pass/store stat config data,
> > so report tools (report/script) know how to interpret
> > stat data.
> >
> > The config data
On 09/10/15 03:23, Yong Wu wrote:
[...]
+#include
+#include
+#include
+#include
+#include
+#include
+#include
+#include
+#include
+#include
+#include
+#include
+#include
Nit: ordering?
+#include
+#include "io-pgtable.h"
[...]
+struct mtk_iommu_data {
+ void __iomem
On 09/10/15 03:23, Yong Wu wrote:
[...]
+static int mtk_smi_probe(struct platform_device *pdev)
+{
+ struct device *dev = >dev;
+ struct mtk_smi_data *smidata;
+ int ret;
+
+ if (!dev->pm_domain)
+ return -EPROBE_DEFER;
+
+ smidata = devm_kzalloc(dev,
Mask the cycle values before subtraction. So we can use this
validation while the clocksource mask is not 64-bits.
Signed-off-by: Yang Yingliang
Cc: Thomas Gleixner
---
kernel/time/timekeeping_internal.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
In multi-core system, if the clock is not sync perfectly, it
will make cycle_last that recorded by CPU-A is a little more
than cycle_now that read by CPU-B. With the negative result,
hrtimer_update_base() return a huge and wrong time. It leads
to the cpu can not finish the while loop in
In multi-core system, if the clock is not sync perfectly, it
will make cycle_last that recorded by CPU-A is a little more
than cycle_now that read by CPU-B. With the negative result,
hrtimer_update_base() return a huge and wrong time. It leads
to the cpu can not finish the while loop in
Em Sun, Oct 25, 2015 at 03:51:27PM +0100, Jiri Olsa escreveu:
> Adding stat config event to pass/store stat config data,
> so report tools (report/script) know how to interpret
> stat data.
>
> The config data are stored in 'tag|value' way to allow
> easy extension and backward compatibility.
I
901 - 1000 of 2552 matches
Mail list logo