Commit-ID: 317c2ce77d8ab73c24f4fb9c75e5bb441fbe3e30
Gitweb: http://git.kernel.org/tip/317c2ce77d8ab73c24f4fb9c75e5bb441fbe3e30
Author: Josh Poimboeuf
AuthorDate: Fri, 23 Sep 2016 16:49:39 -0500
Committer: Ingo Molnar
CommitDate: Sat, 24 Sep 2016 09:30:03 +0200
x86/alternatives: Add sta
Commit-ID: 907241dccb4ce5d9413cf3c030b32b0cfc184914
Gitweb: http://git.kernel.org/tip/907241dccb4ce5d9413cf3c030b32b0cfc184914
Author: Mark Rutland
AuthorDate: Fri, 23 Sep 2016 18:24:07 +0100
Committer: Ingo Molnar
CommitDate: Sat, 24 Sep 2016 09:35:06 +0200
thread_info: Use unsigned l
>> @@ -57,27 +57,29 @@ static int joydump_connect(struct gameport *gameport,
>> struct gameport_driver *dr
>>> unsigned long flags;
>>> unsigned char u;
>>
>>> - printk(KERN_INFO "joydump: ,-- START
>>> .\n");
>>> - printk(KERN_INFO "joydump: | Dumpin
On 09/24/2016 11:15 AM, Joe Perches wrote:
On Sat, 2016-09-24 at 17:55 +0200, Jean Delvare wrote:
Would it make sense to explicitly set the enum values, or add them as
comments, to make such look-ups easier?
If you want to create enum->#ENUM structs and
"const char *" lookup functions, please
On 24/09/16 17:18, SF Markus Elfring wrote:
>> Applied to the togreg branch of iio.git - initially pushed out
>> as testing for the autobuilders to play with it.
>
> Thanks for your positive response.
>
>
> Do you see any need to improve the software situation around
> the macro "BITS_TO_LONGS"
On Sat, Sep 24, 2016 at 06:32:29PM +0200, SF Markus Elfring wrote:
> >> @@ -57,27 +57,29 @@ static int joydump_connect(struct gameport *gameport,
> >> struct gameport_driver *dr
> >>> unsigned long flags;
> >>> unsigned char u;
> >>
> >>> - printk(KERN_INFO "joydump: ,-- STAR
On Sat, Sep 24, 2016 at 03:48:54PM +0200, SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Sat, 24 Sep 2016 14:30:44 +0200
>
> Add a definition for the macro "pr_fmt" so that its information can be used
> for consistent message output.
>
> Signed-off-by: Markus Elfring
> ---
> drivers/i
>> @@ -57,27 +57,29 @@ static int joydump_connect(struct gameport *gameport,
>> struct gameport_driver *dr
>>> unsigned long flags;
>>> unsigned char u;
>>
>>> - printk(KERN_INFO "joydump: ,-- START
>>> .\n");
>>> - printk(KERN_INFO "joydump: | Dumpin
On Sat, Sep 24, 2016 at 03:46:21PM +0200, SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Sat, 24 Sep 2016 13:50:20 +0200
>
> * A multiplication for the size determination of a memory allocation
> indicated that an array data structure should be processed.
> Thus use the corresponding
On Sat, 2016-09-24 at 18:32 +0200, SF Markus Elfring wrote:
> > > @@ -57,27 +57,29 @@ static int joydump_connect(struct gameport *gameport,
> > > struct gameport_driver *dr
> > > unsigned long flags;
> > > unsigned char u;
> >
> >
> > > - printk(KERN_INFO "joydump: ,-- START
>> +#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
>> +
>
> Why did you split it off from the patch where you attempt to use it?
I chose a special patch granularity once again.
> What good does this change do on its own?
I find that it is a preparation. - If this addition could not be accepted,
t
On 09/23/2016 05:42 PM, Christoph Hellwig wrote:
Jens,
can we at least get patches 1 and 2 in while pondering the fate
of the right interface for patch 3?
Yes definitely, I have no beef with the first two patches. I'll add them
for 4.9.
--
Jens Axboe
[Added linux-scsi to the cc list.]
I will resubmit an updated version of this patch.
On 09/23/2016 02:34 PM, Lee Duncan wrote:
> Chris Leech and I are taking over open-iscsi
> maintenance from Mike Christie.
>
> Signed-off-by: Lee Duncan
> ---
> MAINTAINERS | 3 ++-
> 1 file changed, 2 inserti
Due to my lack of familiarity with the how git send-email works, I've
unintentionally had my name listed as the first 'from' whereas I
intended Allan Chou to be listed as the first 'from' in the patch. If
anyone can correct this on my behalf, I would appreciate it.
Regards,
Chris
On Sat, Sep 24,
On Sat, 2016-09-24 at 18:55 +0200, SF Markus Elfring wrote:
> > > +#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
> > Why did you split it off from the patch where you attempt to use it?
> I chose a special patch granularity once again.
> > What good does this change do on its own?
> I find that it is
On 21/09/16 21:15, William Breathitt Gray wrote:
> Quadrature encoders, such as rotary encoders and linear encoders, are
> devices which are capable of encoding the relative position and
> direction of motion of a shaft. This patch introduces several IIO
> constants for supporting quadrature encode
>> * A multiplication for the size determination of a memory allocation
>> indicated that an array data structure should be processed.
>> Thus use the corresponding function "kmalloc_array".
>>
>
> -ENOPARSE.
Do you find an other wording better?
The script "checkpatch.pl" can point informat
Hi Jonathan,
> Am 24.09.2016 um 18:07 schrieb Jonathan Cameron :
>
> On 23/09/16 13:41, H. Nikolaus Schaller wrote:
>> The tsc2007 chip not only has a resistive touch screen controller but
>> also an external AUX adc imput which can be used for an ambient
>> light sensor, battery voltage monitori
> The desired output should not be broken in conversion, which you did.
I dared to change another source code place a bit too much perhaps.
> Do you know how syslog works and why the transformation is not correct.
I imagine that there are chances to improve the software situation a bit more,
ar
On 21/09/16 21:16, William Breathitt Gray wrote:
> The ACCES 104-QUAD-8 is a general purpose quadrature encoder
> counter/interface board. The 104-QUAD-8 is capable of monitoring the
> outputs of eight encoders via four on-board LSI/CSI LS7266R1 24-bit
> dual-axis quadrature counter chips. Core fun
On 24/09/16 18:07, H. Nikolaus Schaller wrote:
> Hi Jonathan,
>
>> Am 24.09.2016 um 18:07 schrieb Jonathan Cameron :
>>
>> On 23/09/16 13:41, H. Nikolaus Schaller wrote:
>>> The tsc2007 chip not only has a resistive touch screen controller but
>>> also an external AUX adc imput which can be used f
(adding Jes Sorensen to recipients)
On Sat, 2016-09-24 at 11:35 -0500, Larry Finger wrote:
> I have patches that makes HAL_DEF_WOWLAN be a no-op for the rest of the
> drivers,
> and one that sets the enum values for that particular statement to hex
> values. I
> also looked at the other large
Hi Jonathan,
Sorry for the (long) delay, I did not have time to work on it. I'll
mainly work in my free time now.
Keep in mind this patch was proposed based on the v2 of the ADC patches.
Since then, substantial changes have been made and I'm working on
rebasing this series of patches on the v6, s
On Fri, Sep 23, 2016 at 6:37 PM, Herbert, Marc wrote:
> On 03/09/2016 11:10, Dmitry Torokhov wrote:
>> I was thinking if we kernel could post
>> "conditions" (maybe simple stings) that it waits for, and userspace
>> could unlock these "conditions". One of them might be "firmware
>> available".
>
>
Currently, if info is null, the dev_err message is dereferencing an
uninitialized module pointer. Instead, it should use codec->dev pointer
in dev_err call and better align with other err msg in this function.
Also, ret variable might be used uninitialized in a specific case.
Avoid using it this
Hi Jonathan,
> Am 24.09.2016 um 19:26 schrieb Jonathan Cameron :
>
> On 24/09/16 18:07, H. Nikolaus Schaller wrote:
>> Hi Jonathan,
>>
>>> Am 24.09.2016 um 18:07 schrieb Jonathan Cameron :
>>>
>>> On 23/09/16 13:41, H. Nikolaus Schaller wrote:
The tsc2007 chip not only has a resistive touc
On Sat, Sep 24, 2016 at 10:07 AM, H. Nikolaus Schaller
wrote:
> Hi Jonathan,
>
>> Am 24.09.2016 um 18:07 schrieb Jonathan Cameron :
>>
>> On 23/09/16 13:41, H. Nikolaus Schaller wrote:
>>> The tsc2007 chip not only has a resistive touch screen controller but
>>> also an external AUX adc imput whic
>> I find that it is a preparation. - If this addition could not be accepted,
>> the following update step would also be discussed under an other perspective,
>> wouldn't it?
>
> It's purposeless, creates unnecessary patches to review
> and generally wastes other people's time.
I have got an othe
On Sat, Sep 24, 2016 at 4:10 AM, SF Markus Elfring
wrote:
> From: Markus Elfring
> Date: Sat, 24 Sep 2016 12:50:31 +0200
>
> Adjust a jump label according to the current Linux coding style convention.
> Thus replace the identifier "out" by "unlock".
No, just no.
>
> Signed-off-by: Markus Elfrin
On Sat, Sep 24, 2016 at 09:02:02AM +0800, Chao Yu wrote:
> On 2016/9/24 5:11, Jaegeuk Kim wrote:
> > When getting EIO while handling orphan inodes, we can get some dirty node
> > pages. Then, f2fs_write_node_pages() called by iput(node_inode) will try
> > to flush node pages. But in this case, we s
Hi Dmitry,
> Am 24.09.2016 um 19:44 schrieb Dmitry Torokhov :
>
> On Sat, Sep 24, 2016 at 10:07 AM, H. Nikolaus Schaller
> wrote:
>> Hi Jonathan,
>>
>>> Am 24.09.2016 um 18:07 schrieb Jonathan Cameron :
>>>
>>> On 23/09/16 13:41, H. Nikolaus Schaller wrote:
The tsc2007 chip not only has a
On Sat, Sep 24, 2016 at 4:08 AM, SF Markus Elfring
wrote:
> From: Markus Elfring
> Date: Sat, 24 Sep 2016 12:42:45 +0200
>
> * A multiplication for the size determination of a memory allocation
> indicated that an array data structure should be processed.
> Thus use the corresponding function
On Sat, 2016-09-24 at 19:45 +0200, SF Markus Elfring wrote:
> > It's purposeless, creates unnecessary patches to review
> > and generally wastes other people's time.
> I have got an other opinion about this.
Nice for you, not nice for others that have to act on
your patch proposals to get them for
This prevent future potential pointer leaks when an unprivileged eBPF
program will read a pointer value from its context. Even if
is_valid_access() returns a pointer type, the eBPF verifier replace it
with UNKNOWN_VALUE. The register value that contains a kernel address is
then allowed to leak. Mor
On Sat, Sep 24, 2016 at 11:32:08AM +0800, Chao Yu wrote:
> On 2016/9/24 8:52, Jaegeuk Kim wrote:
> > On Sat, Sep 24, 2016 at 08:46:54AM +0800, Chao Yu wrote:
> >> Hi Jaegeuk,
> >>
> >> On 2016/9/24 7:53, Jaegeuk Kim wrote:
> >>> Hi Chao,
> >>>
> >>> The basic rule is to stop every operations once C
> So we have to multiply twice now, once in kmalloc_array, the second
> time in memcpy().
It looks so in the source code after the suggested refactoring.
> No, thank you.
Would you like to check any further if a specific compiler implementation
will still optimise common subexpressions as you d
On Sat, Sep 24, 2016 at 02:00:41PM +0800, Chao Yu wrote:
> On 2016/9/21 8:45, Jaegeuk Kim wrote:
> > @@ -259,40 +290,26 @@ static inline void fill_node_footer_blkaddr(struct
> > page *page, block_t blkaddr)
> > {
> > struct f2fs_checkpoint *ckpt = F2FS_CKPT(F2FS_P_SB(page));
> > struct f2
Hi Dimitry,
Sorry for the (long) delay, I did not have time to work on it. I'll
mainly work in my free time now.
On 20/07/2016 19:25, Dmitry Torokhov wrote:
> Hi Quentin,
>
> On Wed, Jul 20, 2016 at 10:29:10AM +0200, Quentin Schulz wrote:
>> This adds support for Allwinner SoCs' (A10, A13 and A3
On Sat, Sep 24, 2016 at 8:27 AM, James Smart wrote:
>
> add u64 number parser
>
> Prior patch revised to use kasprintf.
> Modified match_number to use kasprintf as well
Why would you do this? It's insane. kasprintf() is not the right thing
at all to use for anything like this. It appears that you
On Sat, Sep 24, 2016 at 08:16:16PM +0200, SF Markus Elfring wrote:
> > So we have to multiply twice now, once in kmalloc_array, the second
> > time in memcpy().
>
> It looks so in the source code after the suggested refactoring.
>
>
> > No, thank you.
>
> Would you like to check any further if
Hi Quentin,
On Sat, Sep 24, 2016 at 08:26:08PM +0200, Quentin Schulz wrote:
> Hi Dimitry,
>
> Sorry for the (long) delay, I did not have time to work on it. I'll
> mainly work in my free time now.
>
> On 20/07/2016 19:25, Dmitry Torokhov wrote:
> > Hi Quentin,
> >
> > On Wed, Jul 20, 2016 at 10
> When a chorus of voices says to you that you are wasting
> their time, perhaps you listen to their song.
I am listening to some degree. - I guess that I am making some decisions
and conclusions in a way that you would prefer to be different.
I know that my update suggestions won't be liked by a
On 09/24/2016 02:00 PM, Pavel Machek wrote:
Hi!
Instead I propose the following:
n_iled_max =
31 - (priv->led_iref * 2400 / priv->leds_max_current +
!!(priv->led_iref * 2400 % priv->leds_max_current))
div_round_up?
Exactly. Thanks.
--
Best regards,
Jacek Anaszewski
On the rk3288 USB host-only port (the one that's not the OTG-enabled
port) the PHY can get into a bad state when a wakeup is asserted (not
just a wakeup from full system suspend but also a wakeup from
autosuspend).
We can get the PHY out of its bad state by asserting its "port reset",
but unfortun
At this stage it is the only "full features" and well verified
USB EHCI controller in this platform. More review is always necessary.
Although V7 is almost fine but a forget a dummy function for the phy
system.
changelog:
v7
add the forgot dummy phy_reset() for the generic phy is disabled.
v
I forget to add a dummy function in case the CONFIG_GENERIC_PHY
is disabled.
Signed-off-by: Randy Li
---
include/linux/phy/phy.h | 7 +++
1 file changed, 7 insertions(+)
diff --git a/include/linux/phy/phy.h b/include/linux/phy/phy.h
index ee1bed7..78bb0d7 100644
--- a/include/linux/phy/phy.
The "host1" port (AKA the dwc2 port that isn't the OTG port) on rk3288
has a hardware errata that causes everything to get confused when we get
a remote wakeup. We'll use the reset that's in the CRU to reset the
port when it's in a bad state.
Note that we add the reset to both dwc2 controllers ev
On 23/09/2016 19:36, Rob Herring wrote:
On Tue, Sep 20, 2016 at 06:48:58PM +0800, John Garry wrote:
Add support for hip07 chipset to bindings.
Chipset hip07 has v2 hw.
The sas-v2 quirk amt is expanded to cover hip07.
Signed-off-by: John Garry
Signed-off-by: Xiang Chen
---
Documentation/dev
On 09/23/2016 03:10 PM, Norbert Lange wrote:
Hi,
I wanted to reply to the Linux mailing List (your patch thread), I
dont know exactly where since the addresses are blanked out.
Just tell me where I should move this discussion.
The board can use both USB peripherals (haven`t tested the OTG Port
> Again, what is the purpose of the changes?
I came also along a few source code places where their maintainers
requested the usage of a function like "kmalloc_array".
How do you think about to clarify the consistent use of programming
interfaces for Linux a bit more?
> Are you working on the c
On 09/24/2016 12:32 PM, Joe Perches wrote:
(adding Jes Sorensen to recipients)
On Sat, 2016-09-24 at 11:35 -0500, Larry Finger wrote:
I have patches that makes HAL_DEF_WOWLAN be a no-op for the rest of the drivers,
and one that sets the enum values for that particular statement to hex values. I
On 09/24/2016 01:58 PM, Pavel Machek wrote:
Hi!
+Example
+===
+
+led1: ncp5623@38 {
+ #address-cells = <1>;
+ #size-cells = <0>;
+ compatible = "onnn,ncp5623";
+ reg = <0x38>;
+ onnn,led-iref-microamp = <10>;
+
+ led1r@0 {
+ label = "ncp:pow
> I'm not necessarily against this change which does perhaps clarify the code
> ever so slightly,
Thanks for another bit of positive feedback.
> but I am interested to know where 'current Linux coding style convention'
> comes from?
How often do you check the status of a document like "CodingS
Linus,
please pull the latest irq-urgent-for-linus git tree from:
git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git
irq-urgent-for-linus
Three fixes for irq core and irq chip drivers
- Do not set the irq type if type is NONE. Fixes a boot regression on
various SoCs
- Use th
Linus,
please pull the latest efi-urgent-for-linus git tree from:
git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git
efi-urgent-for-linus
Two fixes for EFI/PAT:
- A 32bit overflow bug in the PAT code which was unearthed by the large
EFI mappings
- Prevent a boot hang on large s
Linus,
please pull the latest locking-urgent-for-linus git tree from:
git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git
locking-urgent-for-linus
Two smallish fixes:
- Use the proper asm constraint in the Super-H atomic_fetch_ops
- A trivial typo fix in the Kconfig help text
Linus,
please pull the latest perf-urgent-for-linus git tree from:
git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git
perf-urgent-for-linus
Three fixlets for perf:
- Add a missing NULL pointer check in the intel BTS driver.
- Make BTS an exclusive PMU because BTS can only handle
Hi,
On Sat, Sep 24, 2016 at 10:58:15AM +0200, Arnd Bergmann wrote:
> On Saturday, September 24, 2016 1:11:56 PM CEST Baoyou Xie wrote:
> > ---
> > drivers/hsi/clients/ssi_protocol.c | 2 --
> > include/linux/hsi/hsi.h| 1 +
> > 2 files changed, 1 insertion(+), 2 deletions(-)
> >
> >
Convert g_NCR5380 to use scsi_add_host instead of scsi_module.c
Use pnp_driver and isa_driver to manage cards.
Signed-off-by: Ondrej Zary
---
drivers/scsi/g_NCR5380.c | 310 +-
drivers/scsi/g_NCR5380.h |8 --
2 files changed, 169 insertions(+), 14
This small patch series removes deprecated code from g_NCR5380 driver
and converts it from scsi_module.c to scsi_add_host().
Tested with:
HP C2502 (53C400A chip)
Canon FG2-5202 (53C400 chip, memory-mapped)
DTC-3181L (DTCT-436P chip, PnP)
P.S. I don't like the ifdef mess needed for g_NCR5380_mmio
Remove deprecated __setup for parsing command line parameters.
g_NCR5380.* parameters could be used instead.
Signed-off-by: Ondrej Zary
---
drivers/scsi/g_NCR5380.c | 135 --
1 file changed, 135 deletions(-)
diff --git a/drivers/scsi/g_NCR5380.c b/dr
Remove compile-time card type definition GENERIC_NCR5380_OVERRIDE.
Then remove all code iterating the overrides[] array and reduce it to
struct card.
Signed-off-by: Ondrej Zary
---
drivers/scsi/g_NCR5380.c | 351 ++
1 file changed, 167 insertions(+),
>> Should the desired output be the same when the relevant data are passed by a
>> single function call
>> (instead of three as before)?
>
> Adding a singleton for a pr_fmt #define constant string and
> updating the printk subsystem to prepend that constant string
> to each use of a pr_ at runtim
Larry Finger writes:
> On 09/24/2016 12:32 PM, Joe Perches wrote:
>> Is there any value in that or is Jes' work going to make
>> doing any or all of this unnecessary and futile?
>
> That is not yet determined. The only driver that is to be replaced at
> this point is rtl8192cu. Jes only has USB I/
Hello,
I would like to discuss this commit:
commit d3d7e6c65f75de18ced10a98595a847f9f95f0ce
Author: Raghu Vatsavayi
Date: Tue Jun 21 22:53:07 2016 -0700
liquidio: Firmware image download
This patch has firmware image related changes for: firmware
release upon failure, support lat
Some of the registers defined in the driver are only usable on the
A20. Rename these registers.
Signed-off-by: Danny Milosavljevic
---
sound/soc/sunxi/sun4i-codec.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/sound/soc/sunxi/sun4i-codec.c b/sound/soc/sunxi/sun4i-cod
The A20 has a few extra registers that the A10 doesn't have.
Therefore, use different regmaps for A10 as compared to A20.
Signed-off-by: Danny Milosavljevic
---
sound/soc/sunxi/sun4i-codec.c | 38 +++---
1 file changed, 35 insertions(+), 3 deletions(-)
diff --git
Introduce mechanism to detect sun7i and provide a different regmap
(different compared to sun4i Allwinner A10).
The controls will be extended in a forthcoming patch - it is necessary
to distinguish between sun4i and sun7i controls because they have different
registers.
Rename SUN4I_CODEC_AC_SYS_V
Rename "sun4i_codec_widgets" to "sun4i_codec_controls" for
consistency with the struct field name.
Signed-off-by: Danny Milosavljevic
---
sound/soc/sunxi/sun4i-codec.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/sound/soc/sunxi/sun4i-codec.c b/sound/soc/sunxi/sun4i-
'brick' has been allocated by 'devm_kzalloc', so there is no need here to
free it explicitly.
There is only one caller of 'tegra_clk_sor_brick_register()'. This function
is a probe function which correctly checks and handles the return value of
'tegra_clk_sor_brick_register()'.
Signed-off-by: Chri
Hello,
did you try without your patches, to see if you can reproduce the problem?
I can fix my issues if I disable usb autosuspend
To narrow down the causes I can think of:
*) Some hardware issue only on my side -> please try to reproduce it
with an unpached kernel
*) Some other stuff I modified
On Fri, Sep 23, 2016 at 1:16 PM, Matthew Wilcox wrote:
>
> #ifdef CONFIG_RADIX_TREE_MULTIORDER
> if (radix_tree_is_internal_node(entry)) {
> - unsigned long siboff = get_slot_offset(parent, entry);
> + unsigned long siboff = get_slot_offset(parent,
> +
On Sat, 2016-09-24 at 14:06 -0500, Larry Finger wrote:
> On 09/24/2016 12:32 PM, Joe Perches wrote:
[]
> o Reindent all the switch/case blocks to a more normal
> kernel style (git diff -w would show no changes here)
> That sounds like busy work to me, but if you want to do it, go ahead.
It's rea
>
>
> >
> > Signed-off-by: Tomas Winkler
> > Signed-off-by: Alexander Usyskin
> Tested-by: Avri Altman
>
> - mmc - full functionality. One issue found that was fixed on V6: patch V6
> 2/9.
> - ufs - read & read counter only. Testing is still wip.
>
>
> > +static int rpmb_request_verify(s
Joe Perches writes:
> On Sat, 2016-09-24 at 14:06 -0500, Larry Finger wrote:
>> On 09/24/2016 12:32 PM, Joe Perches wrote:
> []
>> o Reindent all the switch/case blocks to a more normal
>> kernel style (git diff -w would show no changes here)
>> That sounds like busy work to me, but if you want
From: Rafał Miłecki
There are two protocols used by Broadcom FullMAC devices: BCDC and
msgbuf. They use different ways for (some part of) communication with
the firmware. Firmware Signaling is required for the first one only
(BCDC).
So far we were always initializing fws and always calling it's
From: Rafał Miłecki
It's not needed by the other (msgbuf) protocol, so let's save some size
and compile it conditionally.
Signed-off-by: Rafał Miłecki
---
.../wireless/broadcom/brcm80211/brcmfmac/Makefile | 4 +-
.../broadcom/brcm80211/brcmfmac/fwsignal.h | 59 ++
On Sat, Sep 24, 2016 at 1:21 PM, Linus Torvalds
wrote:
>
> That said, if this code isn't even used, as Konstantin says (THP
> selects it - doesn't THP use it?), then the fix really should be to
> just remove the odd code instead of adding to it.
>
> Looking around for uses that set "order" to anyt
On Saturday, September 24, 2016 4:14:15 PM CEST zhichang wrote:
>
> In V3, the outb is :
>
> void outb(u8 value, unsigned long addr)
> {
> if (!arm64_extio_ops || arm64_extio_ops->start > addr ||
> arm64_extio_ops->end < addr)
> writeb(value, PCI_IO
On Sat, Sep 24, 2016 at 01:21:36PM -0700, Linus Torvalds wrote:
> On Fri, Sep 23, 2016 at 1:16 PM, Matthew Wilcox
> wrote:
> >
> > #ifdef CONFIG_RADIX_TREE_MULTIORDER
> > if (radix_tree_is_internal_node(entry)) {
> > - unsigned long siboff = get_slot_offset(parent, entry);
On Fri, Sep 23, 2016 at 02:13:52PM +0200, Greg KH wrote:
> On Thu, Sep 22, 2016 at 09:15:45PM +0300, Moshe Green wrote:
> > Rename CamelCased function getChipType to get_chip_type.
> > This issue was found by checkpatch.pl
>
> As this is a global function, can you rename it to something like
> "sm
Krzysztof Kozlowski writes:
> On Thu, Sep 22, 2016 at 08:59:03PM +0200, Wolfgang Wiedmeyer wrote:
>>
>> Krzysztof Kozlowski writes:
>>
>> > On Thu, Sep 22, 2016 at 06:48:35PM +0200, Wolfgang Wiedmeyer wrote:
>> >> This allows to reboot the device into recovery mode and into the download
>> >> m
There is no check if ioremap_nocache() returns a valid pointer.
Potentially it can lead to null pointer dereference.
Found by Linux Driver Verification project (linuxtesting.org).
Signed-off-by: Alexey Khoroshilov
---
drivers/firewire/nosy.c | 7 +++
1 file changed, 7 insertions(+)
diff --
If 'clk_hw_register()' fails, it is likely that we expect to return an
error instead of a valid pointer (which would mean success).
'clk_hw_register()' returns a bool, so we don't have the detail of the
error. Return -ENOMEM instead, as it seems to be the most common error
returned by clk_register(
The definition of the flush hint table as:
void __iomem *flush_wpq[0][0];
...passed the unit test, but is broken as flush_wpq[0][1] and
flush_wpq[1][0] refer to the same entry. Fix this to use a helper that
calculates a slot in the table based on the geometry of flush hints in
the region
On Sat, Sep 24, 2016 at 2:04 PM, Kirill A. Shutemov
wrote:
>
> Well, my ext4-with-huge-pages patchset[1] uses multi-order entries.
> It also converts shmem-with-huge-pages and hugetlb to them.
Ok, so that code actually has a chance of being used. I guess we'll
not remove it. But I *would* like th
Send an immediate ACK if we fill in a hole in the buffer left by an
out-of-sequence packet. This may allow the congestion management in the peer
to avoid a retransmission if packets got reordered on the wire.
Signed-off-by: David Howells
---
net/rxrpc/input.c | 10 +-
1 file changed,
ux/kernel/git/dhowells/linux-fs.git/log/?h=rxrpc-rewrite
Tagged thusly:
git://git.kernel.org/pub/scm/linux/kernel/git/dhowells/linux-fs.git
rxrpc-rewrite-20160924
David
---
David Howells (8):
rxrpc: Send an ACK after every few DATA packets we receive
rxrpc: Send an immedi
Send an ACK if we haven't sent one for the last two packets we've received.
This keeps the other end apprised of where we've got to - which is
important if they're doing slow-start.
We do this in recvmsg so that we can dispatch a packet directly without the
need to wake up the background thread.
If we've sent all the request data in a client call but haven't seen any
sign of the reply data yet, schedule an ACK to be sent to the server to
find out if the reply data got lost.
If the server hasn't yet hard-ACK'd the request data, we send a PING ACK to
demand a response to find out whether we
When determining the resend timer value, we have a value in nsec but the
timer is in jiffies which may be a million or more times more coarse.
nsecs_to_jiffies() rounds down - which means that the resend timeout
expressed as jiffies is very likely earlier than the one expressed as
nanoseconds from
Generate a summary of the Tx buffer packet state when an ACK is received
for use in a later patch that does congestion management.
Signed-off-by: David Howells
---
net/rxrpc/ar-internal.h | 14 ++
net/rxrpc/input.c | 45 ++---
2 file
Implement RxRPC slow-start, which is similar to RFC 5681 for TCP. A
tracepoint is added to log the state of the congestion management algorithm
and the decisions it makes.
Notes:
(1) Since we send fixed-size DATA packets (apart from the final packet in
each phase), counters and calculation
In a client call, include the serial number of the last DATA packet of the
reply in the final ACK.
Signed-off-by: David Howells
---
net/rxrpc/recvmsg.c |6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/net/rxrpc/recvmsg.c b/net/rxrpc/recvmsg.c
index a7458c398b9e..038a
Clear the ACK reason, ACK timer and resend timer when entering the client
reply phase when the first DATA packet is received. New ACKs will be
proposed once the data is queued.
The resend timer is no longer relevant and we need to cancel ACKs scheduled
to probe for a lost reply.
Signed-off-by: D
On 22 September 2016 at 20:53, Matthew Wilcox
wrote:
> From: Matthew Wilcox
>
> When compiling the radix tree with -O2, GCC thinks it can optimise:
>
> void *entry = parent->slots[offset];
> int siboff = entry - parent->slots;
If entry is a pointer to void, how can you do pointer
From: Chris Roth
Date: Sat, 24 Sep 2016 10:59:04 -0600
> Due to my lack of familiarity with the how git send-email works, I've
> unintentionally had my name listed as the first 'from' whereas I
> intended Allan Chou to be listed as the first 'from' in the patch. If
> anyone can correct this on my
On 09/23/2016 07:56 PM, zhong jiang wrote:
> On 2016/9/24 1:19, Mike Kravetz wrote:
>> On 09/22/2016 06:53 PM, zhong jiang wrote:
>>> At present, we need to call hugetlb_fix_reserve_count when
>>> hugetlb_unrserve_pages fails,
>>> and PagePrivate will decide hugetlb reserves counts.
>>>
>>> we obt
On Sat, Sep 24, 2016 at 4:35 PM, Cedric Blancher
wrote:
>>
>> void *entry = parent->slots[offset];
>> int siboff = entry - parent->slots;
>
> If entry is a pointer to void, how can you do pointer arithmetic with it?
It's actually void **.
(That said, gcc has an extension that con
From: Colin Ian King
There is an earlier check and return if err is non-zero, so
the check to see if it is zero is redundant in every iteration
of the loop and hence the check can be removed.
Signed-off-by: Colin Ian King
---
drivers/net/ethernet/chelsio/cxgb4/cxgb4_filter.c | 10 +-
1
101 - 200 of 247 matches
Mail list logo