[PATCH v7 2/4] ata: Enabling ATA Command Priorities

From: Adam Manzanares This patch checks to see if an ATA device supports NCQ command priorities. If so and the user has specified an iocontext that indicates IO_PRIO_CLASS_RT then we build a tf with a high priority command. This is done to improve the tail latency of

[PATCH v7 2/4] ata: Enabling ATA Command Priorities

From: Adam Manzanares This patch checks to see if an ATA device supports NCQ command priorities. If so and the user has specified an iocontext that indicates IO_PRIO_CLASS_RT then we build a tf with a high priority command. This is done to improve the tail latency of commands that are high

Re: [PATCH v2 02/11] locking/ww_mutex: Re-check ww->ctx in the inner optimistic spin loop

On 12/06/2016 01:29 PM, Peter Zijlstra wrote: > On Tue, Dec 06, 2016 at 11:03:28AM -0500, Waiman Long wrote: >> The mutex_spin_on_owner() function was originally marked noinline >> because it could be a major consumer of CPU cycles in a contended lock. >> Having it shown separately in the perf

Re: [PATCH v2 02/11] locking/ww_mutex: Re-check ww->ctx in the inner optimistic spin loop

On 12/06/2016 01:29 PM, Peter Zijlstra wrote: > On Tue, Dec 06, 2016 at 11:03:28AM -0500, Waiman Long wrote: >> The mutex_spin_on_owner() function was originally marked noinline >> because it could be a major consumer of CPU cycles in a contended lock. >> Having it shown separately in the perf

Re: [PATCH v2 3/3] tpm_crb: request and relinquish locality 0

On Mon, Dec 05, 2016 at 09:32:35AM -0700, Jason Gunthorpe wrote: > On Sat, Dec 03, 2016 at 07:52:13PM +0200, Jarkko Sakkinen wrote: > > + iowrite32(CRB_LOC_CTRL_RELINQUISH, >regs_h->loc_ctrl); > > Since regs_h can be null shouldn't there be some guards ? Yes, there should. Thanks for pointing

Re: [PATCH v2 3/3] tpm_crb: request and relinquish locality 0

On Mon, Dec 05, 2016 at 09:32:35AM -0700, Jason Gunthorpe wrote: > On Sat, Dec 03, 2016 at 07:52:13PM +0200, Jarkko Sakkinen wrote: > > + iowrite32(CRB_LOC_CTRL_RELINQUISH, >regs_h->loc_ctrl); > > Since regs_h can be null shouldn't there be some guards ? Yes, there should. Thanks for pointing

Re: [PATCH v6 3/3] sound/usb: Use Media Controller API to share media resources

