with extended GigaMAC
reigisters have the same problem, I meet it in 8168e-vl with
RTL_GIGA_MAC_VER_34, so I make this patch just for it.]
Signed-off-by: Wang YanQing
---
drivers/net/ethernet/realtek/r8169.c | 24
1 file changed, 24 insertions(+)
diff --git a/drive
On Fri, Nov 30, 2012 at 07:35:00AM +0100, Francois Romieu wrote:
> Which kernel version is it ?
I have done the test and debug with mainline
e23739b4ade80a3a7f87198f008f6c44a7cbc9fd, v3.7-rc7-51-ge23739b
> I'd rather see the GigaMAC registers written through a call to
> rtl_rar_set when the mac
call to
rtl_rar_set when the mac address is read in rtl_init_one instead of
duplicating most of rtl_rar_set in a quite different place.
Signed-off-by: Wang YanQing
---
drivers/net/ethernet/realtek/r8169.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/net/ethernet/realtek/r8169.c
b/d
On Fri, Nov 30, 2012 at 05:04:29PM +0800, Wang YanQing wrote:
> On Fri, Nov 30, 2012 at 07:35:00AM +0100, Francois Romieu wrote:
> > Which kernel version is it ?
> I have done the test and debug with mainline
> e23739b4ade80a3a7f87198f008f6c44a7cbc9fd, v3.7-rc7-51-ge23739b
More
r Broadcom Corporation BCM43225 802.11b/g/n
Signed-off-by: Wang YanQing
---
net/wireless/Kconfig | 22 +-
1 file changed, 17 insertions(+), 5 deletions(-)
diff --git a/net/wireless/Kconfig b/net/wireless/Kconfig
index 16d08b3..6e83f0a 100644
--- a/net/wireless/Kconfig
+++ b/net/wirele
On Fri, Apr 05, 2013 at 10:51:30AM +1100, Julian Calaby wrote:
> Just as a bit of explanation for Johannes' NACK:
>
> 1. The only reason lib80211 still exists is because a couple of
> in-tree drivers still use it. If this weren't the case, the code would
> have been removed a long time ago as it's
IPI mask"))
return;
then the problem 83d349f3 fix will happen again.
Signed-off-by: Wang YanQing
---
kernel/smp.c | 10 +-
1 file changed, 9 insertions(+), 1 deletion(-)
diff --git a/kernel/smp.c b/kernel/smp.c
index 29dd40a..7c56aba 100644
--- a/kernel/smp.c
+++ b/kernel/s
platform_device_alloc could failed and return NULL,
we should check this before call platform_device_put.
Signed-off-by: Wang YanQing
---
drivers/video/uvesafb.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/video/uvesafb.c b/drivers/video/uvesafb.c
index 2f8f82d
c7b798525b50256c8084215a139fa40b0114bfcc
[smp: Fix SMP function call empty cpu mask race]
use the un-initialized variable cpumask_ipi when
enable CONFIG_CPUMASK_OFFSTACK.
Signed-off-by: Wang YanQing
---
I am sorry for miss it, I just think it when I
was lying on the bed last night
c7b798525b50256c8084215a139fa40b0114bfcc
[smp: Fix SMP function call empty cpu mask race]
use the un-initialized variable cpumask_ipi when
enable CONFIG_CPUMASK_OFFSTACK.
Signed-off-by: Wang YanQing
---
I am sorry for miss it first.
kernel/smp.c | 4
1 file changed, 4 insertions(+)
diff
c7b798525b50256c8084215a139fa40b0114bfcc
[smp: Fix SMP function call empty cpu mask race]
use the un-initialized variable cpumask_ipi when
enable CONFIG_CPUMASK_OFFSTACK.
Signed-off-by: Wang YanQing
---
I am sorry for miss it first.
kernel/smp.c | 4
1 file changed, 4 insertions(+)
diff
On Mon, Jan 28, 2013 at 09:25:55AM +, Jan Beulich wrote:
> But the patch is obviously incomplete for the CPUMASK_OFFSTACK
> case, as the newly added cpumask_ipi member never gets
> its bit array allocated.
>
> Jan
Yes, I have found it, and the patch is in lkml some hours ago.
Thanks.
--
To un
The linux/iommu.h header uses ERR_PTR defined
in linux/err.h but doesn't include it.
Reviewed-by: Alex Williamson
Signed-off-by: Wang YanQing
---
I send this twice to joerg.roe...@amd.com more than one
month, but it had been ignored, I don't why.
include/linux/iommu.h | 1 +
1 fi
commit f9f23a77f07506a32d9dc1d925bf85c0e7507b66(gma500: remove no_fb bits)
remove all the drm_psb_no_fb relations code in gma500 except this line code,
so remove it also.
Signed-off-by: Wang YanQing
---
drivers/gpu/drm/gma500/psb_drv.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers
er save config, and then use "%" or fg to
resume menuconfig and go on tuning.
Signed-off-by: Wang YanQing
---
scripts/kconfig/lxdialog/menubox.c | 7 +--
scripts/kconfig/mconf.c| 32 +++-
2 files changed, 36 insertions(+), 3 deletions(-)
line.
>
> Furthermore, the help texts now should fit in 80 columns again when viewed
> in mconf.
>
> Signed-off-by: Martin Walch
Reviewed-by: Wang YanQing
Thanks, I like it. :)
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a m
at the second type
> replaces the first type.
>
> This patch clarifies this by changing the warning to
>
> ignoring type redefinition of from to
>
> Signed-off-by: Martin Walch
Acked-by: Wang YanQing
--
To unsubscribe from this list: send the line "unsubscribe li
> Signed-off-by: Martin Walch
Acked-by: Wang YanQing
Thanks.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
r old jump key label in
current implementation, I can't image out any case
which a search result could change its jump key label,
unless user search new string, but then we will re-assign
jump key label, and it will be fix after that.
Signed-off-by: Wang YanQing
---
scripts/kconfig/mconf.c |
On Sun, Oct 06, 2013 at 01:57:15AM +0800, Wang YanQing wrote:
> In function update_text we assign jump key label
> per page (1)-(9) cycled, and use three space char
> as the header after k exceed JUMP_NB.
>
> We don't need to call memcpy with header of three
> space c
On Mon, Jul 08, 2013 at 04:50:24PM +0800, Chen Gang wrote:
> When failure occurs in hotplug_cfd(), need release related resources,
> or will cause memory leak.
>
> Also beautify the related code.
>
> Signed-off-by: Chen Gang
> ---
> kernel/smp.c | 13 +
> 1 files changed, 9 insert
free_cpumask_var(cfd->cpumask);
> return notifier_from_errno(-ENOMEM);
> }
> --
> 1.7.7.6
Acked-by: Wang YanQing
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vge
gt; Later commit 9a46ad6d6df3 "smp: make smp_call_function_many() use logic
> similar to smp_call_function_single()" fixed the same issue in another
> way when optimizing smp_call_function_many(), which then obsoletes
> changes introduced by commit f44310b98ddb7. So revert it.
&g
On Sat, Sep 14, 2013 at 12:41:36PM +0800, Wang YanQing wrote:
> On Thu, Sep 12, 2013 at 12:07:05AM +0800, Jiang Liu wrote:
> > From: Jiang Liu
> >
> > Commit f44310b98ddb7 "smp: Fix SMP function call empty cpu mask race"
> > introduced field call_functio
else
tracing_reset(&tr->max_buffer, iter->cpu_file);
}
break;
}
It does nothing if it isn't allocated.
Perhaps we need it to say "(but does not allocate or free)"
-- Steve
Signed-off-by: Wang YanQing
---
I think
gt;lock, flags);
> }
>
Your optimization don't need to keep cpumask_ipi, just clear cfd->cpumask,
and test whether cfd->cpumask is empty after "for_each_cpu(cpu, cfd->cpumask)".
Acked-by: Wang YanQing
Thanks.
--
To unsubscribe from this list: send the
apshot"
should looks like below:
" echo 2 > snapshot : Clears snapshot buffer (but does not free)
(Doesn't have to be '2' works with any number that
is not a '0' or '1')"
Signed-off-by: Wang YanQing
-
er save config, and then
use "%" or fg to resume menuconfig and go on tuning.
Changes v1-v2:
* Rewrite the most code to make it more correct
Signed-off-by: Wang YanQing
---
scripts/kconfig/lxdialog/menubox.c | 17
scripts/kconfig/mconf.c| 40 +
er save config, and then use "%" or fg to
resume menuconfig and go on tuning.
v2: Rewrite the most code to make it more correct
v3: Fix the behavior of conf_message_callback when exit.
Signed-off-by: Wang YanQing
---
scripts/kconfig/lxdialog/menubox.c | 17 +++---
s
On Mon, Oct 08, 2012 at 08:03:40AM +0200, Borislav Petkov wrote:
> Well, there are a bunch of patches from Benjamin Poirier adding the
> possibility to jump to search results and you could use them to go to
> your desired submenu.
First there are two ways to save the .config in menuconfig
1:Exit me
On Mon, Oct 08, 2012 at 10:28:32AM +0200, Borislav Petkov wrote:
> On Mon, Oct 08, 2012 at 03:22:49PM +0800, Wang YanQing wrote:
> > With one terminal, I can do:
> > 1: make menuconfig
> > 2: ajust and use "Save" button to save .config
> > 3: use CRTL-Z(konso
call to
rtl_rar_set when the mac address is read in rtl_init_one instead of
duplicating most of rtl_rar_set in a quite different place.
V2-V3:
1:Add conditon code to around this fix, because it make no sense for
most NIC
2:Add comment in code
Signed-off-by: Wang YanQing
---
drivers/net/et
On Fri, Nov 30, 2012 at 06:00:02PM +0800, Wang YanQing wrote:
> I found the mainline's r8169 works the same as realtek's driver,
> the first time don't work, but it works after reboot, the reason
> is the pci driver's shutdown don't call rtl_rar_set in 3.0 s
configuration file with Save button.
Signed-off-by: Wang YanQing
---
scripts/kconfig/lxdialog/menubox.c | 21 +-
scripts/kconfig/mconf.c| 45 ++
2 files changed, 47 insertions(+), 19 deletions(-)
diff --git a/scripts/kconfig/lxdialog
On Sat, Dec 01, 2012 at 12:44:01PM +0100, Francois Romieu wrote:
> Wang YanQing :
> > + /*
> > +*This is a fix for BIOS forget to set
> > +*extend GigaMAC registers
> > +*Wang YanQing 12/1/2012
> > +*/
>
> This part will go into the change
This patch fix it.
Signed-off-by: Wang YanQing
---
scripts/kconfig/lxdialog/inputbox.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/scripts/kconfig/lxdialog/inputbox.c
b/scripts/kconfig/lxdialog/inputbox.c
index dd8e587..7841f46 100644
--- a/scripts/kconfig/lxdialog/
On Sun, Dec 02, 2012 at 06:32:43PM +0100, Yann E. MORIN wrote:
> Wang, All,
>
> On Sunday 02 December 2012 Wang YanQing wrote:
> > inputbox code don't support use Left/Right to change input
> > position in the input box, so it use continue to skip them
> > when
w.
>
> > V1-V2: Rewrite the most code to make it more correct
> > V2-V3: Fix the behavior of conf_message_callback when exit.
> > V3-V4:
> > 1: Move Buttons a little left to make them look like symmetrical.
> > 2: Exchange the position between Save button a
space while at it for below line:
"*) Formerly when I used Page Down and Page Up, the cursor would be set"
]
Changes:
V1-V2:
1:use PATH_MAX instead of hard code suggested by Yann E. MORIN
2:drop the spurious empty-line removal suggested by Yann E. MORIN
Signed-off-by: Wang YanQing
--
ition well centered reported by Yann E. MORIN
Signed-off-by: Wang YanQing
---
Hi Yann E. MORIN, can you give your Rev'ed-by and Tested-by to his patch,
Thanks very much for your help, I just don't know how to computer out the
right position :)
The Changes appear in commit log is ok, I foun
ition well centered reported by Yann E. MORIN
Signed-off-by: Wang YanQing
---
Hi Yann E. MORIN, can you give your Rev'ed-by and Tested-by to this patch,
Thanks very much for your help, I just don't know how to computer out the
right position :)
The Changes appear in commit log is ok,
On Mon, Dec 17, 2012 at 07:19:07PM +0100, Yann E. MORIN wrote:
> Wang, All,
>
> On Monday 17 December 2012 Wang YanQing wrote:
> > dialog.h has two line the same below:
> > extern char dialog_input_result[];
> > This patch remove one of them.
> >
> > Signed
The linux/iommu.h header uses ERR_PTR defined
in linux/err.h but doesn't include it.
Cc:j...@8bytes.org
Reviewed-by: Alex Williamson
Signed-off-by: Wang YanQing
---
This patch has been in LKML for more than 2 months,
I even forget it, but the compile error when I compile
a kernel modul
On Mon, Apr 15, 2013 at 07:47:38PM -0700, Yinghai Lu wrote:
> On Mon, Apr 15, 2013 at 6:44 PM, Wang YanQing wrote:
> >
> > The parameter of memblock_reserve is start address,
> > and size, not address range.
> >
> > Signed-off-by: Wang YanQing
> > ---
>
This patch convert __init to __init_memblock
for functions which make reference to memblock variable
with attribute __meminitdata.
Signed-off-by: Wang YanQing
---
mm/memblock.c | 24
1 file changed, 12 insertions(+), 12 deletions(-)
diff --git a/mm/memblock.c b/mm
On Wed, Jun 12, 2013 at 10:29:17AM -0700, Yinghai Lu wrote:
> On Wed, Jun 12, 2013 at 9:08 AM, Wang YanQing wrote:
> >
> > This patch convert __init to __init_memblock
> > for functions which make reference to memblock variable
> > with attribute __meminitdata.
>
&
changelog
> * Add a comment above the block of new definitions ]
>
> Signed-off-by: Sedat Dilek
Acked-by: Wang YanQing
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at
On Tue, Jun 04, 2013 at 10:13:18PM +0200, Pavel Machek wrote:
> On Tue 2013-05-21 13:15:12, Wang YanQing wrote:
> > Impact:
> > 1:convert all remain take_over_console to do_take_over_console
>
> This is step backwards.
What is step backwards? do_take_over_console appear MUCH
On Thu, Jun 06, 2013 at 01:47:46PM +0200, Pavel Machek wrote:
> On Thu 2013-06-06 09:23:13, Wang YanQing wrote:
> > On Tue, Jun 04, 2013 at 10:13:18PM +0200, Pavel Machek wrote:
> > > On Tue 2013-05-21 13:15:12, Wang YanQing wrote:
> > > > Impact:
> > > >
On Thu, May 30, 2013 at 09:18:43AM +0800, Chen Gang wrote:
>
> 'boot_args' is an input args, and 'boot_command_line' has a fix length.
>
> So need use strlcpy() instead of strcpy() to avoid memory overflow.
>
>
> Signed-off-by: Chen Gang
> ---
> arch/parisc/kernel/setup.c |3 ++-
> 1 file
can get better performance,
this patch add support to do merge directly in memblock_insert_region.
Signed-off-by: Wang YanQing
---
mm/memblock.c | 87 +--
1 file changed, 42 insertions(+), 45 deletions(-)
diff --git a/mm/memblock.c b/mm
On Mon, Jul 01, 2013 at 12:14:45PM -0400, Joseph Salisbury wrote:
> Hi Wang,
>
> A bug was opened against the Ubuntu kernel[0]. After a kernel bisect,
> it was found that reverting the following commit resolved this bug:
>
> commit 8d2f8cd424ca0b99001f3ff4f5db87c4e525f366
>
On Fri, Jul 05, 2013 at 09:57:01PM +0530, Preeti U Murthy wrote:
> cfd->cpumask_ipi is used only in smp_call_function_many().The existing
> comment around it says that this additional mask is used because
> cfd->cpumask can get overwritten.
>
> There is no reason why the cfd->cpumask can be overwr
On Fri, Jul 05, 2013 at 09:57:21PM +0530, Preeti U Murthy wrote:
> call_single_data is always locked by all callers of
> arch_send_call_function_single_ipi() or
> arch_send_call_function_ipi_mask() which results in execution of
> generic_call_function_interrupt() handler.
>
> Hence remove the chec
On Sat, Jul 06, 2013 at 10:59:39AM +0530, Preeti U Murthy wrote:
> Hi Wang,
>
> On 07/06/2013 08:43 AM, Wang YanQing wrote:
> > On Fri, Jul 05, 2013 at 09:57:01PM +0530, Preeti U Murthy wrote:
> >> cfd->cpumask_ipi is used only in smp_call_function_many().The existing
On Fri, Jul 05, 2013 at 09:57:11PM +0530, Preeti U Murthy wrote:
> Elaborate on when deadlocks can occur when a call is made to
> smp_call_function_single() and its friends. This avoids ambiguity about
> when to use these calls.
>
> Signed-off-by: Preeti U Murthy
> Cc: Peter Zijlstra
> Cc: Ingo
On Sat, Jul 06, 2013 at 01:36:27PM +0530, Preeti U Murthy wrote:
> Ideally it should be under a WARN_ON(). csd_unlock() has that WARN_ON().
> Unlocking a parameter which is not locked should be seen as a bug, which
> the above code is not doing. In fact it avoids it being reported as a bug.
Althou
On Fri, Jul 05, 2013 at 03:50:57PM +0200, Thomas Gleixner wrote:
> On Sat, 16 Feb 2013, Chuansheng Liu wrote:
> > There is a real case for softirq DEADLOCK case:
> >
> > CPUACPUB
> > spin_lock(&spinlock)
> >
On Fri, Jul 05, 2013 at 04:37:14PM +0200, Thomas Gleixner wrote:
> Hmm, even there it matters, because of the following scenario:
>
> CPU 0
> smp_call_function_single(CPU 1)
> csd_lock(CPU 1)
No, smpl_call_function_single(CPU 1)
will csd_lock(CPU 0), not csd_lock(CPU 1)
> irq_enter()
>
On Sun, Jul 07, 2013 at 09:53:48PM +0530, Preeti U Murthy wrote:
> > "
> > /*
> >
> > * Unlocked CSDs are valid through generic_exec_single():
> >
> > */
>
will over write all the information
in early_cpu_init, and before early_cpu_init,
there is no user of new_cpu_data, but this is
another patch in the future.
Signed-off-by: Wang YanQing
---
arch/x86/kernel/head_32.S | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/x86/kernel
On Thu, Jun 27, 2013 at 12:54:35PM +0200, Borislav Petkov wrote:
> On Thu, Jun 27, 2013 at 12:26:40AM +0800, Wang YanQing wrote:
> >
> > We should set X86 to 486 before use cpuid
> > to detect the cpu type, if we set X86 to 486
> > after cpuid, then we will get 486 for
We should set X86 to 486 before use cpuid
to detect the cpu type, if we set X86 to 486
after cpuid, then we will get 486 until
cpu_detect runs.
Signed-off-by: Wang YanQing
---
ChangeLog
v1-v2:
1:Use more accurate and short commit log
arch/x86/kernel/head_32.S | 2 +-
1 file changed, 1
01:08.0 Communication controller: NetMos Technology PCI 9835 Multi-I/O
Controller (rev 01)
Subsystem: Device [1000:0012]
Control: I/O+ Mem+ BusMaster- SpecCycle- MemWINV- VGASnoop- ParErr-
Stepping- SERR- FastB2B- DisINTx-
Status: Cap- 66MHz- UDF- FastB2B+ ParErr- DEVSEL=m
it is harmless for current code, we will set
UART_CAP_RTOIE if it is XScale port.
Signed-off-by: Wang YanQing
---
drivers/tty/serial/8250/8250.c | 18 ++
1 file changed, 14 insertions(+), 4 deletions(-)
diff --git a/drivers/tty/serial/8250/8250.c b/drivers/tty/serial/8250/8250.c
index 0e
On Fri, Mar 01, 2013 at 02:17:16PM -0500, Paul Gortmaker wrote:
> On 13-03-01 12:56 AM, Wang YanQing wrote:
> > Some UARTs add enhanced functions with unused bit in
> > 16550 standard, like UART_IER_UUE bit, it cause XScale
>
> Which xscale platform? It would be nice to kn
This patch fix a problem XScale port
detect code in autoconfig_16550a treat
16550A port as XScale port, caused by
CH352 device use the 6th bit(UART_IER_UUE)
in IER as a enhance function make this bit
w/r.
for the below device:
"
01:08.0 Serial controller: Device 4348:3253 (rev 10) (prog-if 02 [165
On Sat, Mar 02, 2013 at 05:45:55PM +0800, Wang YanQing wrote:
> This patch fix a problem XScale port
> detect code in autoconfig_16550a treat
> 16550A port as XScale port, caused by
> CH352 device use the 6th bit(UART_IER_UUE)
> in IER as a enhance function make this bit
> w/r.
or fixing the 352 as well.
Signed-off-by: Wang YanQing
Signed-off-by: Paul Gortmaker
---
Changes
V1-V2:
1: Use a better descriptive subject by Paul Gortmaker
2: Add more detail log by Paul Gortmaker
I resend this version, because it will make merge job easy, and I
will not owe gregkh jui
I get a Multi-I/O Controller below:
01:08.0 Communication controller: NetMos Technology PCI 9835 Multi-I/O
Controller (rev 01)
Subsystem: LSI Logic / Symbios Logic 1P2S
Control: I/O+ Mem+ BusMaster- SpecCycle- MemWINV- VGASnoop- ParErr-
Stepping- SERR- FastB2B- DisINTx-
St
iles included
in header.S.
Signed-off-by: Wang YanQing
---
Changes:
V1-V2:
1:add more detail and long long log
arch/x86/boot/header.S | 4
1 file changed, 4 deletions(-)
diff --git a/arch/x86/boot/header.S b/arch/x86/boot/header.S
index 9ec06a1..43a3502 100644
--- a/arch/x86/boot/header.S
+++
The linux/iommu.h header uses ERR_PTR defined
in linux/err.h but doesn't include it.
Reviewed-by: Alex Williamson
Signed-off-by: Wang YanQing
---
I send this twice to joerg.roe...@amd.com more than one
month, but it had been ignored, I don't why.
include/linux/iommu.h | 1 +
1 fi
This patch fix the below compilation warning:
linux/drivers/net/ethernet/broadcom/tg3.c: In function
'tg3_hwclock_to_timestamp':
linux/drivers/net/ethernet/broadcom/tg3.c:5686: warning: integer constant is
too large for 'long' type
Signed-off-by: Wang YanQing
---
drivers/n
Fix the follwing warnings:
linux/drivers/ata/pata_pdc2027x.c:66: warning: 'pdc2027x_reinit_one' declared
'static' but never defined
pdc2027x_reinit_one are defined only when CONFIG_PM is defined,
thus making it conditional.
Signed-off-by: Wang YanQing
---
drivers/ata/pata_
Signed-off-by: Wang YanQing
---
drivers/dma/timb_dma.c | 2 +-
drivers/mfd/omap-usb-host.c| 2 +-
drivers/mfd/twl4030-madc.c | 2 +-
drivers/mmc/host/wmt-sdmmc.c | 2 +-
drivers/spi/spi-atmel.c| 2 +-
drivers/tty/serial/xilinx_uartps.c | 2 +-
dri
59.9
640x48060.0 59.9
DP1 disconnected (normal left inverted right x axis y axis)
Signed-off-by: Wang YanQing
---
drivers/gpu/drm/drm_edid.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index dfd9ed3..
On Mon, May 06, 2013 at 05:28:32PM +0200, Jean Delvare wrote:
> Le Monday 06 May 2013 à 15:02 +0200, Yann E. MORIN a écrit :
> > From: "Yann E. MORIN"
> >
> > When searching for symbols, return the symbols sorted by relevance.
> >
> > Relevance is the ratio of the length of the matched string an
ent in azx_probe.
This will cause Oops due dereference NULL pointer. I meet it, good luck:)
Signed-off-by: Wang YanQing
---
sound/pci/hda/hda_codec.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/sound/pci/hda/hda_codec.c b/sound/pci/hda/hda_codec.c
index 622f726..9c76752 100644
--- a/sou
with vim in ssh, and also see below bug report:
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=465823
Signed-off-by: Wang YanQing
---
drivers/tty/tty_ioctl.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/tty/tty_ioctl.c b/drivers/tty/tty_ioctl.c
index e4455e0..42e08
On Tue, May 07, 2013 at 11:58:00AM -0700, Greg KH wrote:
> On Wed, May 08, 2013 at 02:47:34AM +0800, Wang YanQing wrote:
> > The reason is before emacs takeover control the tty,
> > we use CTRL-S to XOFF it. Then when emacs takeover the
> > control, it may don't use the f
On Tue, May 07, 2013 at 04:50:05PM -0400, Peter Hurley wrote:
> This should be fixed in n_tty_set_termios() instead of fixing userspace
> workarounds.
Sorry, I misuse the word "workaround", emacs just do the right thing I think.
But your suggestion maybe import policy.
> The problem occurs when t
On Tue, May 07, 2013 at 07:02:00PM -0700, Greg KH wrote:
> What about Peter's comments on this patch?
>
Peter's comments will import policy,
I means we should let userspace to decide whether
and when to restart tty with the mechanism of TCXONC
instead of restart tty accidental when make n_tty_set
On Wed, May 08, 2013 at 11:18:07AM -0400, Peter Hurley wrote:
> On 05/08/2013 09:16 AM, Wang YanQing wrote:
> > On Tue, May 07, 2013 at 07:02:00PM -0700, Greg KH wrote:
> >> What about Peter's comments on this patch?
> >>
> > Peter's comments will i
After commit 50e244cc793d511b86adea24972f3a7264cae114
(fb: rework locking to fix lock ordering on takeover)
and
commit e93a9a868792ad71cdd09d75e5a02d8067473c4e
(fb: Yet another band-aid for fixing lockdep mess)
We have two version functions implement almost the same function,
exce
and duplication.
Signed-off-by: Wang YanQing
---
drivers/video/console/fbcon.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/video/console/fbcon.c b/drivers/video/console/fbcon.c
index 3cd6759..1660644 100644
--- a/drivers/video/console/fbcon.c
+++ b/drivers/video
Now there is no place use fbcon_takeover, and
fbcon_takeover has huge duplication code with
do_fbcon_takeover, we can achieve fbcon_takeover's
function with do_fbcon_takeover easily, so we
can just delete it.
Signed-off-by: Wang YanQing
---
drivers/video/console/fbcon.c
Now there is no place use register_con_driver|take_over_console,
and we can achieve their function with do_register_con_driver|
do_take_over_console easily, so just delete them to reduce code duplication.
Signed-off-by: Wang YanQing
---
drivers/tty/vt/vt.c | 45
There is only one place use unbind_con_driver, this patch
convert it to do_unbind_con_driver too, then we can delete
unbind_con_driver to reduce code size and duplication.
Signed-off-by: Wang YanQing
---
drivers/tty/vt/vt.c | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff
There is only one place use bind_con_driver now, this patch
convert it to do_bind_con_driver too, then we can delete
bind_con_driver whos function can be replaced by do_bind_con_driver
easily to reduce code size and duplication.
Signed-off-by: Wang YanQing
---
drivers/tty/vt/vt.c | 7 +--
1
Now there is no place use bind_con_driver,
and do_bind_con_driver can achieve bind_con_driver's
function easily, so just delete it to reduce code size and
duplication.
Signed-off-by: Wang YanQing
---
drivers/tty/vt/vt.c | 11 ---
1 file changed, 11 deletions(-)
diff --git a/driver
There are only two place use unregister_con_driver now, this patch
convert them to do_unregister_con_driver too, then we can delete
unregister_con_driver whos function can be achieved with
do_unregister_con_driver
easily to reduce code size and duplication.
Signed-off-by: Wang YanQing
Now there is no place use unregister_con_driver,
and we can achieve unregister_con_driver's function
with unregister_con_driver easily, so just delete it
to reduce code size and duplication.
Signed-off-by: Wang YanQing
---
drivers/tty/vt/vt.c | 13 +
include/linux/console.h
Now there is no place use unbind_con_driver,
and we can achieve unbind_con_driver's function
with do_unbind_con_driver easily, so just delete
it to reduce code size and duplication.
Signed-off-by: Wang YanQing
---
drivers/tty/vt/vt.c | 28
include/
On Wed, May 08, 2013 at 02:32:23PM -0400, Peter Hurley wrote:
> Perhaps you misunderstood. The snippet above does indeed restart
> a tty which has been stopped via STOP_CHAR(tty) and the termios
> IXON flag cleared.
Thanks Peter, I get your meaning, and I think your suggestion
is right, not only b
report:
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=465823
Signed-off-by: Wang YanQing
---
Changes v1-v2:
1: move the fix into n_tty_set_termios instead of change TCXONC's behavior
suggested by Peter Hurley
2: rewrite some ChangeLog and use more reasonable subject.
drive
report:
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=465823
Signed-off-by: Wang YanQing
---
Changes v1-v2:
1: move the fix into n_tty_set_termios instead of change TCXONC's behavior
suggested by Peter Hurley
2: rewrite some ChangeLog and use more reasonable subject.
Changes v2-v3:
On Mon, May 06, 2013 at 11:15:31PM +0200, Yann E. MORIN wrote:
> From: "Yann E. MORIN"
>
> When searching for symbols, return the symbols sorted by relevance.
>
> Sorting is done as thus:
> - first, symbols with a prompt, [1]
> - then, smallest offset, [2]
> - then, shortest mat
On Thu, May 09, 2013 at 06:12:17PM +0200, Yann E. MORIN wrote:
> Wang, All,
>
> On Thu, May 09, 2013 at 11:27:31PM +0800, Wang YanQing wrote:
> > On Mon, May 06, 2013 at 11:15:31PM +0200, Yann E. MORIN wrote:
> > > From: "Yann E. MORIN"
> > >
> >
Use memblock_is_region_reserved instead of memblock_find_in_range
to simplify the check codes, and gain a little speed benefit.
Signed-off-by: Wang YanQing
---
arch/x86/kernel/setup.c | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/arch/x86/kernel/setup.c b/arch/x86
This patch don't change code function,
it just do prepare for import the
memblock_alloc_in_range to do the code pattern
which looks like below in one function:
memblock_find_in_range()
follow by
memblock_reserve()
Signed-off-by: Wang YanQing
---
arch/x86/kernel/aperture_64.c | 4 ++--
1
1 - 100 of 277 matches
Mail list logo
