Re: Re: [Testing] Kernel with unicode tty patch

On Sunday 28 April 2013 19:20:34 Fuminobu TAKEYAMA Wrote:
> Hi Marguerite,
> 
> I've tried the patched kernel on VirtualBox. I could successfully see
> directories with Japanese names by ls.
> 
> It is great work though I think we should use another font for Japanese.
> It will be nice if this patch is pulled into the kernel.
> 
> Thanks,
> Fuminobu TAKEYAMA
> 
> (2013/04/26 19:52), Marguerite Su wrote:
> > On Fri, Apr 26, 2013 at 6:28 PM, Takashi Iwai <[email protected]> wrote:
> >> Just took a quick glance at the patch (not tried yet), I see that the
> >> amount of changes isn't that big, less than I was afraid.  But, it's
> >> intrusive changes and doesn't look like keeping compatible with the
> >> old code (e.g. the buffer is always allocated doubly after the
> >> patch).  And it's pretty hard to read such a "big merged patch"
> >> without any comment about the changes it does.
> > 
> > Really appreciate your help, tiwai!
> > 
> > I just used the 3.0+ patch, actually it's a "patch series",  2.6
> > patches are here:
> > 
> > http://blog.chinaunix.net/uid-436750-id-2123586.html
> > 
> > And his coding style is really...well, maybe he didn't mean to
> > upstream it since so many years passed.
> > 
> > As he said:
> > 
> > This patch was done in 2006, with kernel 2.6.17, and used on my
> > "self-designed" distribution "Olive"
> > 
> >> In general, for openSUSE kernel, we want to avoid wild patches as much
> >> as possible.  So, for any new patches, "upstream first" rule wins.
> >> IIRC, there have been multiple attempts for UTF8 wide-char support on
> >> fbcon, but they all failed in the upstream.
> > 
> > Agreed with our policy. It's the first patch with such functions I
> > found, so I'm un-experienced with that.
> > 
> >> Hence, I guess it's a quite long way to go in this case, too.
> >> Let's see...
> > 
> > Maybe I should ask him for code comments and efforts to upstream it
> > from his side.
> > 
> > Because I know nothing about coding, if I do that, I think Linus will
> > certainly rant at me in public...
> > 
> > 
> > Marguerite

 AFAIK this patch is kinds of doing hack on kernel tty code, and I heard
 there is some issue about font license used with this patch. (Though I
 thought there is unifont without license issue can be used, since grub2 can
 display utf8 string)

 So I guess that's also some reason this never gets mainlined.

 And I think further work maybe expected to see on
 https://github.com/dvdhrm/kmscon , while fbterm is already a not bad
 solution (and with im support).

Regards,
Xuetian Weng
-- 
To unsubscribe, e-mail: [email protected]
To contact the owner, e-mail: [email protected]