Hi Takashi, On 12/05/2016 11:50 PM, Takashi Iwai wrote: > On Wed, 30 Nov 2016 23:01:16 +0100, > Shuah Khan wrote: >> >> --- a/sound/usb/card.c >> +++ b/sound/usb/card.c > (snip) >> @@ -616,6 +617,11 @@ static int usb_audio_probe(struct usb_interface *intf, >> if (err < 0) >>

Re: [PATCH v6 3/3] sound/usb: Use Media Controller API to share media resources

Hi Takashi, On 12/05/2016 11:50 PM, Takashi Iwai wrote: > On Wed, 30 Nov 2016 23:01:16 +0100, > Shuah Khan wrote: >> >> --- a/sound/usb/card.c >> +++ b/sound/usb/card.c > (snip) >> @@ -616,6 +617,11 @@ static int usb_audio_probe(struct usb_interface *intf, >> if (err < 0) >>

Re: [PATCH] x86/irq: Add additional unhandled IRQ debug information

Hi Prarit, [auto build test ERROR on tip/x86/core] [also build test ERROR on v4.9-rc8 next-20161206] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Prarit-Bhargava/x86-irq-Add-additional

Re: [tpmdd-devel] [PATCH v2 3/3] tpm_crb: request and relinquish locality 0

On Mon, Dec 05, 2016 at 09:51:42PM +, Winkler, Tomas wrote: > > > > CC linux-security-module > > > > On Sat, Dec 03, 2016 at 07:52:13PM +0200, Jarkko Sakkinen wrote: > > > Request and relinquish locality for the driver use in order to be a > > > better citizen in a multi locality environment

Re: [PATCH] x86/irq: Add additional unhandled IRQ debug information

Hi Prarit, [auto build test ERROR on tip/x86/core] [also build test ERROR on v4.9-rc8 next-20161206] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Prarit-Bhargava/x86-irq-Add-additional

Re: [tpmdd-devel] [PATCH v2 3/3] tpm_crb: request and relinquish locality 0

On Mon, Dec 05, 2016 at 09:51:42PM +, Winkler, Tomas wrote: > > > > CC linux-security-module > > > > On Sat, Dec 03, 2016 at 07:52:13PM +0200, Jarkko Sakkinen wrote: > > > Request and relinquish locality for the driver use in order to be a > > > better citizen in a multi locality environment

Re: [tpmdd-devel] [PATCH v2 2/3] tpm_crb: encapsulate crb_wait_for_reg_32

On Mon, Dec 05, 2016 at 09:34:50PM +, Winkler, Tomas wrote: > > CC linux-security-module > > > > On Sat, Dec 03, 2016 at 07:52:12PM +0200, Jarkko Sakkinen wrote: > > > Encapsulated crb_wait_for_reg32() so that state changes in other CRB > > > registers than TPM_CRB_CTRL_REQ_x can be waited. >

Re: [tpmdd-devel] [PATCH v2 2/3] tpm_crb: encapsulate crb_wait_for_reg_32

On Mon, Dec 05, 2016 at 09:34:50PM +, Winkler, Tomas wrote: > > CC linux-security-module > > > > On Sat, Dec 03, 2016 at 07:52:12PM +0200, Jarkko Sakkinen wrote: > > > Encapsulated crb_wait_for_reg32() so that state changes in other CRB > > > registers than TPM_CRB_CTRL_REQ_x can be waited. >

Re: [PATCH v7 0/4] Enabling ATA Command Priorities

Adam, On Tue, Dec 06, 2016 at 09:18:01AM -0800, Adam Manzanares wrote: > From: Adam Manzanares > > This patch builds ATA commands with high priority if the iocontext of a > process > is set to real time. The goal of the patch is to improve tail latencies of >

Re: [PATCH v7 0/4] Enabling ATA Command Priorities

Adam, On Tue, Dec 06, 2016 at 09:18:01AM -0800, Adam Manzanares wrote: > From: Adam Manzanares > > This patch builds ATA commands with high priority if the iocontext of a > process > is set to real time. The goal of the patch is to improve tail latencies of > workloads that use higher queue

Re: [tpmdd-devel] [PATCH v2 1/3] tpm_crb: map locality registers

On Mon, Dec 05, 2016 at 09:31:57PM +, Winkler, Tomas wrote: > > > > On Mon, Dec 05, 2016 at 12:07:51PM +, Winkler, Tomas wrote: > > > > > --- > > > > > drivers/char/tpm/tpm_crb.c | 96 > > > > > ++ > > > > > 1 file changed, 64 insertions(+), 32

Re: [tpmdd-devel] [PATCH v2 1/3] tpm_crb: map locality registers

On Mon, Dec 05, 2016 at 09:31:57PM +, Winkler, Tomas wrote: > > > > On Mon, Dec 05, 2016 at 12:07:51PM +, Winkler, Tomas wrote: > > > > > --- > > > > > drivers/char/tpm/tpm_crb.c | 96 > > > > > ++ > > > > > 1 file changed, 64 insertions(+), 32

[PATCH v2 2/2] vfio iommu: Fix size argument to vfio_find_dma() in pin_pages/unpin_pages

Passing zero for the size to vfio_find_dma() isn't compatible with matching the start address of an existing vfio_dma. Doing so triggers a corner case. In vfio_find_dma(), when the start address is equal to dma->iova and size is 0, check for the end of search range makes it to take wrong side of

Re: [PATCH V2 03/13] perf/x86: output sampling overhead

On Tue, Dec 06, 2016 at 03:47:40PM +, Liang, Kan wrote: > > It doesn't record anything, it generates the output. And it doesn't explain > > why that needs to be in pmu::del(), in general that's a horrible thing to > > do. > > Yes, it only generate/log the output. Sorry for the confused

[PATCH v2 2/2] vfio iommu: Fix size argument to vfio_find_dma() in pin_pages/unpin_pages

Passing zero for the size to vfio_find_dma() isn't compatible with matching the start address of an existing vfio_dma. Doing so triggers a corner case. In vfio_find_dma(), when the start address is equal to dma->iova and size is 0, check for the end of search range makes it to take wrong side of

Re: [PATCH V2 03/13] perf/x86: output sampling overhead

On Tue, Dec 06, 2016 at 03:47:40PM +, Liang, Kan wrote: > > It doesn't record anything, it generates the output. And it doesn't explain > > why that needs to be in pmu::del(), in general that's a horrible thing to > > do. > > Yes, it only generate/log the output. Sorry for the confused

[PATCH] staging/android: remove Sync Framework tasks from TODO

From: Gustavo Padovan The destaging work is now fully complete. Cc: Arve Hjønnevåg Cc: Riley Andrews Signed-off-by: Gustavo Padovan --- drivers/staging/android/TODO | 8 1

[PATCH] staging/android: remove Sync Framework tasks from TODO

From: Gustavo Padovan The destaging work is now fully complete. Cc: Arve Hjønnevåg Cc: Riley Andrews Signed-off-by: Gustavo Padovan --- drivers/staging/android/TODO | 8 1 file changed, 8 deletions(-) diff --git a/drivers/staging/android/TODO b/drivers/staging/android/TODO index

[PATCH v2 1/2] vfio iommu type1: Fix size argument to vfio_find_dma() during DMA UNMAP.

Passing zero for the size to vfio_find_dma() isn't compatible with matching the start address of an existing vfio_dma. Doing so triggers a corner case. In vfio_find_dma(), when the start address is equal to dma->iova and size is 0, check for the end of search range makes it to take wrong side of

[PATCH v2 1/2] vfio iommu type1: Fix size argument to vfio_find_dma() during DMA UNMAP.

Passing zero for the size to vfio_find_dma() isn't compatible with matching the start address of an existing vfio_dma. Doing so triggers a corner case. In vfio_find_dma(), when the start address is equal to dma->iova and size is 0, check for the end of search range makes it to take wrong side of

Re: [RFC v3 06/10] iommu: iommu_get_group_resv_regions

On 15/11/16 13:09, Eric Auger wrote: > Introduce iommu_get_group_resv_regions whose role consists in > enumerating all devices from the group and collecting their > reserved regions. It checks duplicates. > > Signed-off-by: Eric Auger > > --- > > - we do not move list

Re: [RFC v3 06/10] iommu: iommu_get_group_resv_regions

On 15/11/16 13:09, Eric Auger wrote: > Introduce iommu_get_group_resv_regions whose role consists in > enumerating all devices from the group and collecting their > reserved regions. It checks duplicates. > > Signed-off-by: Eric Auger > > --- > > - we do not move list elements from device to

Re: [RESEND][PATCH v4] cgroup: Use CAP_SYS_RESOURCE to allow a process to migrate other tasks between cgroups

Hello, On Tue, Dec 06, 2016 at 10:13:53AM -0800, Andy Lutomirski wrote: > > Delegation is an explicit operation and reflected in the ownership of > > the subdirectories and cgroup interface files in them. The > > subhierarchy containment is achieved by requiring the user who's > > trying to

Re: [RESEND][PATCH v4] cgroup: Use CAP_SYS_RESOURCE to allow a process to migrate other tasks between cgroups

Hello, On Tue, Dec 06, 2016 at 10:13:53AM -0800, Andy Lutomirski wrote: > > Delegation is an explicit operation and reflected in the ownership of > > the subdirectories and cgroup interface files in them. The > > subhierarchy containment is achieved by requiring the user who's > > trying to

Re: [PATCH v2 1/2] vfio iommu type1: Fix size argument to vfio_find_dma() during DMA UNMAP.

On 12/6/2016 11:08 PM, Alex Williamson wrote: > On Tue, 6 Dec 2016 22:43:30 +0530 > Kirti Wankhede wrote: > >> vfio_dma keeps track of address range from (dma->iova + 0) to >> (dma->iova + dma->size - 1), while vfio_find_dma() search logic looks for >> range from

Re: [PATCH v2 1/2] vfio iommu type1: Fix size argument to vfio_find_dma() during DMA UNMAP.

On 12/6/2016 11:08 PM, Alex Williamson wrote: > On Tue, 6 Dec 2016 22:43:30 +0530 > Kirti Wankhede wrote: > >> vfio_dma keeps track of address range from (dma->iova + 0) to >> (dma->iova + dma->size - 1), while vfio_find_dma() search logic looks for >> range from (dma->iova + 1) to (dma->iova

Re: [PATCHv4 09/10] mm/usercopy: Switch to using lm_alias

On Tue, Nov 29, 2016 at 10:55:28AM -0800, Laura Abbott wrote: > > The usercopy checking code currently calls __va(__pa(...)) to check for > aliases on symbols. Switch to using lm_alias instead. > > Signed-off-by: Laura Abbott I've given this a go on Juno, which boots

Re: [PATCHv4 09/10] mm/usercopy: Switch to using lm_alias

On Tue, Nov 29, 2016 at 10:55:28AM -0800, Laura Abbott wrote: > > The usercopy checking code currently calls __va(__pa(...)) to check for > aliases on symbols. Switch to using lm_alias instead. > > Signed-off-by: Laura Abbott I've given this a go on Juno, which boots happily. LKDTM triggers as

Re: [RFC v2 0/4] ACPI: SPCR: 32-bit access and non-standard baud rate

Hi Aleksey, On 12/06/2016 12:58 PM, Aleksey Makarov wrote: > It turns out that this approach does not work for all the existing hardware. > There are two problems with AppliedMicro X-Gene based boards > ([discussion], [v1]): > > 1. Their console is a 16550 port that requires 32-bit access. Now

Re: [RFC v2 0/4] ACPI: SPCR: 32-bit access and non-standard baud rate

Hi Aleksey, On 12/06/2016 12:58 PM, Aleksey Makarov wrote: > It turns out that this approach does not work for all the existing hardware. > There are two problems with AppliedMicro X-Gene based boards > ([discussion], [v1]): > > 1. Their console is a 16550 port that requires 32-bit access. Now

Re: [PATCHv4 09/10] mm/usercopy: Switch to using lm_alias

On Tue, Nov 29, 2016 at 11:39:44AM -0800, Kees Cook wrote: > On Tue, Nov 29, 2016 at 10:55 AM, Laura Abbott wrote: > > > > The usercopy checking code currently calls __va(__pa(...)) to check for > > aliases on symbols. Switch to using lm_alias instead. > > > > Signed-off-by:

Re: [PATCHv4 09/10] mm/usercopy: Switch to using lm_alias

On Tue, Nov 29, 2016 at 11:39:44AM -0800, Kees Cook wrote: > On Tue, Nov 29, 2016 at 10:55 AM, Laura Abbott wrote: > > > > The usercopy checking code currently calls __va(__pa(...)) to check for > > aliases on symbols. Switch to using lm_alias instead. > > > > Signed-off-by: Laura Abbott > >

[PATCH v3 1/7] irda: w83977af_ir: Whitespace neatening

Remove leading and trailing whitespace. git diff -w shows no differences. Signed-off-by: Joe Perches --- drivers/net/irda/w83977af_ir.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/irda/w83977af_ir.c b/drivers/net/irda/w83977af_ir.c index

[PATCH v3 1/7] irda: w83977af_ir: Whitespace neatening

Remove leading and trailing whitespace. git diff -w shows no differences. Signed-off-by: Joe Perches --- drivers/net/irda/w83977af_ir.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/irda/w83977af_ir.c b/drivers/net/irda/w83977af_ir.c index

[PATCH v3 4/7] irda: w83977af_ir: Neaten pointer comparisons

Convert pointer comparisons to NULL. Signed-off-by: Joe Perches --- drivers/net/irda/w83977af_ir.c | 28 ++-- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/drivers/net/irda/w83977af_ir.c b/drivers/net/irda/w83977af_ir.c index

[PATCH v3 5/7] irda: w83977af_ir: Use the common brace style

Add braces where appropriate and remove an unnecessary else. Signed-off-by: Joe Perches --- drivers/net/irda/w83977af_ir.c | 20 +++- 1 file changed, 11 insertions(+), 9 deletions(-) diff --git a/drivers/net/irda/w83977af_ir.c b/drivers/net/irda/w83977af_ir.c

[PATCH v3 6/7] irda: w83977af_ir: Parenthesis alignment

Neaten function declaration and definition arguments. Signed-off-by: Joe Perches --- drivers/net/irda/w83977af_ir.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/net/irda/w83977af_ir.c b/drivers/net/irda/w83977af_ir.c index

[PATCH v3 3/7] irda: w83977af_ir: Remove and add blank lines

Use a more typical vertical spacing style. Signed-off-by: Joe Perches --- drivers/net/irda/w83977af_ir.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/net/irda/w83977af_ir.c b/drivers/net/irda/w83977af_ir.c index 166dd4e524fc..e30cbf005320

[PATCH v3 4/7] irda: w83977af_ir: Neaten pointer comparisons

Convert pointer comparisons to NULL. Signed-off-by: Joe Perches --- drivers/net/irda/w83977af_ir.c | 28 ++-- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/drivers/net/irda/w83977af_ir.c b/drivers/net/irda/w83977af_ir.c index e30cbf005320..92f1525b0570

[PATCH v3 5/7] irda: w83977af_ir: Use the common brace style

Add braces where appropriate and remove an unnecessary else. Signed-off-by: Joe Perches --- drivers/net/irda/w83977af_ir.c | 20 +++- 1 file changed, 11 insertions(+), 9 deletions(-) diff --git a/drivers/net/irda/w83977af_ir.c b/drivers/net/irda/w83977af_ir.c index

[PATCH v3 6/7] irda: w83977af_ir: Parenthesis alignment

Neaten function declaration and definition arguments. Signed-off-by: Joe Perches --- drivers/net/irda/w83977af_ir.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/net/irda/w83977af_ir.c b/drivers/net/irda/w83977af_ir.c index 5d5ec8938e2b..2f95ee26ed49 100644

[PATCH v3 3/7] irda: w83977af_ir: Remove and add blank lines

Use a more typical vertical spacing style. Signed-off-by: Joe Perches --- drivers/net/irda/w83977af_ir.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/net/irda/w83977af_ir.c b/drivers/net/irda/w83977af_ir.c index 166dd4e524fc..e30cbf005320 100644 ---

[PATCH v2] iommu/intel-iommu: fix pasid table size encoding

Different encodings are used to represent supported PASID bits and number of PASID table entries. The current code assigns ecap_pss directly to extended context table entry PTS which is wrong and could result in writing non-zero bits to the reserved fields. IOMMU fault reason 11 will be reported

[PATCH v3 7/7] irda: w83977af_ir: Neaten logging

Use more common logging style, standardize function output logging use. Miscellanea: o Add and use pr_fmt o Convert printks to pr_ Signed-off-by: Joe Perches --- drivers/net/irda/w83977af_ir.c | 56 -- 1 file changed, 27 insertions(+),

[PATCH v3 0/7] irda: w83977af_ir: Neatening

Originally on top of Arnd's overly long udelay patches because I noticed a misindented block. That's now already fixed along with some other whitespace problems. These patches are the remainder style issues from my original series. Even though I haven't turned on the netwinder in a box in the

[PATCH v2] iommu/intel-iommu: fix pasid table size encoding

Different encodings are used to represent supported PASID bits and number of PASID table entries. The current code assigns ecap_pss directly to extended context table entry PTS which is wrong and could result in writing non-zero bits to the reserved fields. IOMMU fault reason 11 will be reported

[PATCH v3 7/7] irda: w83977af_ir: Neaten logging

Use more common logging style, standardize function output logging use. Miscellanea: o Add and use pr_fmt o Convert printks to pr_ Signed-off-by: Joe Perches --- drivers/net/irda/w83977af_ir.c | 56 -- 1 file changed, 27 insertions(+), 29 deletions(-)

[PATCH v3 0/7] irda: w83977af_ir: Neatening

Originally on top of Arnd's overly long udelay patches because I noticed a misindented block. That's now already fixed along with some other whitespace problems. These patches are the remainder style issues from my original series. Even though I haven't turned on the netwinder in a box in the

[PATCH v3 2/7] irda: w83977af_ir: More whitespace neatening

Add spaces around operators. git diff -w shows no differences. Signed-off-by: Joe Perches --- drivers/net/irda/w83977af_ir.c | 230 - 1 file changed, 115 insertions(+), 115 deletions(-) diff --git a/drivers/net/irda/w83977af_ir.c

[PATCH v3 2/7] irda: w83977af_ir: More whitespace neatening

Add spaces around operators. git diff -w shows no differences. Signed-off-by: Joe Perches --- drivers/net/irda/w83977af_ir.c | 230 - 1 file changed, 115 insertions(+), 115 deletions(-) diff --git a/drivers/net/irda/w83977af_ir.c

Re: [RESEND][PATCH v4] cgroup: Use CAP_SYS_RESOURCE to allow a process to migrate other tasks between cgroups

On Tue, Dec 6, 2016 at 10:12 AM, Tejun Heo wrote: > Hello, > > On Tue, Dec 06, 2016 at 09:01:17AM -0800, Andy Lutomirski wrote: >> How would one be granted the right to move processes around in one's >> own subtree? > > Through expicit delegation - chowning of the directory and >

Re: [RESEND][PATCH v4] cgroup: Use CAP_SYS_RESOURCE to allow a process to migrate other tasks between cgroups

On Tue, Dec 6, 2016 at 10:12 AM, Tejun Heo wrote: > Hello, > > On Tue, Dec 06, 2016 at 09:01:17AM -0800, Andy Lutomirski wrote: >> How would one be granted the right to move processes around in one's >> own subtree? > > Through expicit delegation - chowning of the directory and > cgroup.procs

Re: [RESEND][PATCH v4] cgroup: Use CAP_SYS_RESOURCE to allow a process to migrate other tasks between cgroups

Hello, On Tue, Dec 06, 2016 at 09:01:17AM -0800, Andy Lutomirski wrote: > How would one be granted the right to move processes around in one's > own subtree? Through expicit delegation - chowning of the directory and cgroup.procs file. > Are you imagining that, if you're in /a/b and you want to

Re: [RESEND][PATCH v4] cgroup: Use CAP_SYS_RESOURCE to allow a process to migrate other tasks between cgroups

Hello, On Tue, Dec 06, 2016 at 09:01:17AM -0800, Andy Lutomirski wrote: > How would one be granted the right to move processes around in one's > own subtree? Through expicit delegation - chowning of the directory and cgroup.procs file. > Are you imagining that, if you're in /a/b and you want to

Re: [PATCH 4/6] net: ethernet: ti: cpts: add ptp pps support

On Wed, Nov 30, 2016 at 11:05:19AM +0100, Richard Cochran wrote: > Can you adjust the frequency of the keystone devices in hardware? If > so, then please implement it, and just disable PPS for the CPSW. > > The only reason I used the timecounter for frequency adjustment was > because the am335x

Re: [PATCH 4/6] net: ethernet: ti: cpts: add ptp pps support

On Wed, Nov 30, 2016 at 11:05:19AM +0100, Richard Cochran wrote: > Can you adjust the frequency of the keystone devices in hardware? If > so, then please implement it, and just disable PPS for the CPSW. > > The only reason I used the timecounter for frequency adjustment was > because the am335x

[PATCH] staging: dgnc: fix blank line after '{' warnings.

Remove blank lines between open brace and comment. Remove blank lines after comment in line with the rest of the comments of the file. Signed-off-by: Fernando Apesteguia --- drivers/staging/dgnc/dgnc_tty.c | 6 -- 1 file changed, 6 deletions(-) diff --git

[PATCH] staging: dgnc: fix blank line after '{' warnings.

Remove blank lines between open brace and comment. Remove blank lines after comment in line with the rest of the comments of the file. Signed-off-by: Fernando Apesteguia --- drivers/staging/dgnc/dgnc_tty.c | 6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_tty.c

Re: [RFC PATCH 4/6] x86/fpu: Fix "x86/fpu: Legacy x87 FPU detected" message

On Tue, Dec 06, 2016 at 09:59:56AM -0800, Andy Lutomirski wrote: > I did something like this for FSAVE vs FXSAVE. XSAVE has its own pile > of printouts and I don't feel like I need to add another. So frankly, replacing "legacy" with "pre-XSAVE" is not making it any clearer - it is just calling

Re: [RFC PATCH 4/6] x86/fpu: Fix "x86/fpu: Legacy x87 FPU detected" message

On Tue, Dec 06, 2016 at 09:59:56AM -0800, Andy Lutomirski wrote: > I did something like this for FSAVE vs FXSAVE. XSAVE has its own pile > of printouts and I don't feel like I need to add another. So frankly, replacing "legacy" with "pre-XSAVE" is not making it any clearer - it is just calling

Re: [PATCH v4 09/13] net: ethernet: ti: cpts: rework initialization/deinitialization

On Tue, Dec 06, 2016 at 11:49:14AM -0600, Grygorii Strashko wrote: > But we do reset whole cpsw :( and that's required to support PM use cases as > suspend/resume. The code is resetting the clock unconditionally on ifup/down. That sucks. If you reset the clock *only* after resume, that would be

Re: [PATCH v4 09/13] net: ethernet: ti: cpts: rework initialization/deinitialization

On Tue, Dec 06, 2016 at 11:49:14AM -0600, Grygorii Strashko wrote: > But we do reset whole cpsw :( and that's required to support PM use cases as > suspend/resume. The code is resetting the clock unconditionally on ifup/down. That sucks. If you reset the clock *only* after resume, that would be

Re: [PATCH 7/8] staging: fsl-dpaa2/eth: Add TODO file

On Tue, Dec 06, 2016 at 12:59:59PM +, Stuart Yoder wrote: > > > > -Original Message- > > From: Greg KH [mailto:gre...@linuxfoundation.org] > > Sent: Tuesday, December 06, 2016 4:20 AM > > To: Ruxandra Ioana Radulescu > > Cc: de...@driverdev.osuosl.org;

Re: [PATCH 1/2] arm: kernel: Add SMC structure parameter

On Tue, Dec 06, 2016 at 11:55:08AM +, Will Deacon wrote: > Hi Andy, > > On Tue, Nov 29, 2016 at 01:44:22AM -0600, Andy Gross wrote: > > This patch adds a quirk parameter to the arm_smccc_smc call. The quirk > > structure allows for specialized SMC operations due to SoC specific > >

Re: [PATCH 7/8] staging: fsl-dpaa2/eth: Add TODO file

On Tue, Dec 06, 2016 at 12:59:59PM +, Stuart Yoder wrote: > > > > -Original Message- > > From: Greg KH [mailto:gre...@linuxfoundation.org] > > Sent: Tuesday, December 06, 2016 4:20 AM > > To: Ruxandra Ioana Radulescu > > Cc: de...@driverdev.osuosl.org; linux-kernel@vger.kernel.org;

Re: [PATCH 1/2] arm: kernel: Add SMC structure parameter

On Tue, Dec 06, 2016 at 11:55:08AM +, Will Deacon wrote: > Hi Andy, > > On Tue, Nov 29, 2016 at 01:44:22AM -0600, Andy Gross wrote: > > This patch adds a quirk parameter to the arm_smccc_smc call. The quirk > > structure allows for specialized SMC operations due to SoC specific > >

Re: [RFC PATCH 4/6] x86/fpu: Fix "x86/fpu: Legacy x87 FPU detected" message

On Tue, Dec 6, 2016 at 1:24 AM, Borislav Petkov wrote: > On Mon, Dec 05, 2016 at 05:01:13PM -0800, Andy Lutomirski wrote: >> That message isn't at all clear -- what does "Legacy x87" even >> mean? >> >> Clarify it. If there's no FPU, say "x86/fpu: No FPU detected". If >> there's

[RFC v2 0/4] ACPI: SPCR: 32-bit access and non-standard baud rate

ACPI [SPCR] (Serial Port Console Redirection Table) specifies which console should be used by system. 'ARM Server Base Boot Requirements' [SBBR] mentions SPCR as a mandatory ACPI table. Support for this table for Linux kernel ([SPCR v10]) was merged for 4.9 (patches 1/4-3/4) and 4.10 (patch

[RFC v2 4/4] ACPI: SPCR: support 16550 UART with 32-bit access

It was suggested to add a new Microsoft Debug Port Table 2 (DBG2) (the table used to enumerate the various subtypes of serial port covered by the SPCR) 16550 UART subtype that may be needed for some additional platforms, such as those based upon AppliedMicro X-Gene ARMv8 SoCs. This new subtype

Re: [RFC PATCH 4/6] x86/fpu: Fix "x86/fpu: Legacy x87 FPU detected" message

On Tue, Dec 6, 2016 at 1:24 AM, Borislav Petkov wrote: > On Mon, Dec 05, 2016 at 05:01:13PM -0800, Andy Lutomirski wrote: >> That message isn't at all clear -- what does "Legacy x87" even >> mean? >> >> Clarify it. If there's no FPU, say "x86/fpu: No FPU detected". If >> there's an FPU that

[RFC v2 0/4] ACPI: SPCR: 32-bit access and non-standard baud rate

ACPI [SPCR] (Serial Port Console Redirection Table) specifies which console should be used by system. 'ARM Server Base Boot Requirements' [SBBR] mentions SPCR as a mandatory ACPI table. Support for this table for Linux kernel ([SPCR v10]) was merged for 4.9 (patches 1/4-3/4) and 4.10 (patch

[RFC v2 4/4] ACPI: SPCR: support 16550 UART with 32-bit access

It was suggested to add a new Microsoft Debug Port Table 2 (DBG2) (the table used to enumerate the various subtypes of serial port covered by the SPCR) 16550 UART subtype that may be needed for some additional platforms, such as those based upon AppliedMicro X-Gene ARMv8 SoCs. This new subtype

[RFC v2 1/4] ACPI: SPCR: check bit width for the 16550 UART

Check the 'Register Bit Width' field of the ACPI Generic Address Structure that specifies the address of the UART registers to decide if the driver should use "mmio32" access instead of "mmio". If the driver is other than 16550 the access width is defined by the Interface Type field of the SPCR

[RFC v2 2/4] ACPI: SPCR: don't initialize baud rate

AppliedMicro X-Gene based boards don't use the "standard" 16550 clock rate so supplying a baud rate makes it change to a random baud rate. I suggest to introduce a new value '0' for the "Baud Rate" field of SPCR (now this value is reserved). This patch introduces a check for this value. In this

[RFC v2 3/4] ACPI: DBG2: add 16550 UART with 32-bit access

It was suggested to add a new Microsoft Debug Port Table 2 (DBG2) (the table used to enumerate the various subtypes of serial port covered by the SPCR) 16550 UART subtype that may be needed for some additional platforms, such as those based upon AppliedMicro X-Gene ARMv8 SoCs. This new subtype

[RFC v2 1/4] ACPI: SPCR: check bit width for the 16550 UART

Check the 'Register Bit Width' field of the ACPI Generic Address Structure that specifies the address of the UART registers to decide if the driver should use "mmio32" access instead of "mmio". If the driver is other than 16550 the access width is defined by the Interface Type field of the SPCR

[RFC v2 2/4] ACPI: SPCR: don't initialize baud rate

AppliedMicro X-Gene based boards don't use the "standard" 16550 clock rate so supplying a baud rate makes it change to a random baud rate. I suggest to introduce a new value '0' for the "Baud Rate" field of SPCR (now this value is reserved). This patch introduces a check for this value. In this

[RFC v2 3/4] ACPI: DBG2: add 16550 UART with 32-bit access

It was suggested to add a new Microsoft Debug Port Table 2 (DBG2) (the table used to enumerate the various subtypes of serial port covered by the SPCR) 16550 UART subtype that may be needed for some additional platforms, such as those based upon AppliedMicro X-Gene ARMv8 SoCs. This new subtype

Re: [RFC PATCH 5/6] x86/fpu: Fix CPUID-less FPU detection

On Tue, Dec 6, 2016 at 1:40 AM, Borislav Petkov wrote: > On Mon, Dec 05, 2016 at 05:01:14PM -0800, Andy Lutomirski wrote: >> The old code didn't work at all because it adjusted the current caps >> instead of the forced caps. Anything it did would be undone later >> during cpu

Re: [RFC PATCH 5/6] x86/fpu: Fix CPUID-less FPU detection

On Tue, Dec 6, 2016 at 1:40 AM, Borislav Petkov wrote: > On Mon, Dec 05, 2016 at 05:01:14PM -0800, Andy Lutomirski wrote: >> The old code didn't work at all because it adjusted the current caps >> instead of the forced caps. Anything it did would be undone later >> during cpu identification.

Re: [PATCH] ipmi: bt-bmc: Use a regmap for register access

On 12/06/2016 03:57 AM, Andrew Jeffery wrote: > The registers for the bt-bmc device live under the Aspeed LPC > controller. Devicetree bindings have recently been introduced for the > LPC controller where the "host" portion of the LPC register space is > described as a syscon device. Future

Re: [PATCH] ipmi: bt-bmc: Use a regmap for register access

On 12/06/2016 03:57 AM, Andrew Jeffery wrote: > The registers for the bt-bmc device live under the Aspeed LPC > controller. Devicetree bindings have recently been introduced for the > LPC controller where the "host" portion of the LPC register space is > described as a syscon device. Future

Re: [PATCH v3 4/4] x86/asm: Rewrite sync_core() to use IRET-to-self

On Mon, Dec 5, 2016 at 11:52 PM, Borislav Petkov wrote: > On Mon, Dec 05, 2016 at 01:32:43PM -0800, Andy Lutomirski wrote: >> Aside from being excessively slow, CPUID is problematic: Linux runs >> on a handful of CPUs that don't have CPUID. Use IRET-to-self >> instead.

Re: [PATCH v3 4/4] x86/asm: Rewrite sync_core() to use IRET-to-self

On Mon, Dec 5, 2016 at 11:52 PM, Borislav Petkov wrote: > On Mon, Dec 05, 2016 at 01:32:43PM -0800, Andy Lutomirski wrote: >> Aside from being excessively slow, CPUID is problematic: Linux runs >> on a handful of CPUs that don't have CPUID. Use IRET-to-self >> instead. IRET-to-self works

[PATCH v7 1/4] block: Add iocontext priority to request

From: Adam Manzanares Patch adds an association between iocontext ioprio and the ioprio of a request. This is done to enable request based drivers the ability to act on priority information stored in the request. An example being ATA devices that support command

[PATCH v7 1/4] block: Add iocontext priority to request

From: Adam Manzanares Patch adds an association between iocontext ioprio and the ioprio of a request. This is done to enable request based drivers the ability to act on priority information stored in the request. An example being ATA devices that support command priorities. If the ATA driver

Re: [PATCH v4 09/13] net: ethernet: ti: cpts: rework initialization/deinitialization

Hi Richard, On 12/06/2016 11:18 AM, Richard Cochran wrote: > On Tue, Dec 06, 2016 at 10:45:55AM -0600, Grygorii Strashko wrote: >> On 12/06/2016 07:40 AM, Richard Cochran wrote: >>> [ BTW, resetting the timecounter here makes no sense either. Why >>> reset the clock just because the interface

Re: [PATCH v4 09/13] net: ethernet: ti: cpts: rework initialization/deinitialization

Hi Richard, On 12/06/2016 11:18 AM, Richard Cochran wrote: > On Tue, Dec 06, 2016 at 10:45:55AM -0600, Grygorii Strashko wrote: >> On 12/06/2016 07:40 AM, Richard Cochran wrote: >>> [ BTW, resetting the timecounter here makes no sense either. Why >>> reset the clock just because the interface

Re: [Xen-devel] [PATCH v3 4/4] x86/asm: Rewrite sync_core() to use IRET-to-self

On Tue, Dec 6, 2016 at 1:49 AM, Jan Beulich wrote: On 06.12.16 at 10:25, wrote: >> On Tue, Dec 06, 2016 at 01:46:37AM -0700, Jan Beulich wrote: >>> > + asm volatile ( >>> > + "pushfl\n\t" >>> > + "pushl %%cs\n\t" >>> > +

Re: [Xen-devel] [PATCH v3 4/4] x86/asm: Rewrite sync_core() to use IRET-to-self

On Tue, Dec 6, 2016 at 1:49 AM, Jan Beulich wrote: On 06.12.16 at 10:25, wrote: >> On Tue, Dec 06, 2016 at 01:46:37AM -0700, Jan Beulich wrote: >>> > + asm volatile ( >>> > + "pushfl\n\t" >>> > + "pushl %%cs\n\t" >>> > + "pushl $1f\n\t" >>> > +

Re: [PATCHv4 08/10] mm/kasan: Switch to using __pa_symbol and lm_alias

On Tue, Nov 29, 2016 at 10:55:27AM -0800, Laura Abbott wrote: > __pa_symbol is the correct API to find the physical address of symbols. > Switch to it to allow for debugging APIs to work correctly. Other > functions such as p*d_populate may call __pa internally. Ensure that the > address passed is

Re: [PATCHv4 08/10] mm/kasan: Switch to using __pa_symbol and lm_alias

On Tue, Nov 29, 2016 at 10:55:27AM -0800, Laura Abbott wrote: > __pa_symbol is the correct API to find the physical address of symbols. > Switch to it to allow for debugging APIs to work correctly. Other > functions such as p*d_populate may call __pa internally. Ensure that the > address passed is

Re: [PATCH v2 2/2] vfio iommu: Fix size argument to vfio_find_dma() in pin_pages/unpin_pages

On Tue, 6 Dec 2016 22:43:45 +0530 Kirti Wankhede wrote: > vfio_dma keeps track of address range from (dma->iova + 0) to > (dma->iova + dma->size - 1), while vfio_find_dma() search logic looks for > range from (dma->iova + 1) to (dma->iova + dma->size). Also not true as

Re: [PATCH v2 2/2] vfio iommu: Fix size argument to vfio_find_dma() in pin_pages/unpin_pages

On Tue, 6 Dec 2016 22:43:45 +0530 Kirti Wankhede wrote: > vfio_dma keeps track of address range from (dma->iova + 0) to > (dma->iova + dma->size - 1), while vfio_find_dma() search logic looks for > range from (dma->iova + 1) to (dma->iova + dma->size). Also not true as described in reply to

<    4   5   6   7   8   9   10   11   12   13   >